This is an automated email from the ASF dual-hosted git repository.
markt-asf pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new 1e8b11fd73 Partial revert of 99d1d567
1e8b11fd73 is described below
commit 1e8b11fd73e2356d2a7d97c5259744b3dc9369da
Author: Mark Thomas <[email protected]>
AuthorDate: Wed Sep 30 13:28:00 2026 +0100
Partial revert of 99d1d567
"" is a valid URL pattern
---
.../org/apache/catalina/core/ApplicationServletRegistration.java | 2 +-
.../apache/catalina/core/TestApplicationServletRegistration.java | 9 +--------
2 files changed, 2 insertions(+), 9 deletions(-)
diff --git a/java/org/apache/catalina/core/ApplicationServletRegistration.java
b/java/org/apache/catalina/core/ApplicationServletRegistration.java
index ae2d63f7bd..01805aa8f6 100644
--- a/java/org/apache/catalina/core/ApplicationServletRegistration.java
+++ b/java/org/apache/catalina/core/ApplicationServletRegistration.java
@@ -181,7 +181,7 @@ public class ApplicationServletRegistration implements
ServletRegistration.Dynam
String[] decodedUrlPatterns = new String[urlPatterns.length];
for (int i = 0; i < urlPatterns.length; i++) {
- if (urlPatterns[i] == null || urlPatterns[i].isEmpty()) {
+ if (urlPatterns[i] == null) {
throw new
IllegalArgumentException(sm.getString("applicationServletRegistration.nullUrlPattern"));
}
if (context.getUrlPatternsProvidedInDecodedForm()) {
diff --git
a/test/org/apache/catalina/core/TestApplicationServletRegistration.java
b/test/org/apache/catalina/core/TestApplicationServletRegistration.java
index bde12ac4af..30cb63ebdc 100644
--- a/test/org/apache/catalina/core/TestApplicationServletRegistration.java
+++ b/test/org/apache/catalina/core/TestApplicationServletRegistration.java
@@ -55,7 +55,7 @@ public class TestApplicationServletRegistration extends
TomcatBaseTest {
@Test
- public void testAddMappingNullAndEmptyPatterns() {
+ public void testAddMappingNullPatterns() {
StandardContext context = new StandardContext();
Wrapper wrapper = context.createWrapper();
@@ -77,13 +77,6 @@ public class TestApplicationServletRegistration extends
TomcatBaseTest {
} catch (IllegalArgumentException e) {
// Expected
}
-
- try {
- registration.addMapping("");
- Assert.fail("Expected an IllegalArgumentException for an empty
pattern");
- } catch (IllegalArgumentException e) {
- // Expected
- }
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
