-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Closed
Labels
:Security/AuthenticationLogging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc)Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc)>enhancementv8.0.0-alpha1
Description
The current methods for checking API key creator realm (ApiKeyService#getCreatorRealmXxx) can be improved by:
- Use sourceRealm instead authenticatedBy so lookedUp is always in effect (if there is one).
Right now this does not cause real problem since API key cannot impersonate another key.
But it is better to be consistent with other checks and be future proof. - Test for AuthenticateType is better done using Enum instead of string comparison.
Metadata
Metadata
Assignees
Labels
:Security/AuthenticationLogging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc)Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc)>enhancementv8.0.0-alpha1