Consolidate the authentication result caches #70900
Assignees
Labels
>enhancement
:Security/Authentication
Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc)
Team:Security
Meta label for security team
Comments
ywangd
added
>enhancement
:Security/Authentication
|
Pinging @elastic/es-security (Team:Security) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As discussed over a comment of #70543, there is an opportunity to consolidate existing caches used for authentication results. These are the one used for
CachingUsernamePasswordRealm,ApiKeyServiceandCachingServiceAccountsTokenStore. Their implementations are largely identical. The different authentication result can be generialised. We should be able to have a common implementation for all of them. We could also consider using composition over inheritance when we are at it.The text was updated successfully, but these errors were encountered: