Skip to content

Commit 558b7e0

Browse files
committed
Use PathPatternRequestMatcher in cas
Issue gh-16887
1 parent e8ed0f1 commit 558b7e0

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

cas/src/main/java/org/springframework/security/cas/web/CasAuthenticationFilter.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,12 @@
5151
import org.springframework.security.web.savedrequest.HttpSessionRequestCache;
5252
import org.springframework.security.web.savedrequest.RequestCache;
5353
import org.springframework.security.web.savedrequest.SavedRequest;
54-
import org.springframework.security.web.servlet.util.matcher.PathPatternRequestMatcher;
55-
import org.springframework.security.web.util.matcher.AntPathRequestMatcher;
5654
import org.springframework.security.web.util.matcher.RequestMatcher;
5755
import org.springframework.util.Assert;
5856
import org.springframework.util.StringUtils;
5957

58+
import static org.springframework.security.web.servlet.util.matcher.PathPatternRequestMatcher.pathPattern;
59+
6060
/**
6161
* Processes a CAS service ticket, obtains proxy granting tickets, and processes proxy
6262
* tickets.
@@ -216,7 +216,7 @@ public class CasAuthenticationFilter extends AbstractAuthenticationProcessingFil
216216

217217
public CasAuthenticationFilter() {
218218
super("/login/cas");
219-
RequestMatcher processUri = PathPatternRequestMatcher.withDefaults().matcher("/login/cas");
219+
RequestMatcher processUri = pathPattern("/login/cas");
220220
setRequiresAuthenticationRequestMatcher(processUri);
221221
setAuthenticationFailureHandler(new SimpleUrlAuthenticationFailureHandler());
222222
setSecurityContextRepository(this.securityContextRepository);
@@ -335,7 +335,7 @@ public final void setProxyReceptorMatcher(RequestMatcher proxyReceptorMatcher) {
335335
}
336336

337337
public final void setProxyReceptorUrl(final String proxyReceptorUrl) {
338-
this.proxyReceptorMatcher = new AntPathRequestMatcher("/**" + proxyReceptorUrl);
338+
this.proxyReceptorMatcher = pathPattern(proxyReceptorUrl);
339339
}
340340

341341
public final void setProxyGrantingTicketStorage(final ProxyGrantingTicketStorage proxyGrantingTicketStorage) {

cas/src/test/java/org/springframework/security/cas/web/CasAuthenticationFilterTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@
4343
import org.springframework.security.web.authentication.AuthenticationSuccessHandler;
4444
import org.springframework.security.web.context.SecurityContextRepository;
4545
import org.springframework.security.web.savedrequest.HttpSessionRequestCache;
46-
import org.springframework.security.web.servlet.util.matcher.PathPatternRequestMatcher;
4746
import org.springframework.test.util.ReflectionTestUtils;
4847

4948
import static org.assertj.core.api.Assertions.assertThat;
@@ -57,6 +56,7 @@
5756
import static org.mockito.Mockito.verifyNoMoreInteractions;
5857
import static org.springframework.security.web.servlet.TestMockHttpServletRequests.get;
5958
import static org.springframework.security.web.servlet.TestMockHttpServletRequests.post;
59+
import static org.springframework.security.web.servlet.util.matcher.PathPatternRequestMatcher.pathPattern;
6060

6161
/**
6262
* Tests {@link CasAuthenticationFilter}.
@@ -266,7 +266,7 @@ public void requiresAuthenticationWhenProxyRequestMatcherThenMatches() {
266266
MockHttpServletRequest request = get("/pgtCallback").build();
267267
MockHttpServletResponse response = new MockHttpServletResponse();
268268
assertThat(filter.requiresAuthentication(request, response)).isFalse();
269-
filter.setProxyReceptorMatcher(PathPatternRequestMatcher.withDefaults().matcher(request.getServletPath()));
269+
filter.setProxyReceptorMatcher(pathPattern(request.getServletPath()));
270270
assertThat(filter.requiresAuthentication(request, response)).isFalse();
271271
filter.setProxyGrantingTicketStorage(mock(ProxyGrantingTicketStorage.class));
272272
assertThat(filter.requiresAuthentication(request, response)).isTrue();

0 commit comments

Comments
 (0)