remm        01/12/02 14:24:49

  Modified:    catalina/src/share/org/apache/catalina/core Tag:
                        tomcat_40_branch StandardContext.java
  Log:
  - Revert this patch (sorry Costin):
  - Port patch which makes the parent class loader of a privileged webapp be
    the Catalina loader, which gives it full access to the Catalina internals.
  - The problem is that Catalina wants to load-on-startup Jasper for all webapps,
    and Jasper is not available in the Catalina CL.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.78.2.9  +17 -11    
jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/core/StandardContext.java
  
  Index: StandardContext.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/core/StandardContext.java,v
  retrieving revision 1.78.2.8
  retrieving revision 1.78.2.9
  diff -u -r1.78.2.8 -r1.78.2.9
  --- StandardContext.java      2001/12/02 17:38:17     1.78.2.8
  +++ StandardContext.java      2001/12/02 22:24:49     1.78.2.9
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/core/StandardContext.java,v
 1.78.2.8 2001/12/02 17:38:17 remm Exp $
  - * $Revision: 1.78.2.8 $
  - * $Date: 2001/12/02 17:38:17 $
  + * $Header: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/core/StandardContext.java,v
 1.78.2.9 2001/12/02 22:24:49 remm Exp $
  + * $Revision: 1.78.2.9 $
  + * $Date: 2001/12/02 22:24:49 $
    *
    * ====================================================================
    *
  @@ -142,7 +142,7 @@
    *
    * @author Craig R. McClanahan
    * @author Remy Maucherat
  - * @version $Revision: 1.78.2.8 $ $Date: 2001/12/02 17:38:17 $
  + * @version $Revision: 1.78.2.9 $ $Date: 2001/12/02 22:24:49 $
    */
   
   public class StandardContext
  @@ -3307,15 +3307,21 @@
               }
           }
           if (getLoader() == null) {      // (2) Required by Manager
  -            if (getPrivileged()) {
  -                if (debug >= 1)
  -                    log("Configuring privileged default Loader");
  -                setLoader(new WebappLoader(this.getClass().getClassLoader()));
  +            // FIXME: All webapps require Jasper, but Jasper isn't available in
  +            // the Catalina CL.
  +            /*
  +              if (getPrivileged()) {
  +              if (debug >= 1)
  +              log("Configuring privileged default Loader");
  +              setLoader(new WebappLoader(this.getClass().getClassLoader()));
               } else {
  -                if (debug >= 1)
  -                    log("Configuring non-privileged default Loader");
  -                setLoader(new WebappLoader(getParentClassLoader()));
  +            */
  +            if (debug >= 1)
  +                log("Configuring non-privileged default Loader");
  +            setLoader(new WebappLoader(getParentClassLoader()));
  +            /*
               }
  +            */
           }
           if (getManager() == null) {     // (3) After prerequisites
               if (debug >= 1)
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to