You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2012/08/27 00:29:04 UTC
svn commit: r1377535 -
/tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java
Author: markt
Date: Sun Aug 26 22:29:04 2012
New Revision: 1377535
URL: http://svn.apache.org/viewvc?rev=1377535&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53758
Fix Dynamic Filter Registration mapping logic inverted
Modified:
tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java
Modified: tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java?rev=1377535&r1=1377534&r2=1377535&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterRegistration.java Sun Aug 26 22:29:04 2012
@@ -72,9 +72,9 @@ public class ApplicationFilterRegistrati
}
if (isMatchAfter) {
- context.addFilterMapBefore(filterMap);
- } else {
context.addFilterMap(filterMap);
+ } else {
+ context.addFilterMapBefore(filterMap);
}
}
// else error?
@@ -101,9 +101,9 @@ public class ApplicationFilterRegistrati
}
if (isMatchAfter) {
- context.addFilterMapBefore(filterMap);
- } else {
context.addFilterMap(filterMap);
+ } else {
+ context.addFilterMapBefore(filterMap);
}
}
// else error?
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org