-
Notifications
You must be signed in to change notification settings - Fork 409
feat(jmespath): add jmespath support #381
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
Conversation
|
Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement. After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.
Regards, |
1bedf09 to
7faf9ca
Compare
|
Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement. After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.
Regards, |
7faf9ca to
653d11b
Compare
|
Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement. After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.
Regards, |
Signed-off-by: Marco Klaassen <klaassen.marco@googlemail.com>
653d11b to
1bf27ea
Compare
|
Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement. After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.
Regards, |
|
Cool feature, thank you. Putting this into a v0.14 milestone since we are behind on getting v0.13 out |
Signed-off-by: Josh Dolitsky <josh@dolit.ski>
Signed-off-by: Josh Dolitsky <josh@dolit.ski>
|
@marcoklaassen - thanks for you patience with this, merged |
JMESPath Support
Not every JWT token provider is able to define the token the way the chartmuseum wants it.
In my case, my sso produces e.g.:
If you are not able to configure the JWT token provider yourself, i think it is a good idea to provide a solution by the chartmuseum.
With JMESPath Support we can make the path to find the allowed actions configurable.
To avoid a breaking change, i defined a default JMESPath in the auth module. This default configuration does the same job as the old implementation.
The pull request for the auth repository: chartmuseum/auth#10