rave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matt Franklin" <mfrank...@apache.org>
Subject Re: Review Request: Update SSO for Rave 0.14-SNAPSHOT
Date Sat, 28 Jul 2012 18:26:54 GMT

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


Not sure why this isn't working.  The only thing I saw is you are not setting a default page
layout...


/sandbox/rave-extensions/rave-extension-sso/src/main/java/org/apache/rave/portal/security/impl/DefaultSSOLoginHandler.java
<https://reviews.apache.org/r/6075/#comment20418>

    Why are you no longer setting the default page layout?


- Matt Franklin


On July 21, 2012, 5:02 a.m., Chris Geer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6075/
> -----------------------------------------------------------
> 
> (Updated July 21, 2012, 5:02 a.m.)
> 
> 
> 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