(tomcat) branch 11.0.x updated: Better log message trying to remove Processor from waitingProcessors

[markt]

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

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


The following commit(s) were added to refs/heads/11.0.x by this push:
     new f25e4e8428 Better log message trying to remove Processor from 
waitingProcessors
f25e4e8428 is described below

commit f25e4e8428923867b78ccab9d3affe0c0caf8fb2
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Fri Sep 13 14:06:17 2024 +0100

    Better log message trying to remove Processor from waitingProcessors
---
 java/org/apache/coyote/AbstractProtocol.java   | 5 +++--
 java/org/apache/coyote/LocalStrings.properties | 2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/java/org/apache/coyote/AbstractProtocol.java 
b/java/org/apache/coyote/AbstractProtocol.java
index be928ed3d5..b73b673d46 100644
--- a/java/org/apache/coyote/AbstractProtocol.java
+++ b/java/org/apache/coyote/AbstractProtocol.java
@@ -444,10 +444,11 @@ public abstract class AbstractProtocol<S> implements 
ProtocolHandler, MBeanRegis
 
 
     public void removeWaitingProcessor(Processor processor) {
+        boolean result = waitingProcessors.remove(processor);
         if (getLog().isTraceEnabled()) {
-            
getLog().trace(sm.getString("abstractProtocol.waitingProcessor.remove", 
processor));
+            getLog().trace(sm.getString(
+                    "abstractProtocol.waitingProcessor.remove", processor, 
Boolean.valueOf(result)));
         }
-        waitingProcessors.remove(processor);
     }
 
 
diff --git a/java/org/apache/coyote/LocalStrings.properties 
b/java/org/apache/coyote/LocalStrings.properties
index a2d1acf103..bd4e432e64 100644
--- a/java/org/apache/coyote/LocalStrings.properties
+++ b/java/org/apache/coyote/LocalStrings.properties
@@ -39,7 +39,7 @@ abstractProtocol.mbeanDeregistrationFailed=Failed to 
deregister MBean named [{0}
 abstractProtocol.processorRegisterError=Error registering request processor
 abstractProtocol.processorUnregisterError=Error unregistering request processor
 abstractProtocol.waitingProcessor.add=Adding processor [{0}] to waiting 
processors
-abstractProtocol.waitingProcessor.remove=Removed processor [{0}] from waiting 
processors
+abstractProtocol.waitingProcessor.remove=Called remove for processor [{0}] 
from waiting processors with result [{1}]
 
 abstractProtocolHandler.asyncTimeoutError=Error processing async timeouts
 abstractProtocolHandler.destroy=Destroying ProtocolHandler [{0}]


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org