Steve Riesenberg

Results 276 comments of Steve Riesenberg

Thanks @NotFound403. I think you're just requesting a general enhancement (nice-to-have) to make `OAuth2ClientSpec` consistent with `OAuth2LoginSpec` in the way it obtains a `ReactiveOAuth2AccessTokenResponseClient` for creating the default `OAuth2AuthorizationCodeReactiveAuthenticationManager`. Let...

@mjeffrey thanks for starting the conversation on this! To clarify, are you specifically directing this issue toward providing support for `RestClient` in OAuth2 equivalent to the existing [`ServletOAuth2AuthorizedClientExchangeFilterFunction`](https://docs.spring.io/spring-security/reference/servlet/oauth2/client/authorized-clients.html#oauth2Client-webclient-servlet) which supports...

Ok, I've changed the title of the issue to what I think aligns with the request as a starting point, and we can go from there. Since the only part...

@tudburley we already have **Servlet**OAuth2AuthorizedClientExchangeFilterFunction, is that what you're asking about? Otherwise, I'm not sure I understand the question. If you have a different enhancement suggestion than what is being...

Ah, thanks for clarifying @tudburley! > Its hard to understand what is the difference of **Servlet**OAuth2AuthorizedClientExchangeFilterFunction and **Server**OAuth2AuthorizedClientExchangeFilterFunction. My guess is, that the servlet variant stores authorized clients in the...

@rawadolb > i don't understand why this issue was closed. This issue is still open. Perhaps you saw that I closed gh-8882 which is a separate issue? We do still...

Thanks everyone for the feedback, conversation and upvotes on this issue. > May I please have an update on this matter? We're watching this issue closely as it already has...

> It is not difficult to implement this ourselves (with guidance) so maybe it does not need to be provided by Spring Security? I agree that it is not difficult...

Thanks for reaching out @Y-k-Y! Apologies that this issue was not responded to for a long time. Thank you for providing a sample. It appears that you are not configuring...

Thanks for the feedback on using the documentation, @NZhuravlev! I'd like to understand the feedback better though, to know whether this specific page of the documentation is problematic (in isolation)...