Skip to content

Migrate Authentication (OpenID) logic from quickcase/spring-oidc#32

Merged
vlaurin merged 10 commits intomainfrom
auth
Mar 20, 2025
Merged

Migrate Authentication (OpenID) logic from quickcase/spring-oidc#32
vlaurin merged 10 commits intomainfrom
auth

Conversation

@vlaurin
Copy link
Copy Markdown
Contributor

@vlaurin vlaurin commented Mar 19, 2025

Cleaned up and simplified for re-release as part of standalone SDK.

  • Update README

vlaurin added 10 commits March 19, 2025 14:28
Where possible the code is simplified:
- Merge modules `api` and `implementation`
- Merge interfaces with single implementations

This also changes the base class of `QuickcaseAuthentication` to `JwtAuthenticationToken` which is more accurate and removes some boilerplate code.
As parsing from UserInfo and parsing of JSON organisation profiles are both scheduled for deletion, there will soon not be any need for `JwtClaimsParser` as native JWT methods are sufficient.
- UserInfo fetching and parsing to be removed in favour of access token authorisation
- Organisation profiles to be removed in favour of role-driven authorisation
Not scheduled for removal yet as no replacement currently available.
Make explicit a configuration property used internally by data-store. This should be removed once legacy strategy is dropped.
Make explicit a configuration property used for conditional selection of beans. This should be removed once `user-info` mode is dropped.
Provides high-level customiser for ease of use in consuming Spring applications.
@vlaurin vlaurin marked this pull request as ready for review March 20, 2025 13:04
@vlaurin vlaurin requested a review from gbenadikar March 20, 2025 14:48
@vlaurin vlaurin added the type:feature Feature request label Mar 20, 2025
@vlaurin vlaurin merged commit dd25094 into main Mar 20, 2025
2 checks passed
@vlaurin vlaurin deleted the auth branch March 20, 2025 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature Feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants