-
Notifications
You must be signed in to change notification settings - Fork 5
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
[radar-gateway] Update configmap variables #270
Conversation
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files. |
charts/radar-gateway/values.yaml
Outdated
@@ -228,3 +228,7 @@ cc: | |||
schemaRegistryApiKey: srApiKey | |||
# -- Confluent Cloud schema registry API secret | |||
schemaRegistryApiSecret: srApiSecret | |||
|
|||
public_key_endpoints: | |||
# List of public key endpoints for token verification |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The documentation should be above the item
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have updated the doc and default. LGTM
Thanks! |
@mpgxvii @yatharthranjan I was trying to apply this update and got this error message:
|
Description of the change
authorizer.yml
configmap to allow setting of public key urls for token verification (other than MP)Benefits
Possible drawbacks
Applicable issues
Additional information
Checklist
Chart.yaml
according to semver.