Handle no-context, no ending slash with the same wildcard rule

Kalle Korhonen added a comment - 03/Jun/12 4:52 PM I implemented a following fix before noticing the ant pattern matcher handles the case already. Index: src/main/java/org/tynamo/security/services/impl/SecurityConfiguration.java =================================================================== — src/main/java/org/tynamo/security/services/impl/SecurityConfiguration.java (revision 2388) +++ src/main/java/org/tynamo/security/services/impl/SecurityConfiguration.java (working copy) @@ -89,6 +89,10 @@ SecurityFilterChain configureChain = null; for (String path : chainMap.keySet()) { + // make sure paths /admin and /admin/ are handled by the same wildcard url + // intentionally don't consider questionmark character + if (!requestURI.endsWith("/") && path.endsWith("*")) requestURI += "/"; + // If the path does match, then pass on to the subclass implementation for specific checks: if (pathMatcher.matches(path, requestURI)) { configureChain = chainMap.get(path);

Kalle Korhonen added a comment - 03/Jun/12 5:15 PM Just implemented a test for it, the current codebase handles the case already