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

remm 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 9f6075bfb4 Improve exception for unexpected error
9f6075bfb4 is described below

commit 9f6075bfb47568eec17c0e0c7e853bd902a13fef
Author: remm <[email protected]>
AuthorDate: Tue Mar 24 15:15:03 2026 +0100

    Improve exception for unexpected error
---
 java/org/apache/catalina/mapper/LocalStrings.properties | 1 +
 java/org/apache/catalina/mapper/Mapper.java             | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/java/org/apache/catalina/mapper/LocalStrings.properties 
b/java/org/apache/catalina/mapper/LocalStrings.properties
index b5544fc2fc..5b41c3bcd3 100644
--- a/java/org/apache/catalina/mapper/LocalStrings.properties
+++ b/java/org/apache/catalina/mapper/LocalStrings.properties
@@ -19,6 +19,7 @@ mapper.addHost.sameHost=Duplicate registration of the same 
host [{0}]. Ignored.
 mapper.addHost.success=Registered host [{0}]
 mapper.addHostAlias.sameHost=Duplicate registration of alias [{0}] for the 
same host [{1}]. Ignored.
 mapper.addHostAlias.success=Registered alias [{0}] for host [{1}]
+mapper.alreadyDone=Mapping was already done on this request on host [{0}]
 mapper.duplicateHost=Duplicate Host [{0}]. The name is already used by Host 
[{1}]. This Host will be ignored.
 mapper.duplicateHostAlias=Duplicate host Alias [{0}] in Host [{1}]. The name 
is already used by Host [{2}]. This Alias will be ignored.
 mapper.findContext.noContext=No context found [{0}]
diff --git a/java/org/apache/catalina/mapper/Mapper.java 
b/java/org/apache/catalina/mapper/Mapper.java
index 37719cb10f..2bbb54bdb7 100644
--- a/java/org/apache/catalina/mapper/Mapper.java
+++ b/java/org/apache/catalina/mapper/Mapper.java
@@ -710,7 +710,7 @@ public final class Mapper {
             // skipped all mapping work in this case. That behaviour has a risk
             // of returning an inconsistent result.
             // I do not see a valid use case for it.
-            throw new AssertionError();
+            throw new IllegalStateException(sm.getString("mapper.alreadyDone", 
mappingData.host));
         }
 
         // Virtual host mapping


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

Reply via email to