This is an automated email from the ASF dual-hosted git repository.
markt 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 274945612c Remove unnecessary code
274945612c is described below
commit 274945612c96a9c24b84ef21e06bee8488d2dca8
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue Nov 8 11:41:41 2022 +0000
Remove unnecessary code
The default async timeout is stored on the Connector and retrieved from
there by the request. The request then sets it on the Processor. It is
never used by the protocol or the endpoint.
---
java/org/apache/catalina/connector/Connector.java | 1 -
webapps/docs/changelog.xml | 4 ++++
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/java/org/apache/catalina/connector/Connector.java
b/java/org/apache/catalina/connector/Connector.java
index d8bc16428c..151da536eb 100644
--- a/java/org/apache/catalina/connector/Connector.java
+++ b/java/org/apache/catalina/connector/Connector.java
@@ -422,7 +422,6 @@ public class Connector extends LifecycleMBeanBase {
*/
public void setAsyncTimeout(long asyncTimeout) {
this.asyncTimeout= asyncTimeout;
- setProperty("asyncTimeout", String.valueOf(asyncTimeout));
}
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index f501abd8c4..21ecf497b1 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -142,6 +142,10 @@
not to do so and remove checks for this return value. Based on
<pr>562</pr> by tianshuang. (markt)
</scode>
+ <scode>
+ Remove unnecessary code that exposed the <code>asyncTimeout</code> to
+ components that never used it. (markt)
+ </scode>
</changelog>
</subsection>
<subsection name="Jasper">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
