-----------------------------------------------------------
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

Reply via email to