Author: markt Date: Mon Sep 3 17:58:46 2018 New Revision: 1839977 URL: http://svn.apache.org/viewvc?rev=1839977&view=rev Log: Remove / deprecate code originally added to enable the CGI Servlet to work as an include with SSI. The CGI servlet has generic handling for the include case and this flag is no longer required.
Modified: tomcat/trunk/java/org/apache/catalina/Globals.java tomcat/trunk/java/org/apache/catalina/ssi/SSIFilter.java tomcat/trunk/java/org/apache/catalina/ssi/SSIServlet.java Modified: tomcat/trunk/java/org/apache/catalina/Globals.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/Globals.java?rev=1839977&r1=1839976&r2=1839977&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/Globals.java (original) +++ tomcat/trunk/java/org/apache/catalina/Globals.java Mon Sep 3 17:58:46 2018 @@ -118,9 +118,13 @@ public final class Globals { * We do this because of the pathInfo mangling happening when using * the CGIServlet in conjunction with the SSI servlet. (value stored * as an object of type String) + * + * @deprecated Unused. This is no longer used as the CGIO servlet now has + * generic handling for when it is used as an include. + * This will be removed in Tomcat 10 */ - public static final String SSI_FLAG_ATTR = - "org.apache.catalina.ssi.SSIServlet"; + @Deprecated + public static final String SSI_FLAG_ATTR = "org.apache.catalina.ssi.SSIServlet"; /** Modified: tomcat/trunk/java/org/apache/catalina/ssi/SSIFilter.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/SSIFilter.java?rev=1839977&r1=1839976&r2=1839977&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/ssi/SSIFilter.java (original) +++ tomcat/trunk/java/org/apache/catalina/ssi/SSIFilter.java Mon Sep 3 17:58:46 2018 @@ -35,8 +35,6 @@ import javax.servlet.ServletRequest; import javax.servlet.ServletResponse; import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; - -import org.apache.catalina.Globals; /** * Filter to process SSI requests within a webpage. Mapped to a content types * from within web.xml. @@ -92,9 +90,6 @@ public class SSIFilter extends GenericFi HttpServletRequest req = (HttpServletRequest)request; HttpServletResponse res = (HttpServletResponse)response; - // indicate that we're in SSI processing - req.setAttribute(Globals.SSI_FLAG_ATTR, "true"); - // setup to capture output ByteArrayServletOutputStream basos = new ByteArrayServletOutputStream(); ResponseIncludeWrapper responseIncludeWrapper = Modified: tomcat/trunk/java/org/apache/catalina/ssi/SSIServlet.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ssi/SSIServlet.java?rev=1839977&r1=1839976&r2=1839977&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/ssi/SSIServlet.java (original) +++ tomcat/trunk/java/org/apache/catalina/ssi/SSIServlet.java Mon Sep 3 17:58:46 2018 @@ -32,8 +32,6 @@ import javax.servlet.ServletException; import javax.servlet.http.HttpServlet; import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; - -import org.apache.catalina.Globals; /** * Servlet to process SSI requests within a webpage. Mapped to a path from * within web.xml. @@ -178,7 +176,6 @@ public class SSIServlet extends HttpServ res.setDateHeader("Expires", (new java.util.Date()).getTime() + expires.longValue() * 1000); } - req.setAttribute(Globals.SSI_FLAG_ATTR, "true"); processSSI(req, res, resource); } --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org