Skip to content

Bulk invalidate API keys using a list of IDs (#63224)#63320

Merged
ywangd merged 9 commits intoelastic:7.xfrom
ywangd:es-47609-invalidate-api-keys-with-ids-7.x
Oct 6, 2020
Merged

Bulk invalidate API keys using a list of IDs (#63224)#63320
ywangd merged 9 commits intoelastic:7.xfrom
ywangd:es-47609-invalidate-api-keys-with-ids-7.x

Conversation

@ywangd
Copy link
Copy Markdown
Member

@ywangd ywangd commented Oct 6, 2020

Add a new ids field to the API of invalidating API keys so that it supports bulk
invalidation with a list of IDs.
Note the existing id field is kept as is and it is an error if both id and ids are specified.

ywangd added 2 commits October 6, 2020 22:24
)

Getting the API key document form the security index is the most time consuing part
of the API Key authentication flow (>60% if index is local and >90% if index is remote).
This traffic is now avoided by caching added with this PR.

Additionally, we add a cache invalidator registry so that clearing of different caches will
be managed in a single place (requires follow-up PRs).
@ywangd ywangd added the backport label Oct 6, 2020
ywangd added 2 commits October 6, 2020 22:37
Add a new ids field to the API of invalidating API keys so that it supports bulk
invalidation with a list of IDs.
Note the existing id field is kept as is and it is an error if both id and ids are specified.
@ywangd ywangd force-pushed the es-47609-invalidate-api-keys-with-ids-7.x branch from 8eda735 to 47fae33 Compare October 6, 2020 11:38
@ywangd
Copy link
Copy Markdown
Member Author

ywangd commented Oct 6, 2020

@elasticmachine run elasticsearch-ci/packaging-sample-windows

@ywangd ywangd merged commit abf9b88 into elastic:7.x Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant