-
Notifications
You must be signed in to change notification settings - Fork 38.8k
Closed
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)type: bugA general bugA general bug
Milestone
Description
Stephan Schroevers opened SPR-14141 and commented
The following test passes with version 4.2.5.RELEASE but fails with 4.3.0.RC1:
@Test
public void testPathMatcher() {
PathMatcher matcher = new AntPathMatcher();
assertTrue(matcher.match("https://example.com", "https://example.com"));
}Cursory debugging indicates that the double slashes are problematic. The following test passes with both versions:
@Test
public void testPathMatcher() {
PathMatcher matcher = new AntPathMatcher();
assertTrue(matcher.match("https:/**/example.com", "https://example.com"));
}This regression seems relevant because double slashes are common in patterns provided to the following methods:
MockMvcResultMatchers#forwardedUrlPatternMockMvcResultMatchers#redirectedUrlPattern
I suspect this is related to #18487.
Affects: 4.3 RC1
Issue Links:
- Speed up AntPathMatcher by doing prefix-match [SPR-13913] #18487 Speed up AntPathMatcher by doing prefix-match
Referenced from: commits 368f29d
Metadata
Metadata
Assignees
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)type: bugA general bugA general bug