rave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Geer" <ch...@cxtsoftware.com>
Subject Review Request: Update SSO for Rave 0.14-SNAPSHOT
Date Sat, 21 Jul 2012 05:02:53 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6075/
-----------------------------------------------------------

Review request for rave.


Description
-------

Update SSO for Rave 0.14-SNAPSHOT


This addresses bug RAVE-726.
    https://issues.apache.org/jira/browse/RAVE-726


Diffs
-----

  /sandbox/rave-extensions/rave-extension-sso/pom.xml 1363240 
  /sandbox/rave-extensions/rave-extension-sso/src/main/java/org/apache/rave/portal/security/impl/DefaultSSOLoginHandler.java
1363240 
  /sandbox/rave-extensions/rave-extension-sso/src/test/java/org/apache/rave/portal/security/filter/SSORequestHeaderAuthenticationFilterTest.java
1363240 
  /sandbox/rave-extensions/rave-extension-sso/src/test/java/org/apache/rave/portal/security/impl/DefaultSSOLoginHandlerTest.java
1363240 

Diff: https://reviews.apache.org/r/6075/diff/


Testing
-------

Tested locally and ran into some issues with creating new accounts. I think it might be because
I'm instantiating a UserImpl (POJO) object and not a JPA User so I'm looking for a second
opinion on the best way to handle this (and make it work with multiple backends potentially).


Thanks,

Chris Geer


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message