Skip to content

Commit f16a434

Browse files
committed
Merge branch '6.1.x' into 6.2.x
Closes gh-14922
2 parents 05b9375 + ade4577 commit f16a434

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

web/src/main/java/org/springframework/security/web/access/intercept/RequestMatcherDelegatingAuthorizationManager.java

+6-2
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ private RequestMatcherDelegatingAuthorizationManager(
7171
@Override
7272
public AuthorizationDecision check(Supplier<Authentication> authentication, HttpServletRequest request) {
7373
if (this.logger.isTraceEnabled()) {
74-
this.logger.trace(LogMessage.format("Authorizing %s", request));
74+
this.logger.trace(LogMessage.format("Authorizing %s", requestLine(request)));
7575
}
7676
for (RequestMatcherEntry<AuthorizationManager<RequestAuthorizationContext>> mapping : this.mappings) {
7777

@@ -80,7 +80,7 @@ public AuthorizationDecision check(Supplier<Authentication> authentication, Http
8080
if (matchResult.isMatch()) {
8181
AuthorizationManager<RequestAuthorizationContext> manager = mapping.getEntry();
8282
if (this.logger.isTraceEnabled()) {
83-
this.logger.trace(LogMessage.format("Checking authorization on %s using %s", request, manager));
83+
this.logger.trace(LogMessage.format("Checking authorization on %s using %s", requestLine(request), manager));
8484
}
8585
return manager.check(authentication,
8686
new RequestAuthorizationContext(request, matchResult.getVariables()));
@@ -92,6 +92,10 @@ public AuthorizationDecision check(Supplier<Authentication> authentication, Http
9292
return DENY;
9393
}
9494

95+
private static String requestLine(HttpServletRequest request) {
96+
return request.getMethod() + " " + UrlUtils.buildRequestUrl(request);
97+
}
98+
9599
/**
96100
* Creates a builder for {@link RequestMatcherDelegatingAuthorizationManager}.
97101
* @return the new {@link Builder} instance

0 commit comments

Comments
 (0)