Skip to content

Conversation

@bizybot
Copy link
Contributor

@bizybot bizybot commented Aug 10, 2019

Actual invocation of API key service now takes owner flag from the
request into consideration by setting the values for realm and
username as per the current authentication before invoking API key service.
This allows for retrieving or invalidating API keys owned by the current
authenticated user.

Relates: #40031

…service

Actual invocation of API key service now takes `owner` flag from the
request into consideration by setting the values for `realm` and
`username` as per the current authentication. This allows for
retrieving or invalidating API keys owned by the current authenticated
user.

Relates: #
@bizybot bizybot added :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC v8.0.0 v7.4.0 labels Aug 10, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

@bizybot
Copy link
Contributor Author

bizybot commented Aug 10, 2019

Failed due to #43673
@elasticmachine run elasticsearch-ci/1

Copy link
Contributor

@tvernum tvernum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bizybot
Copy link
Contributor Author

bizybot commented Aug 12, 2019

existing grok test failure
@elasticmachine run elasticsearch-ci/2

Copy link
Contributor

@albertzaharovits albertzaharovits left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bizybot bizybot merged commit 1950e38 into elastic:manage-own-api-key-privilege Aug 14, 2019
@colings86 colings86 added >enhancement and removed :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) labels Aug 30, 2019
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.

6 participants