Skip to content

Commit a0bf098

Browse files
committed
Short circuit way to authorize
1 parent 0998a64 commit a0bf098

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

oauth2/oauth2-client/src/main/java/org/springframework/security/oauth2/client/DelegatingOAuth2AuthorizedClientProvider.java

+2-7
Original file line numberDiff line numberDiff line change
@@ -63,17 +63,12 @@ public DelegatingOAuth2AuthorizedClientProvider(List<OAuth2AuthorizedClientProvi
6363
@Nullable
6464
public OAuth2AuthorizedClient authorize(OAuth2AuthorizationContext context) {
6565
Assert.notNull(context, "context cannot be null");
66-
List<OAuth2AuthorizedClient> clients = new ArrayList<>();
6766
for (OAuth2AuthorizedClientProvider authorizedClientProvider: authorizedClientProviders) {
6867
final OAuth2AuthorizedClient oauth2AuthorizedClient = authorizedClientProvider.authorize(context);
6968
if (oauth2AuthorizedClient != null) {
70-
clients.add(oauth2AuthorizedClient);
69+
return oauth2AuthorizedClient;
7170
}
7271
}
73-
if (!clients.isEmpty()) {
74-
return clients.get(0);
75-
} else {
76-
return null;
77-
}
72+
return null;
7873
}
7974
}

0 commit comments

Comments
 (0)