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
Add UUID to api keys #7972
Add UUID to api keys #7972
Conversation
Visit the preview URL for this PR (updated for commit 01add25): https://gloo-edge--pr7972-rinormaloku-add-uuid-ltxyzvcc.web.app (expires Thu, 23 Mar 2023 08:43:36 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 77c2b86e287749579b7ff9cadb81e099042ef677 |
Issues linked to changelog: |
@@ -977,6 +977,8 @@ message ApiKey { | |||
repeated string labels = 3; | |||
// additional data the client needs associated with this API key | |||
map<string, string> metadata = 4; | |||
// Optional: Unique identifier for the API key |
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.
@ashleywang1 can we use here an annotation to hide this from the public docs as this is an implementation detail for storing this info in the storage?
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.
@jbohanon asked in gloo-platform-core and this doesn't seem to be supported
Required to implement the following GH issue: https://github.com/solo-io/gloo-mesh-enterprise/issues/7756
Which has the option for API Keys to be hashed and in order to query API keys we need the UUID