This is an automated email from the ASF dual-hosted git repository.
mgrigorov pushed a commit to branch 7.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/7.0.x by this push:
new 2b44d30 Do not check for TLS request attribute more than once
2b44d30 is described below
commit 2b44d30a761fa03dc1e682bfda13e60b4eafa234
Author: Martin Tzvetanov Grigorov <mgrigo...@apache.org>
AuthorDate: Tue Aug 25 14:31:11 2020 +0300
Do not check for TLS request attribute more than once
No need to check for special TLS attribute second time if they were parsed
already.
If 'name' is a TLS attribute then it will be in the 'attributes' map.
---
java/org/apache/catalina/connector/Request.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/java/org/apache/catalina/connector/Request.java
b/java/org/apache/catalina/connector/Request.java
index d218c8d..c4fcc2a 100644
--- a/java/org/apache/catalina/connector/Request.java
+++ b/java/org/apache/catalina/connector/Request.java
@@ -979,7 +979,8 @@ public class Request implements HttpServletRequest {
if (attr != null) {
return attr;
}
- if (isSSLAttribute(name) ||
name.equals(SSLSupport.PROTOCOL_VERSION_KEY)) {
+
+ if (!sslAttributesParsed && (isSSLAttribute(name) ||
name.equals(SSLSupport.PROTOCOL_VERSION_KEY))) {
coyoteRequest.action(ActionCode.REQ_SSL_ATTRIBUTE, coyoteRequest);
attr = coyoteRequest.getAttribute(Globals.CERTIFICATES_ATTR);
if (attr != null) {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
