This is an automated email from the ASF dual-hosted git repository.

remm pushed a commit to branch 10.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/10.0.x by this push:
     new a94019e  Fix write timeout check
a94019e is described below

commit a94019ed79ae01196dfab325d9820e2d520cd442
Author: remm <[email protected]>
AuthorDate: Wed Aug 18 16:54:36 2021 +0200

    Fix write timeout check
    
    BZ 65506. Fix write timeout check that was using the read timeout value.
    Patch submitted by Gustavo Mahlow
---
 java/org/apache/tomcat/websocket/WsSession.java | 2 +-
 webapps/docs/changelog.xml                      | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/java/org/apache/tomcat/websocket/WsSession.java 
b/java/org/apache/tomcat/websocket/WsSession.java
index 3a37d66..03812d0 100644
--- a/java/org/apache/tomcat/websocket/WsSession.java
+++ b/java/org/apache/tomcat/websocket/WsSession.java
@@ -1031,7 +1031,7 @@ public class WsSession implements Session {
 
         if (timeoutRead > 0 && (currentTime - lastActiveRead) > timeoutRead) {
             key = "wsSession.timeoutRead";
-        } else if (timeoutWrite > 0 && (currentTime - lastActiveWrite) > 
timeoutRead) {
+        } else if (timeoutWrite > 0 && (currentTime - lastActiveWrite) > 
timeoutWrite) {
             key = "wsSession.timeoutWrite";
         } else if (timeout > 0 && (currentTime - lastActiveRead) > timeout &&
                 (currentTime - lastActiveWrite) > timeout) {
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 1c31290..65f1702 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -145,6 +145,14 @@
       </fix>
     </changelog>
   </subsection>
+  <subsection name="WebSocket">
+    <changelog>
+      <fix>
+        <bug>65506</bug>: Fix write timeout check that was using the read
+        timeout value. Patch submitted by Gustavo Mahlow. (remm)
+      </fix>
+    </changelog>
+  </subsection>
   <subsection name="Web applications">
     <changelog>
       <fix>

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to