Add missing else branch in OAuth2ClientWebMvcSecurityConfiguration#addArgumentResolvers

krisztian-toth · krisztian-toth · commit 3fef1670f71a · 2019-08-26T09:28:00.000+02:00
diff --git a/config/src/main/java/org/springframework/security/config/annotation/web/configuration/OAuth2ClientConfiguration.java b/config/src/main/java/org/springframework/security/config/annotation/web/configuration/OAuth2ClientConfiguration.java
@@ -79,6 +79,8 @@ public void addArgumentResolvers(List<HandlerMethodArgumentResolver> argumentRes
 				if (this.accessTokenResponseClient != null) {
 					authorizedClientProviderBuilder.clientCredentials(configurer ->
 									configurer.accessTokenResponseClient(this.accessTokenResponseClient));
+				} else {
+					authorizedClientProviderBuilder.clientCredentials();
 				}
 				OAuth2AuthorizedClientProvider authorizedClientProvider = authorizedClientProviderBuilder.build();
 

Original file line number	Diff line number	Diff line change
`@@ -79,6 +79,8 @@ public void addArgumentResolvers(List<HandlerMethodArgumentResolver> argumentRes`
`79`	`79`	`if (this.accessTokenResponseClient != null) {`
`80`	`80`	`authorizedClientProviderBuilder.clientCredentials(configurer ->`
`81`	`81`	`configurer.accessTokenResponseClient(this.accessTokenResponseClient));`
	`82`	`+ } else {`
	`83`	`+ authorizedClientProviderBuilder.clientCredentials();`
`82`	`84`	`}`
`83`	`85`	`OAuth2AuthorizedClientProvider authorizedClientProvider = authorizedClientProviderBuilder.build();`
`84`	`86`