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

support external jwt claims to be passed downstream #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ojafri
Copy link

@ojafri ojafri commented Jun 10, 2019

enhances extauth plugin to support extracting claims from externally issued authorization bearer jwt payload and add them as a customizable request header extauth-claims-header to be passed downstream.

This is useful for passing scopes and other relevant information to downstream when authorization/access token is stripped keep-authorization-header: false

It also supports customizing which of the private/sensitive claims from payload are to be omitted via extauth-exclude-claims config property

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@ojafri ojafri changed the title support external jwt claims to be passed downsrream support external jwt claims to be passed downstream Jun 10, 2019
@ojafri
Copy link
Author

ojafri commented Jun 10, 2019

I signed it!

@ojafri
Copy link
Author

ojafri commented Jun 10, 2019 via email

@dkoroth dkoroth requested a review from srinandan June 10, 2019 20:02
@ojafri
Copy link
Author

ojafri commented Jul 2, 2019

can i please request a review for this pr, this has been pending for 3+ weeks already. If this won't be reviewed, I'll close it, but would very much appreciate if it can be commented upon what's the problem

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

Successfully merging this pull request may close these issues.

2 participants