Skip to content

Support invalidating API keys with multiple IDs#63081

Closed
ywangd wants to merge 2 commits intoelastic:masterfrom
ywangd:es-47609-invalidate-api-keys-with-ids
Closed

Support invalidating API keys with multiple IDs#63081
ywangd wants to merge 2 commits intoelastic:masterfrom
ywangd:es-47609-invalidate-api-keys-with-ids

Conversation

@ywangd
Copy link
Copy Markdown
Member

@ywangd ywangd commented Sep 30, 2020

The existing id field of API invalidation request only takes a single
API key ID. This PR adds a new ids field that takes an array of API
key IDs for bulk invalidation.

Due to the user facing nature of this change, we need:

  • Deprecation handling for the existing id field
  • Unit and Integ tests for both id and ids
  • YAML tests for both id and ids
  • Doc update and tests for both id and ids
  • Client code, doc and tests for both id and ids

I am not labeling it as v7.10.0 because I am not 100% sure this could make it. The changes are wider than I expected. In addition to the above list, we also have:

  • Any tests using API key invalidation (with ID) as a helper needs to handle deprecation warning
  • Manage own API key needs to adjust for testing multiple IDs

Resolves: #47609

@ywangd ywangd added >enhancement :Security/Security Security issues without another label v8.0.0 labels Sep 30, 2020
@ywangd
Copy link
Copy Markdown
Member Author

ywangd commented Oct 1, 2020

Superseded by: #63127

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>enhancement :Security/Security Security issues without another label v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support invalidating multiple API keys by their ID

2 participants