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

Add smallrye-jwt and smallrye-jwt-build to existing services #1761

Open
jletberg opened this issue Apr 22, 2024 · 5 comments
Open

Add smallrye-jwt and smallrye-jwt-build to existing services #1761

jletberg opened this issue Apr 22, 2024 · 5 comments

Comments

@jletberg
Copy link

jletberg commented Apr 22, 2024

[Feature request]
It is possible to make smallrye-jwt and smallrye-jwt-build extensions available out of the box for existing Kogito Services (such as data-index, jobs-service, etc.)?

@ricardozanini
Copy link
Member

Hi! Can you give some context? What's the use case?

@jletberg
Copy link
Author

jletberg commented May 16, 2024

Hi! The context is that I run Kogito in AWS cloud (using EKS) and looking for a way to get rid of the Keycloak completely. As a Keycloak alternative, I have an existing authentication solution with Azure AD, which enables getting a cookie with JWT in the web browser. But I need the Kogito to decode that JWT, so I can implement the authorization properly to ensure the user can see only what he is meant to see (according to the groups). It is easy with the custom Kogito service as the source code is in my hands. However, access to the Kogito platform services won't work unless they can read the same JWT and treat a user as authenticated (which is not the case, as JWT support is not there by default).

@ricardozanini
Copy link
Member

Have you considered using the images as base for your custom one?
But I believe we can add an ARG in these images to accept adding external dependencies with ease. So that you can build your own Containerfile using these services as bases. WDYT?

@jletberg
Copy link
Author

Extending the Kogito images was one of the options I identified as a workaround - I'll continue investigating this approach. But adding an ARG will be a great improvement! Please do that.

BR,
Juri

@ricardozanini
Copy link
Member

@jletberg feel free to send a PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants