This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit c64b6fd21cb1e2578620502e0503369da16c3bc1 Author: Mark Thomas <ma...@apache.org> AuthorDate: Tue Mar 31 10:59:22 2020 +0100 Group constants by type to aid maintenance --- java/org/apache/catalina/Globals.java | 241 ++++++++++++++++------------------ 1 file changed, 114 insertions(+), 127 deletions(-) diff --git a/java/org/apache/catalina/Globals.java b/java/org/apache/catalina/Globals.java index e92f83b..5472e37 100644 --- a/java/org/apache/catalina/Globals.java +++ b/java/org/apache/catalina/Globals.java @@ -23,104 +23,67 @@ package org.apache.catalina; */ public final class Globals { - /** - * The servlet context attribute under which we store the alternate - * deployment descriptor for this web application - */ - public static final String ALT_DD_ATTR = - "org.apache.catalina.deploy.alt_dd"; - + // ------------------------------------------------- Request attribute names - /** - * The request attribute under which we store the array of X509Certificate - * objects representing the certificate chain presented by our client, - * if any. - */ - public static final String CERTIFICATES_ATTR = - "jakarta.servlet.request.X509Certificate"; + public static final String ASYNC_SUPPORTED_ATTR = "org.apache.catalina.ASYNC_SUPPORTED"; - /** - * The request attribute under which we store the name of the cipher suite - * being used on an SSL connection (as an object of type - * java.lang.String). - */ - public static final String CIPHER_SUITE_ATTR = - "jakarta.servlet.request.cipher_suite"; + public static final String GSS_CREDENTIAL_ATTR = "org.apache.catalina.realm.GSS_CREDENTIAL"; /** * Request dispatcher state. */ - public static final String DISPATCHER_TYPE_ATTR = - "org.apache.catalina.core.DISPATCHER_TYPE"; + public static final String DISPATCHER_TYPE_ATTR = "org.apache.catalina.core.DISPATCHER_TYPE"; /** * Request dispatcher path. */ - public static final String DISPATCHER_REQUEST_PATH_ATTR = - "org.apache.catalina.core.DISPATCHER_REQUEST_PATH"; + public static final String DISPATCHER_REQUEST_PATH_ATTR = "org.apache.catalina.core.DISPATCHER_REQUEST_PATH"; /** - * The WebResourceRoot which is associated with the context. This can be - * used to manipulate static files. - */ - public static final String RESOURCES_ATTR = - "org.apache.catalina.resources"; - - - /** - * The servlet context attribute under which we store the class path - * for our application class loader (as an object of type String), - * delimited with the appropriate path delimiter for this platform. - */ - public static final String CLASS_PATH_ATTR = - "org.apache.catalina.jsp_classpath"; - - - /** - * The request attribute under which we store the key size being used for - * this SSL connection (as an object of type java.lang.Integer). + * The request attribute under which we store the servlet name on a + * named dispatcher request. */ - public static final String KEY_SIZE_ATTR = - "jakarta.servlet.request.key_size"; + public static final String NAMED_DISPATCHER_ATTR = "org.apache.catalina.NAMED"; /** - * The request attribute under which we store the session id being used - * for this SSL connection (as an object of type java.lang.String). + * The request attribute that is set to {@code Boolean.TRUE} if some request + * parameters have been ignored during request parameters parsing. It can + * happen, for example, if there is a limit on the total count of parseable + * parameters, or if parameter cannot be decoded, or any other error + * happened during parameter parsing. */ - public static final String SSL_SESSION_ID_ATTR = - "jakarta.servlet.request.ssl_session_id"; + public static final String PARAMETER_PARSE_FAILED_ATTR = "org.apache.catalina.parameter_parse_failed"; /** - * The request attribute key for the session manager. - * This one is a Tomcat extension to the Servlet spec. + * The reason that the parameter parsing failed. */ - public static final String SSL_SESSION_MGR_ATTR = - "jakarta.servlet.request.ssl_session_mgr"; + public static final String PARAMETER_PARSE_FAILED_REASON_ATTR = "org.apache.catalina.parameter_parse_failed_reason"; /** - * The request attribute under which we store the servlet name on a - * named dispatcher request. + * The request attribute set by the RemoteIpFilter, RemoteIpValve (and may + * be set by other similar components) that identifies for the connector the + * remote IP address claimed to be associated with this request when a + * request is received via one or more proxies. It is typically provided via + * the X-Forwarded-For HTTP header. + * + * Duplicated here for neater code in the catalina packages. */ - public static final String NAMED_DISPATCHER_ATTR = - "org.apache.catalina.NAMED"; + public static final String REMOTE_ADDR_ATTRIBUTE = org.apache.coyote.Constants.REMOTE_ADDR_ATTRIBUTE; /** - * The subject under which the AccessControlContext is running. + * The request attribute that is set to the value of {@code Boolean.TRUE} + * by the RemoteIpFilter, RemoteIpValve (and other similar components) that identifies + * a request which been forwarded via one or more proxies. */ - public static final String SUBJECT_ATTR = - "javax.security.auth.subject"; - - - public static final String GSS_CREDENTIAL_ATTR = - "org.apache.catalina.realm.GSS_CREDENTIAL"; + public static final String REQUEST_FORWARDED_ATTRIBUTE = "org.apache.tomcat.request.forwarded"; /** @@ -129,8 +92,7 @@ public final class Globals { * * Duplicated here for neater code in the catalina packages. */ - public static final String SENDFILE_SUPPORTED_ATTR = - org.apache.coyote.Constants.SENDFILE_SUPPORTED_ATTR; + public static final String SENDFILE_SUPPORTED_ATTR = org.apache.coyote.Constants.SENDFILE_SUPPORTED_ATTR; /** @@ -141,8 +103,7 @@ public final class Globals { * * Duplicated here for neater code in the catalina packages. */ - public static final String SENDFILE_FILENAME_ATTR = - org.apache.coyote.Constants.SENDFILE_FILENAME_ATTR; + public static final String SENDFILE_FILENAME_ATTR = org.apache.coyote.Constants.SENDFILE_FILENAME_ATTR; /** @@ -154,8 +115,7 @@ public final class Globals { * * Duplicated here for neater code in the catalina packages. */ - public static final String SENDFILE_FILE_START_ATTR = - org.apache.coyote.Constants.SENDFILE_FILE_START_ATTR; + public static final String SENDFILE_FILE_START_ATTR = org.apache.coyote.Constants.SENDFILE_FILE_START_ATTR; /** @@ -167,97 +127,103 @@ public final class Globals { * * Duplicated here for neater code in the catalina packages. */ - public static final String SENDFILE_FILE_END_ATTR = - org.apache.coyote.Constants.SENDFILE_FILE_END_ATTR; + public static final String SENDFILE_FILE_END_ATTR = org.apache.coyote.Constants.SENDFILE_FILE_END_ATTR; /** - * The request attribute set by the RemoteIpFilter, RemoteIpValve (and may - * be set by other similar components) that identifies for the connector the - * remote IP address claimed to be associated with this request when a - * request is received via one or more proxies. It is typically provided via - * the X-Forwarded-For HTTP header. - * - * Duplicated here for neater code in the catalina packages. + * The request attribute under which we store the array of X509Certificate + * objects representing the certificate chain presented by our client, + * if any. */ - public static final String REMOTE_ADDR_ATTRIBUTE = - org.apache.coyote.Constants.REMOTE_ADDR_ATTRIBUTE; + public static final String CERTIFICATES_ATTR = "jakarta.servlet.request.X509Certificate"; /** - * The request attribute that is set to the value of {@code Boolean.TRUE} - * by the RemoteIpFilter, RemoteIpValve (and other similar components) that identifies - * a request which been forwarded via one or more proxies. + * The request attribute under which we store the name of the cipher suite + * being used on an SSL connection (as an object of type + * java.lang.String). */ - public static final String REQUEST_FORWARDED_ATTRIBUTE = - "org.apache.tomcat.request.forwarded"; + public static final String CIPHER_SUITE_ATTR = "jakarta.servlet.request.cipher_suite"; - public static final String ASYNC_SUPPORTED_ATTR = - "org.apache.catalina.ASYNC_SUPPORTED"; + /** + * The request attribute under which we store the key size being used for + * this SSL connection (as an object of type java.lang.Integer). + */ + public static final String KEY_SIZE_ATTR = "jakarta.servlet.request.key_size"; /** - * The request attribute that is set to {@code Boolean.TRUE} if some request - * parameters have been ignored during request parameters parsing. It can - * happen, for example, if there is a limit on the total count of parseable - * parameters, or if parameter cannot be decoded, or any other error - * happened during parameter parsing. + * The request attribute under which we store the session id being used + * for this SSL connection (as an object of type java.lang.String). */ - public static final String PARAMETER_PARSE_FAILED_ATTR = - "org.apache.catalina.parameter_parse_failed"; + public static final String SSL_SESSION_ID_ATTR = "jakarta.servlet.request.ssl_session_id"; /** - * The reason that the parameter parsing failed. + * The request attribute key for the session manager. + * This one is a Tomcat extension to the Servlet spec. */ - public static final String PARAMETER_PARSE_FAILED_REASON_ATTR = - "org.apache.catalina.parameter_parse_failed_reason"; + public static final String SSL_SESSION_MGR_ATTR = "jakarta.servlet.request.ssl_session_mgr"; + + // ------------------------------------------------- Session attribute names /** - * The master flag which controls strict servlet specification - * compliance. + * The subject under which the AccessControlContext is running. */ - public static final boolean STRICT_SERVLET_COMPLIANCE = - Boolean.parseBoolean(System.getProperty("org.apache.catalina.STRICT_SERVLET_COMPLIANCE", "false")); + public static final String SUBJECT_ATTR = "javax.security.auth.subject"; + + // ------------------------------------------ ServletContext attribute names /** - * Has security been turned on? + * The servlet context attribute under which we store the alternate + * deployment descriptor for this web application */ - public static final boolean IS_SECURITY_ENABLED = - (System.getSecurityManager() != null); + public static final String ALT_DD_ATTR = "org.apache.catalina.deploy.alt_dd"; /** - * Default domain for MBeans if none can be determined + * The servlet context attribute under which we store the class path + * for our application class loader (as an object of type String), + * delimited with the appropriate path delimiter for this platform. */ - public static final String DEFAULT_MBEAN_DOMAIN = "Catalina"; + public static final String CLASS_PATH_ATTR = "org.apache.catalina.jsp_classpath"; /** - * Name of the system property containing - * the tomcat product installation path + * Name of the ServletContext attribute under which we store the context + * Realm's CredentialHandler (if both the Realm and the CredentialHandler + * exist). */ - public static final String CATALINA_HOME_PROP = "catalina.home"; + public static final String CREDENTIAL_HANDLER = "org.apache.catalina.CredentialHandler"; /** - * Name of the system property containing - * the tomcat instance installation path + * The WebResourceRoot which is associated with the context. This can be + * used to manipulate static files. */ - public static final String CATALINA_BASE_PROP = "catalina.base"; + public static final String RESOURCES_ATTR = "org.apache.catalina.resources"; /** + * Name of the ServletContext attribute under which we store the web + * application version string (the text that appears after ## when parallel + * deployment is used). + */ + public static final String WEBAPP_VERSION = "org.apache.catalina.webappVersion"; + + + // --------------------------- ServletContext initialisation parameter names + + /** * Name of the ServletContext init-param that determines if the JSP engine * should validate *.tld files when parsing them. * <p> * This must be kept in sync with org.apache.jasper.Constants */ - public static final String JASPER_XML_VALIDATION_TLD_INIT_PARAM = - "org.apache.jasper.XML_VALIDATE_TLD"; + public static final String JASPER_XML_VALIDATION_TLD_INIT_PARAM = "org.apache.jasper.XML_VALIDATE_TLD"; /** @@ -267,22 +233,43 @@ public final class Globals { * <p> * This must be kept in sync with org.apache.jasper.Constants */ - public static final String JASPER_XML_BLOCK_EXTERNAL_INIT_PARAM = - "org.apache.jasper.XML_BLOCK_EXTERNAL"; + public static final String JASPER_XML_BLOCK_EXTERNAL_INIT_PARAM = "org.apache.jasper.XML_BLOCK_EXTERNAL"; + + + // --------------------------------------------------- System property names /** - * Name of the ServletContext attribute under which we store the context - * Realm's CredentialHandler (if both the Realm and the CredentialHandler - * exist). + * Name of the system property containing + * the tomcat product installation path */ - public static final String CREDENTIAL_HANDLER - = "org.apache.catalina.CredentialHandler"; + public static final String CATALINA_HOME_PROP = "catalina.home"; /** - * Name of the ServletContext attribute under which we store the web - * application version string (the text that appears after ## when parallel - * deployment is used). + * Name of the system property containing + * the tomcat instance installation path */ - public static final String WEBAPP_VERSION = "org.apache.catalina.webappVersion"; + public static final String CATALINA_BASE_PROP = "catalina.base"; + + + // -------------------------------------------------------- Global constants + + /** + * The master flag which controls strict servlet specification + * compliance. + */ + public static final boolean STRICT_SERVLET_COMPLIANCE = + Boolean.parseBoolean(System.getProperty("org.apache.catalina.STRICT_SERVLET_COMPLIANCE", "false")); + + + /** + * Has security been turned on? + */ + public static final boolean IS_SECURITY_ENABLED = (System.getSecurityManager() != null); + + + /** + * Default domain for MBeans if none can be determined + */ + public static final String DEFAULT_MBEAN_DOMAIN = "Catalina"; } --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org