Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add support to retrieve all API keys if user has privilege #47274

Merged
merged 3 commits into from
Oct 7, 2019
Merged

Add support to retrieve all API keys if user has privilege #47274

merged 3 commits into from
Oct 7, 2019

Conversation

bizybot
Copy link
Contributor

@bizybot bizybot commented Sep 30, 2019

This commit adds support to retrieve all API keys if the authenticated
user is authorized to do so.
This removes the restriction of specifying one of the
parameters (like id, name, username and/or realm name)
when owner is set to false.

Closes #46887

Add support to retrieve all API keys if user has privilege
8b2e516 
This commit adds support to retrieve all API keys if the
authenticated user is authorized to do so.
This removes the restriction of specifying one of the
parameters (like id, name, username and/or realm name)
when `owner` is set to `false`.

Closes elastic#46887
@bizybot bizybot added >enhancement :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC v8.0.0 v7.5.0 labels Sep 30, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

@bizybot
Copy link
Contributor Author

bizybot commented Sep 30, 2019

@elasticmachine run elasticsearch-ci/2

albertzaharovits
albertzaharovits approved these changes Oct 1, 2019
View reviewed changes
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

@cjcenizal cjcenizal mentioned this pull request Oct 3, 2019
Merged
@bizybot bizybot merged commit 01f9177 into elastic:master Oct 7, 2019
bizybot added a commit to bizybot/elasticsearch that referenced this pull request Oct 7, 2019
@bizybot
Add support to retrieve all API keys if user has privilege (elastic#4…
50c2c06 
…7274)

This commit adds support to retrieve all API keys if the authenticated
user is authorized to do so.
This removes the restriction of specifying one of the
parameters (like id, name, username and/or realm name)
when the `owner` is set to `false`.

Closes elastic#46887
bizybot added a commit that referenced this pull request Oct 7, 2019
@bizybot
Add support to retrieve all API keys if user has privilege (#47274) (#…
7c862fe 
…47641)

This commit adds support to retrieve all API keys if the authenticated
user is authorized to do so.
This removes the restriction of specifying one of the
parameters (like id, name, username and/or realm name)
when the `owner` is set to `false`.

Closes #46887
@Mpdreamz Mpdreamz mentioned this pull request Nov 19, 2019
Closed
24 tasks
@codebrain codebrain mentioned this pull request Dec 16, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@albertzaharovits albertzaharovits albertzaharovits approved these changes

@tvernum tvernum tvernum approved these changes

Assignees
No one assigned
Labels
>enhancement :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC v7.5.0 v8.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support retrieval of all API keys
5 participants
@bizybot @elasticmachine @tvernum @albertzaharovits @jakelandis