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

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


The following commit(s) were added to refs/heads/main by this push:
     new 5394e61289 Follow-up to 4127ca5f
5394e61289 is described below

commit 5394e612892c10e78e89fd09d95e4ecec4064679
Author: Mark Thomas <[email protected]>
AuthorDate: Wed Sep 30 11:48:33 2026 +0100

    Follow-up to 4127ca5f
    
    Don't swallow configuration errors
---
 java/org/apache/catalina/ant/jmx/JMXAccessorConditionBase.java | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/java/org/apache/catalina/ant/jmx/JMXAccessorConditionBase.java 
b/java/org/apache/catalina/ant/jmx/JMXAccessorConditionBase.java
index be21134b5d..0ce599de7e 100644
--- a/java/org/apache/catalina/ant/jmx/JMXAccessorConditionBase.java
+++ b/java/org/apache/catalina/ant/jmx/JMXAccessorConditionBase.java
@@ -23,6 +23,7 @@ import javax.management.MBeanServerConnection;
 import javax.management.ObjectName;
 import javax.management.remote.JMXConnector;
 
+import org.apache.tools.ant.BuildException;
 import org.apache.tools.ant.ProjectComponent;
 import org.apache.tools.ant.taskdefs.condition.Condition;
 
@@ -266,6 +267,9 @@ public abstract class JMXAccessorConditionBase extends 
ProjectComponent implemen
             if (result != null) {
                 return result.toString();
             }
+        } catch (BuildException e) {
+            // Re-throw configuration errors.
+            throw e;
         } catch (Exception e) {
             /*
              * Exceptions are ignored for compatibility with the waitFor task 
when waiting for the server to start. If


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

Reply via email to