tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ma...@apache.org
Subject cvs commit: jakarta-tomcat-4.0/webapps/tomcat-docs cgi-howto.xml
Date Tue, 17 Aug 2004 21:44:33 GMT
markt       2004/08/17 14:44:33

  Modified:    catalina/src/conf web.xml
               catalina/src/share/org/apache/catalina/servlets
                        CGIServlet.java
               webapps/tomcat-docs cgi-howto.xml
  Log:
  Fix bug 18273. Add support for optionally passing the shell environment variables to the
CGI script.
   - Based heavily on <a href="http://www.rgagnon.com/javadetails/java-0150.html">
     Read environment variables from an application</a> by Real Gagnon
  
  Revision  Changes    Path
  1.58      +14 -12    jakarta-tomcat-4.0/catalina/src/conf/web.xml
  
  Index: web.xml
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/conf/web.xml,v
  retrieving revision 1.57
  retrieving revision 1.58
  diff -u -r1.57 -r1.58
  --- web.xml	16 Aug 2004 22:29:33 -0000	1.57
  +++ web.xml	17 Aug 2004 21:44:32 -0000	1.58
  @@ -226,19 +226,22 @@
     <!-- supports the following initialization parameters (default values     -->
     <!-- are in square brackets):                                             -->
     <!--                                                                      -->
  -  <!--   cgiPathPrefix       The CGI search path will start at              -->
  -  <!--                       webAppRootDir + File.separator + this prefix.  -->
  -  <!--                       [WEB-INF/cgi]                                  -->
  +  <!--   cgiPathPrefix        The CGI search path will start at             -->
  +  <!--                        webAppRootDir + File.separator + this prefix. -->
  +  <!--                        [WEB-INF/cgi]                                 -->
     <!--                                                                      -->
  -  <!--   debug               Debugging detail level for messages logged     -->
  -  <!--                       by this servlet.  [0]                          -->
  +  <!--   debug                Debugging detail level for messages logged    -->
  +  <!--                        by this servlet.  [0]                         -->
     <!--                                                                      -->
  -  <!--   executable          Name of the exectuable used to run the script. -->
  -  <!--                       [perl]                                         -->
  +  <!--   executable           Name of the exectuable used to run the        -->
  +  <!--                        script. [perl]                                -->
     <!--                                                                      -->
  -  <!--   parameterEncoding   Name of parameter encoding to be used with CGI -->
  -  <!--                       servlet.                                       -->
  -  <!--                       [System.getProperty("file.encoding","UTF-8")]  -->
  +  <!--   parameterEncoding    Name of parameter encoding to be used with    -->
  +  <!--                        CGI servlet.                                  -->
  +  <!--                        [System.getProperty("file.encoding","UTF-8")] -->
  +  <!--                                                                      -->
  +  <!--   passShellEnvironment Should the shell environment variables (if    -->
  +  <!--                        any) be passed to the CGI script? [false]     -->
     <!--                                                                      -->
     <!-- IMPORTANT: To use the CGI servlet, you also need to rename the       -->
     <!--            $CATALINA_HOME/server/lib/servlets-cgi.renametojar file   -->
  @@ -256,10 +259,9 @@
             <param-name>cgiPathPrefix</param-name>
             <param-value>WEB-INF/cgi</param-value>
           </init-param>
  -         <load-on-startup>5</load-on-startup>
  +        <load-on-startup>5</load-on-startup>
       </servlet>
   -->
  -
   
     <!-- ================ Built In Servlet Mappings ========================= -->
   
  
  
  
  1.26      +69 -9     jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/servlets/CGIServlet.java
  
  Index: CGIServlet.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/servlets/CGIServlet.java,v
  retrieving revision 1.25
  retrieving revision 1.26
  diff -u -r1.25 -r1.26
  --- CGIServlet.java	16 Aug 2004 22:29:33 -0000	1.25
  +++ CGIServlet.java	17 Aug 2004 21:44:32 -0000	1.26
  @@ -315,6 +315,8 @@
       /** object used to ensure multiple threads don't try to expand same file */
       static Object expandFileLock = new Object();
   
  +    /** the shell environment variables to be passed to the CGI script */
  +    static Hashtable shellEnv = new Hashtable();
   
       /**
        * Sets instance variables.
  @@ -343,6 +345,8 @@
               throw new UnavailableException
                   ("Cannot invoke CGIServlet through the invoker");
   
  +        boolean passShellEnvironment = false;
  +        
           // Set our properties from the initialization parameters
           String value = null;
           try {
  @@ -350,11 +354,22 @@
               debug = Integer.parseInt(value);
               cgiPathPrefix =
                   getServletConfig().getInitParameter("cgiPathPrefix");
  +            value = getServletConfig().getInitParameter("passShellEnvironment");
  +            passShellEnvironment = Boolean.valueOf(value).booleanValue();
           } catch (Throwable t) {
               //NOOP
           }
           log("init: loglevel set to " + debug);
   
  +        if (passShellEnvironment) {
  +            try {
  +                shellEnv.putAll(getShellEnvironment());
  +            } catch (IOException ioe) {
  +                ServletException e = new ServletException(
  +                        "Unable to read shell environment variables", ioe);
  +            }
  +        }
  +
           value = getServletConfig().getInitParameter("executable");
           if (value != null) {
               cgiExecutable = value;
  @@ -682,8 +697,49 @@
           System.out.println("$Header$");
       }
   
  -
  -
  +    /**
  +     * Get all shell environment variables. Have to do it this rather ugly way
  +     * as the API to obtain is not available in 1.4 and earlier APIs.
  +     *
  +     * See <a href="http://www.rgagnon.com/javadetails/java-0150.html">Read environment
  +     * variables from an application</a> for original source and article.
  +     */
  +    private Hashtable getShellEnvironment() throws IOException {
  +        Hashtable envVars = new Hashtable();
  +        Process p = null;
  +        Runtime r = Runtime.getRuntime();
  +        String OS = System.getProperty("os.name").toLowerCase();
  +        boolean ignoreCase;
  +
  +        if (OS.indexOf("windows 9") > -1) {
  +            p = r.exec( "command.com /c set" );
  +            ignoreCase = true;
  +        } else if ( (OS.indexOf("nt") > -1)
  +                || (OS.indexOf("windows 2000") > -1)
  +                || (OS.indexOf("windows xp") > -1) ) {
  +            // thanks to JuanFran for the xp fix!
  +            p = r.exec( "cmd.exe /c set" );
  +            ignoreCase = true;
  +        } else {
  +            // our last hope, we assume Unix (thanks to H. Ware for the fix)
  +            p = r.exec( "env" );
  +            ignoreCase = false;
  +        }
  +
  +        BufferedReader br = new BufferedReader
  +            ( new InputStreamReader( p.getInputStream() ) );
  +        String line;
  +        while( (line = br.readLine()) != null ) {
  +            int idx = line.indexOf( '=' );
  +            String key = line.substring( 0, idx );
  +            String value = line.substring( idx+1 );
  +            if (ignoreCase) {
  +                key = key.toUpperCase();
  +            }
  +            envVars.put(key, value);
  +        }
  +        return envVars;
  +    }
   
   
   
  @@ -960,6 +1016,10 @@
   
               Hashtable envp = new Hashtable();
   
  +            // Add the shell environment variables (if any)
  +            envp.putAll(shellEnv);
  +
  +            // Add the CGI environment variables
               String sPathInfoOrig = null;
               String sPathTranslatedOrig = null;
               String sPathInfoCGI = null;
  
  
  
  1.5       +3 -0      jakarta-tomcat-4.0/webapps/tomcat-docs/cgi-howto.xml
  
  Index: cgi-howto.xml
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/webapps/tomcat-docs/cgi-howto.xml,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- cgi-howto.xml	16 Aug 2004 22:29:34 -0000	1.4
  +++ cgi-howto.xml	17 Aug 2004 21:44:32 -0000	1.5
  @@ -59,6 +59,9 @@
   <li><strong>parameterEncoding</strong> - Name of the parameter encoding
   to be used with the GCI servlet. Default is
   <code>System.getProperty("file.encoding","UTF-8")</code>.</li>
  +<li><strong>passShellEnvironment</strong> - Should the shell environment
  +variables (if any) be passed to the CGI script? Default is
  +<code>false</code>.</li>
   </ul>
   </p>
   
  
  
  

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


Mime
View raw message