You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by ta...@apache.org on 2005/02/10 22:09:18 UTC

cvs commit: jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/security/impl AbstractSecurityValve.java

taylor      2005/02/10 13:09:18

  Modified:    portal/src/java/org/apache/jetspeed/security/impl
                        AbstractSecurityValve.java
  Log:
  bug in security valve, not setting subject into session
  patch from Tom Pesendorfer
  
  Revision  Changes    Path
  1.3       +3 -2      jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/security/impl/AbstractSecurityValve.java
  
  Index: AbstractSecurityValve.java
  ===================================================================
  RCS file: /home/cvs/jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/security/impl/AbstractSecurityValve.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- AbstractSecurityValve.java	21 Jan 2005 16:45:19 -0000	1.2
  +++ AbstractSecurityValve.java	10 Feb 2005 21:09:18 -0000	1.3
  @@ -99,7 +99,8 @@
               {
                  throw new PipelineException(e1.getMessage(), e1);
               }
  -            request.getRequest().setAttribute(PortalReservedParameters.SESSION_KEY_SUBJECT, subject);
  +            request.getRequest().getSession().setAttribute(PortalReservedParameters.SESSION_KEY_SUBJECT, subject);            
  +            
               // set request context subject
               request.setSubject(subject);
               
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org