[jboss-cvs] jboss-tomcat/src/main/org/jboss/web/tomcat/tc5 TomcatDeployer.java

[Scott M S. <st...@us...>]

  User: starksm 
  Date: 04/05/12 13:34:42

  Modified:    src/main/org/jboss/web/tomcat/tc5 Tag: Branch_3_2
                        TomcatDeployer.java
  Log:
  Add a comment on the ordering of the security valves after the standard valves
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.12  +5 -2      jboss-tomcat/src/main/org/jboss/web/tomcat/tc5/TomcatDeployer.java
  
  Index: TomcatDeployer.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-tomcat/src/main/org/jboss/web/tomcat/tc5/TomcatDeployer.java,v
  retrieving revision 1.1.2.11
  retrieving revision 1.1.2.12
  diff -u -r1.1.2.11 -r1.1.2.12
  --- TomcatDeployer.java	12 May 2004 20:31:39 -0000	1.1.2.11
  +++ TomcatDeployer.java	12 May 2004 20:34:41 -0000	1.1.2.12
  @@ -34,7 +34,7 @@
    * 
    * @author Sco...@jb...
    * @author Costin Manolache
  - * @version $Revision: 1.1.2.11 $
  + * @version $Revision: 1.1.2.12 $
    */
   public class TomcatDeployer extends AbstractWebDeployer
   {
  @@ -266,7 +266,10 @@
         // Init the container; this will also start it
         server.invoke(objectName, "init", new Object[]{}, new String[]{});
   
  -      // Add JBoss realm as valve
  +      /* Add JBoss realm as valve after the externalized container valves so
  +       since this and the SecurityAssociationValve should be after any
  +       authentication/authorization side effects
  +      */
         server.invoke(objectName, "addValve",
            new Object[]{jbossRealm},
            new String[]{"org.apache.catalina.Valve"});