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

App: Pass user roles along with oauth/user in request #487

Open
ruscoder opened this issue Mar 21, 2022 · 1 comment
Open

App: Pass user roles along with oauth/user in request #487

ruscoder opened this issue Mar 21, 2022 · 1 comment

Comments

@ruscoder
Copy link

ruscoder commented Mar 21, 2022

Problem I need to solve

In my python application I have few operations that's registered in Aidbox App.
Once operation is called, Aidbox provides oauth/user in request. Sometimes user roles is needed for further calculations.

Solution I see

Aidbox loads list of user roles while evaluating access policies - so, I think it's not a big deal to provide user roles to the App operation. It will decrease number of additional queries from the app.

GET /auth/userinfo already returns user with role - list of roles. Maybe it will be good to have the same structure and just expand request's oauth/user attribute with role attribute to have it unified across policies/userinfo endpoint.

Alternatives I've considered, but they don't work

Currently I make an additional query to fetch user roles.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had activity in the last 30 days. Please make sure you've added all the necessary details including clear description and severity level.

@github-actions github-actions bot added the stale label May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant