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

Limit access with Keycloak based on role/group? #504

Closed
tboerger opened this issue Oct 24, 2022 · 6 comments
Closed

Limit access with Keycloak based on role/group? #504

tboerger opened this issue Oct 24, 2022 · 6 comments
Labels

Comments

@tboerger
Copy link

Describe the problem

Is it possible to limit the access based on Vouch and Keycloak as an IdP on roles or groups similar to orgs and teams with the GitHub provider?

Expected behavior

Limiting access based on Keycloak roles or groups

@ShyLionTjmn
Copy link

You may request groups claim and put it into headers. Then read them in HTTP request in your app and do whatever you like, based on group membership.

@tboerger
Copy link
Author

The app does not provide any option like that. I thought hopefully Vouch supports limitung access based on groups like it's oauth2-proxy doing.

@ShyLionTjmn
Copy link

Well, it is not what vouch were designed for, i guess.
But, if you are using Keycloak, you may just fine-tune your realm, i think.

@bnfinet
Copy link
Member

bnfinet commented Oct 27, 2022

@tboerger Hello there!

The suggested method for doing such is currently to use the lua enabled version of Nginx called "open resty". Please see the /examples directory in this repo for detailed documentation.

I'd like to add additional functionality for groups facilities (or really RBAC on any claim) to VP but there are a few other items ahead of this one and I'm currently buried in other projects.

I'm going to close this in favor of #175 but feel free to ask any additional questions.

@ShyLionTjmn Thanks for chiming in and being helpful. Can I ask you to please let people know that you are relatively new to VP when you answer support questions like this. I think that context is important.

@bnfinet bnfinet closed this as completed Oct 27, 2022
@tboerger
Copy link
Author

Thanks for the feedback. Sadly open resty wouldn't work for me as I want to avoid switching my ingress controller.

But since you are open for such a feature I will dig through the code and see if I can contribute something similar to the suggestions mentioned eighth the linked issue.

@bnfinet
Copy link
Member

bnfinet commented Oct 27, 2022

@tboerger that would be wonderful! I'll leave a few more notes in #175

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

No branches or pull requests

3 participants