From: pero <pero@13f79535-47bb-0310-9956-ffa450edef68>
Date: Sat, 15 Sep 2007 07:55:42 +0000 (+0000)
Subject: SRV 6.2.5 says supporting for '*' as the servlet-name in filter-mapping.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=f96e27a8a394e0c30e29a240bcc9fe05459eec4c;p=tomcat7.0

SRV 6.2.5 says supporting for '*' as the servlet-name in filter-mapping.
    <filter-mapping>
        <filter-name>SampleFilter</filter-name>
        <servlet-name>*</servlet-name>
    </filter-mapping>


git-svn-id: https://svn.apache.org/repos/asf/tomcat/tc6.0.x/trunk@575890 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/org/apache/catalina/core/StandardContext.java b/java/org/apache/catalina/core/StandardContext.java
index b5b218419..3f893c027 100644
--- a/java/org/apache/catalina/core/StandardContext.java
+++ b/java/org/apache/catalina/core/StandardContext.java
@@ -2156,7 +2156,13 @@ public class StandardContext
         if (findFilterDef(filterName) == null)
             throw new IllegalArgumentException
                 (sm.getString("standardContext.filterMap.name", filterName));
-        if ((servletNames.length == 0) && (urlPatterns.length == 0))
+//      <= Servlet API 2.4
+//      if ((servletNames.length == 0) && (urlPatterns.length == 0))
+//      Servlet API 2.5 (FIX 43338)
+//      SRV 6.2.5 says supporting for '*' as the servlet-name in filter-mapping.
+        if (!filterMap.getMatchAllServletNames() && 
+            !filterMap.getMatchAllUrlPatterns() && 
+            (servletNames.length == 0) && (urlPatterns.length == 0))
             throw new IllegalArgumentException
                 (sm.getString("standardContext.filterMap.either"));
         // FIXME: Older spec revisions may still check this
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 1a226bb55..2f5b1b771 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -84,7 +84,11 @@
       </update>
       <fix>
         o.a.juli.ClassLoaderLogManager handle more then one system property replacement at file logging.properties. (pero)
-      </fix>     
+      </fix>
+      <fix>
+        <bug>43338</bug>: Support '*' servlet-name mapping at filter-mapping.
+        Patch provided by Keiichi Fujino. (pero)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Coyote">