trimming the whitespace for this provider.

see gh-17487

Author: nosan

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/client/OAuth2ClientPropertiesRegistrationAdapter.java

@@ -75,7 +75,8 @@ private static ClientRegistration getClientRegistration(String registrationId,
 
 	private static Builder getBuilderFromIssuerIfPossible(String registrationId, String configuredProviderId,
 			Map<String, Provider> providers) {
-		String providerId = (configuredProviderId != null) ? configuredProviderId : registrationId;
+		String providerId = StringUtils
+				.trimWhitespace((configuredProviderId != null) ? configuredProviderId : registrationId);
 		if (providers.containsKey(providerId)) {
 			Provider provider = providers.get(providerId);
 			String issuer = provider.getIssuerUri();
@@ -89,7 +90,8 @@ private static Builder getBuilderFromIssuerIfPossible(String registrationId, Str
 
 	private static Builder getBuilder(String registrationId, String configuredProviderId,
 			Map<String, Provider> providers) {
-		String providerId = (configuredProviderId != null) ? configuredProviderId : registrationId;
+		String providerId = StringUtils
+				.trimWhitespace((configuredProviderId != null) ? configuredProviderId : registrationId);
 		CommonOAuth2Provider provider = getCommonProvider(providerId);
 		if (provider == null && !providers.containsKey(providerId)) {
 			throw new IllegalStateException(getErrorMessage(configuredProviderId, registrationId));

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/security/oauth2/client/OAuth2ClientPropertiesRegistrationAdapterTests.java

@@ -251,6 +251,31 @@ void oidcProviderConfigurationWithCustomConfigurationOverridesProviderDefaults()
 		assertThat(userInfoEndpoint.getUserNameAttributeName()).isEqualTo("sub");
 	}
 
+	@Test
+	void getClientRegistrationsWhenRegistrationProviderPropertyShouldBeTrimmed() {
+		OAuth2ClientProperties properties = new OAuth2ClientProperties();
+		OAuth2ClientProperties.Registration registration = createRegistration("  provider  ");
+		Provider provider = createProvider();
+		properties.getProvider().put("provider", provider);
+		properties.getRegistration().put("registration", registration);
+		assertThat(OAuth2ClientPropertiesRegistrationAdapter.getClientRegistrations(properties)).isNotEmpty();
+	}
+
+	@Test
+	void getClientRegistrationsWhenRegistrationProviderPropertyWithIssuerShouldBeTrimmed() throws Exception {
+		this.server = new MockWebServer();
+		this.server.start();
+		String issuer = this.server.url("").toString();
+		setupMockResponse(issuer);
+		OAuth2ClientProperties properties = new OAuth2ClientProperties();
+		OAuth2ClientProperties.Registration registration = createRegistration("   provider  ");
+		Provider provider = createProvider();
+		provider.setIssuerUri(issuer);
+		properties.getProvider().put("provider", provider);
+		properties.getRegistration().put("registration", registration);
+		assertThat(OAuth2ClientPropertiesRegistrationAdapter.getClientRegistrations(properties)).isNotEmpty();
+	}
+
 	private Provider createProvider() {
 		Provider provider = new Provider();
 		provider.setAuthorizationUri("https://example.com/auth");