Cleanup in DefaultBearerTokenResolver.java

Removed redundant method call
spring-projects · Nov 21, 2024 · a9edc44 · a9edc44
1 parent 77233da
commit a9edc44
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...a/org/springframework/security/oauth2/server/resource/web/DefaultBearerTokenResolver.java b/...a/org/springframework/security/oauth2/server/resource/web/DefaultBearerTokenResolver.java
@@ -53,8 +53,8 @@ public final class DefaultBearerTokenResolver implements BearerTokenResolver {
 	@Override
 	public String resolve(final HttpServletRequest request) {
 		final String authorizationHeaderToken = resolveFromAuthorizationHeader(request);
-		final String parameterToken = isParameterTokenSupportedForRequest(request)
-				? resolveFromRequestParameters(request) : null;
+		final boolean isParameterTokenSupported = isParameterTokenSupportedForRequest(request);
+		final String parameterToken = isParameterTokenSupported ? resolveFromRequestParameters(request) : null;
 		if (authorizationHeaderToken != null) {
 			if (parameterToken != null) {
 				BearerTokenError error = BearerTokenErrors
@@ -63,7 +63,7 @@ public String resolve(final HttpServletRequest request) {
 			}
 			return authorizationHeaderToken;
 		}
-		if (parameterToken != null && isParameterTokenEnabledForRequest(request)) {
+		if (parameterToken != null && isParameterTokenSupported) {
 			if (!StringUtils.hasText(parameterToken)) {
 				BearerTokenError error = BearerTokenErrors
 					.invalidRequest("The requested token parameter is an empty string");