You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by mg...@apache.org on 2020/08/27 11:52:08 UTC
[tomcat] branch 8.5.x updated: Do not check for TLS request
attribute more than once
This is an automated email from the ASF dual-hosted git repository.
mgrigorov pushed a commit to branch 8.5.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/8.5.x by this push:
new 593b981 Do not check for TLS request attribute more than once
593b981 is described below
commit 593b9815fbe64a5bbef59e8ac977d152ef22de8f
Author: Martin Tzvetanov Grigorov <mg...@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 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/java/org/apache/catalina/connector/Request.java b/java/org/apache/catalina/connector/Request.java
index ad34a09..1d2c341 100644
--- a/java/org/apache/catalina/connector/Request.java
+++ b/java/org/apache/catalina/connector/Request.java
@@ -915,7 +915,7 @@ public class Request implements HttpServletRequest {
if (attr != null) {
return attr;
}
- if (TLSUtil.isTLSRequestAttribute(name)) {
+ if (!sslAttributesParsed && TLSUtil.isTLSRequestAttribute(name)) {
coyoteRequest.action(ActionCode.REQ_SSL_ATTRIBUTE, coyoteRequest);
attr = coyoteRequest.getAttribute(Globals.CERTIFICATES_ATTR);
if (attr != null) {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org