Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move OAuth2 classes BaseClientDetails to UaaClientDetails #2806

Merged
merged 20 commits into from
Apr 11, 2024

Conversation

strehle
Copy link
Member

@strehle strehle commented Mar 28, 2024

Move the classes because finally we want remove https://github.com/cloudfoundry/uaa/blob/develop/dependencies.gradle#L14

The classes here represent the persistence of UAA. Moved only the classes but not the interfaces yet.
The interfaces used in many other library methods but finally the interfaces will be moved also next to the classes here

The BaseClientDetails class does not know jwtClientConfig field of UAA but now UaaClientDetails class know
Added tests for the base classes and changed many imports

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/187329458

The labels on this github issue will be updated when the story is started.

@strehle strehle changed the title Move to UaaBaseClientDetails [WIP] Move OAuth2 used classes to UAA - Move to UaaBaseClientDetails Mar 28, 2024
@strehle strehle added this to the EOL_Removal milestone Mar 28, 2024
@strehle strehle linked an issue Mar 28, 2024 that may be closed by this pull request
@strehle strehle changed the title [WIP] Move OAuth2 used classes to UAA - Move to UaaBaseClientDetails [WIP] Move OAuth2 used classes to UAA - Move to UaaClientDetails Mar 29, 2024
@strehle strehle changed the title [WIP] Move OAuth2 used classes to UAA - Move to UaaClientDetails Move OAuth2 classes ClientDetails to UaaClientDetails Apr 2, 2024
@strehle strehle marked this pull request as ready for review April 2, 2024 08:58
@strehle strehle changed the title Move OAuth2 classes ClientDetails to UaaClientDetails Move OAuth2 classes BaseClientDetails to UaaClientDetails Apr 2, 2024
@strehle
Copy link
Member Author

strehle commented Apr 4, 2024

@strehle strehle requested a review from a team April 4, 2024 16:43
@strehle strehle requested a review from hsinn0 April 5, 2024 16:37
@strehle strehle merged commit fc3f63c into develop Apr 11, 2024
20 checks passed
@strehle strehle deleted the eol/oauth2/client-persistency-move branch April 11, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Spring Security Oauth2 End of Life
3 participants