From bb9baafad65f217c52a10d413384639d271c7d10 Mon Sep 17 00:00:00 2001 From: Steve Riesenberg <5248162+sjohnr@users.noreply.github.com> Date: Fri, 18 Oct 2024 15:22:42 -0500 Subject: [PATCH] Revert accidental changes --- .../java/example/RestClientConfiguration.java | 2 +- .../java/example/SecurityConfiguration.java | 66 ------------------- 2 files changed, 1 insertion(+), 67 deletions(-) diff --git a/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/RestClientConfiguration.java b/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/RestClientConfiguration.java index 96f9feaf..5ab0e4d6 100644 --- a/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/RestClientConfiguration.java +++ b/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/RestClientConfiguration.java @@ -41,7 +41,7 @@ public RestClientConfiguration(@Value("${messages.base-url}") String baseUrl) { this.baseUrl = baseUrl; } -// @Bean + @Bean public RestClient restClient(OAuth2AuthorizedClientManager authorizedClientManager, OAuth2AuthorizedClientRepository authorizedClientRepository, OAuth2ClientHttpRequestInterceptor.ClientRegistrationIdResolver clientRegistrationIdResolver, diff --git a/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/SecurityConfiguration.java b/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/SecurityConfiguration.java index c93f506e..be79fa5f 100644 --- a/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/SecurityConfiguration.java +++ b/servlet/spring-boot/java/oauth2/restclient/src/main/java/example/SecurityConfiguration.java @@ -19,25 +19,10 @@ import org.springframework.beans.factory.annotation.Value; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; -import org.springframework.core.convert.converter.Converter; -import org.springframework.http.converter.FormHttpMessageConverter; import org.springframework.security.config.Customizer; import org.springframework.security.config.annotation.web.builders.HttpSecurity; import org.springframework.security.config.annotation.web.configuration.EnableWebSecurity; -import org.springframework.security.oauth2.client.endpoint.OAuth2AccessTokenResponseClient; -import org.springframework.security.oauth2.client.endpoint.OAuth2ClientCredentialsGrantRequest; -import org.springframework.security.oauth2.client.endpoint.RestClientClientCredentialsTokenResponseClient; -import org.springframework.security.oauth2.client.http.OAuth2ErrorResponseErrorHandler; -import org.springframework.security.oauth2.core.endpoint.OAuth2AccessTokenResponse; -import org.springframework.security.oauth2.core.endpoint.OAuth2ParameterNames; -import org.springframework.security.oauth2.core.http.converter.OAuth2AccessTokenResponseHttpMessageConverter; import org.springframework.security.web.SecurityFilterChain; -import org.springframework.security.web.util.matcher.RequestMatcher; -import org.springframework.util.LinkedMultiValueMap; -import org.springframework.util.MultiValueMap; -import org.springframework.web.client.RestClient; -import org.springframework.web.context.request.RequestContextHolder; -import org.springframework.web.context.request.ServletRequestAttributes; /** * Security configuration for {@link OAuth2RestClientApplication}. @@ -77,55 +62,4 @@ public SecurityFilterChain securityFilterChain(HttpSecurity http) throws Excepti return http.build(); } - @Bean - public OAuth2AccessTokenResponseClient clientCredentialsAccessTokenResponseClient() { - RequestMatcher requestMatcher = (request) -> false; - Converter> parametersConverter = (grantRequest) -> { - LinkedMultiValueMap parameters = new LinkedMultiValueMap<>(); - if (grantRequest.getClientRegistration().getRegistrationId().equals("okta")) { - parameters.set(OAuth2ParameterNames.CLIENT_ID, "my-client"); - } - return parameters; - }; - - RestClientClientCredentialsTokenResponseClient accessTokenResponseClient = - new RestClientClientCredentialsTokenResponseClient(); - accessTokenResponseClient.setParametersConverter(parametersConverter); - - return accessTokenResponseClient; - } - - private static Converter> parametersConverter() { - RequestMatcher requestMatcher = (request) -> false; - return (grantRequest) -> { - ServletRequestAttributes requestAttributes = (ServletRequestAttributes) RequestContextHolder.getRequestAttributes(); - LinkedMultiValueMap parameters = new LinkedMultiValueMap<>(); - if (requestMatcher.matches(requestAttributes.getRequest())) { - parameters.set(OAuth2ParameterNames.SCOPE, "scope-1 scope-2"); - } - return parameters; - }; - } - - @Bean - public RestClient restClient() { - OAuth2AccessTokenResponseHttpMessageConverter messageConverter = - new OAuth2AccessTokenResponseHttpMessageConverter(); - messageConverter.setAccessTokenResponseConverter((parameters) -> { - // ... - return OAuth2AccessTokenResponse.withToken("custom-token") - // ... - .build(); - }); - - return RestClient.builder() - .messageConverters((messageConverters) -> { - messageConverters.clear(); - messageConverters.add(new FormHttpMessageConverter()); - messageConverters.add(messageConverter); - }) - .defaultStatusHandler(new OAuth2ErrorResponseErrorHandler()) - .build(); - } - }