Skip to content

HLRC support for query API key API#76520

Merged
ywangd merged 5 commits intoelastic:masterfrom
ywangd:api-key-search-hlrc
Aug 17, 2021
Merged

HLRC support for query API key API#76520
ywangd merged 5 commits intoelastic:masterfrom
ywangd:api-key-search-hlrc

Conversation

@ywangd
Copy link
Copy Markdown
Member

@ywangd ywangd commented Aug 14, 2021

@ywangd ywangd force-pushed the api-key-search-hlrc branch from fbdfaa6 to 898c71d Compare August 14, 2021 05:14
@ywangd ywangd mentioned this pull request Aug 14, 2021
5 tasks
@ywangd ywangd force-pushed the api-key-search-hlrc branch 3 times, most recently from 749f9e3 to 40e49ad Compare August 14, 2021 06:36
@ywangd ywangd force-pushed the api-key-search-hlrc branch from 40e49ad to 9f8c5a0 Compare August 14, 2021 07:01
@ywangd ywangd marked this pull request as ready for review August 16, 2021 00:16
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Aug 16, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-security (Team:Security)

@ywangd ywangd requested a review from tvernum August 16, 2021 00:16
return metadata;
}

public Object[] getSortValues() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we javadoc this method?
We typically don't (and rely on the API docs) but this field is weird, since it only exists for ApiKey objects that came from _query/

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good point. Added, thanks!

ywangd and others added 2 commits August 16, 2021 14:13
…documentation/SecurityDocumentationIT.java

Co-authored-by: Tim Vernum <tim@adjective.org>
@ywangd
Copy link
Copy Markdown
Member Author

ywangd commented Aug 17, 2021

@elasticmachine update branch

@ywangd ywangd added the auto-backport Automatically create backport pull requests when merged label Aug 17, 2021
@ywangd ywangd merged commit 7bb1185 into elastic:master Aug 17, 2021
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💔 Backport failed

Status Branch Result
7.x Commit could not be cherrypicked due to conflicts

To backport manually run backport --upstream elastic/elasticsearch --pr 76520

ywangd added a commit to ywangd/elasticsearch that referenced this pull request Aug 17, 2021
This PR adds HLRC for the new Query API key API added with elastic#75335 and elastic#76144

Relates: elastic#71023
ywangd added a commit that referenced this pull request Aug 18, 2021
This PR adds HLRC for the new Query API key API added with #75335 and #76144

Relates: #71023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >enhancement :Security/Security Security issues without another label Team:Security Meta label for security team v7.15.0 v8.0.0-alpha2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants