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

chore(jans-auth-server): add oauth2-oidc-sdk dependency #6545

Closed
jgomer2001 opened this issue Nov 9, 2023 · 0 comments · Fixed by #6554
Closed

chore(jans-auth-server): add oauth2-oidc-sdk dependency #6545

jgomer2001 opened this issue Nov 9, 2023 · 0 comments · Fixed by #6554
Assignees
Labels
comp-jans-auth-server Component affected by issue or PR kind-dependencies Pull requests that update a dependency file
Milestone

Comments

@jgomer2001
Copy link
Contributor

Both agama inbound identity project and the Keycloack authn plugin depend on com.nimbus:oauth2-oidc-sdk (versions 10.13.2 and 10.11 respectively) to build their jar files. These are optional components that users can add to the AS

To allow the ease of deployment (ie. avoid copying jars to custom/libs , editing xml files, or building custom images in Cloud Native envs) we should include the nimbus dependency so that oauth2-oidc-sdk (and its transitives) are already bundled in the AS war.

I propose to use com.nimbus:oauth2-oidc-sdk:10.15 (the latest available in the 10.x series). Currently we only have nimbus-jose-jwt in the bom

@mo-auto mo-auto added comp-jans-auth-server Component affected by issue or PR kind-dependencies Pull requests that update a dependency file labels Nov 9, 2023
@yuriyz yuriyz added this to the 1.0.21 milestone Nov 9, 2023
yuriyz added a commit that referenced this issue Nov 13, 2023
Signed-off-by: YuriyZ <yzabrovarniy@gmail.com>
yuriyz added a commit that referenced this issue Nov 14, 2023
Signed-off-by: YuriyZ <yzabrovarniy@gmail.com>
devrimyatar pushed a commit that referenced this issue Dec 30, 2023
Signed-off-by: YuriyZ <yzabrovarniy@gmail.com>
Signed-off-by: Mustafa Baser <mbaser@mail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-auth-server Component affected by issue or PR kind-dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants