Skip to content

Commit ade4577

Browse files
committed
Merge branch '5.8.x' into 6.1.x
Closes gh-14921
2 parents 8a03482 + 657760a commit ade4577

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
@@ -68,7 +68,7 @@ private RequestMatcherDelegatingAuthorizationManager(
6868
@Override
6969
public AuthorizationDecision check(Supplier<Authentication> authentication, HttpServletRequest request) {
7070
if (this.logger.isTraceEnabled()) {
71-
this.logger.trace(LogMessage.format("Authorizing %s", request));
71+
this.logger.trace(LogMessage.format("Authorizing %s", requestLine(request)));
7272
}
7373
for (RequestMatcherEntry<AuthorizationManager<RequestAuthorizationContext>> mapping : this.mappings) {
7474

@@ -77,7 +77,7 @@ public AuthorizationDecision check(Supplier<Authentication> authentication, Http
7777
if (matchResult.isMatch()) {
7878
AuthorizationManager<RequestAuthorizationContext> manager = mapping.getEntry();
7979
if (this.logger.isTraceEnabled()) {
80-
this.logger.trace(LogMessage.format("Checking authorization on %s using %s", request, manager));
80+
this.logger.trace(LogMessage.format("Checking authorization on %s using %s", requestLine(request), manager));
8181
}
8282
return manager.check(authentication,
8383
new RequestAuthorizationContext(request, matchResult.getVariables()));
@@ -89,6 +89,10 @@ public AuthorizationDecision check(Supplier<Authentication> authentication, Http
8989
return DENY;
9090
}
9191

92+
private static String requestLine(HttpServletRequest request) {
93+
return request.getMethod() + " " + UrlUtils.buildRequestUrl(request);
94+
}
95+
9296
/**
9397
* Creates a builder for {@link RequestMatcherDelegatingAuthorizationManager}.
9498
* @return the new {@link Builder} instance

0 commit comments

Comments
 (0)