Skip to content

HLRC: Add InvalidateToken security API#35114

Merged
tvernum merged 5 commits intoelastic:masterfrom
tvernum:hlrc/invalidate-token
Nov 6, 2018
Merged

HLRC: Add InvalidateToken security API#35114
tvernum merged 5 commits intoelastic:masterfrom
tvernum:hlrc/invalidate-token

Conversation

@tvernum
Copy link
Copy Markdown
Contributor

@tvernum tvernum commented Oct 31, 2018

This change adds the Invalidate Token API
(DELETE /_xpack/security/oauth2/token) to the Elasticsearch
High Level Rest Client.

Relates: #29827

This change adds the Invalidate Token API
(DELETE /_xpack/security/oauth2/token) to the Elasticsearch
High Level Rest Client.

Relates: elastic#29827
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra

@tvernum
Copy link
Copy Markdown
Contributor Author

tvernum commented Oct 31, 2018

CC: @elastic/es-security

Copy link
Copy Markdown
Contributor

@jkakavas jkakavas left a comment

Choose a reason for hiding this comment

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

LGTM, couldn't find a nit to point out :)

# Conflicts:
#	client/rest-high-level/src/test/java/org/elasticsearch/client/documentation/SecurityDocumentationIT.java
Copy link
Copy Markdown
Contributor

@bizybot bizybot left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you.

Copy link
Copy Markdown
Member

@jaymode jaymode left a comment

Choose a reason for hiding this comment

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

LGTM

@tvernum tvernum merged commit 3776de5 into elastic:master Nov 6, 2018
tvernum added a commit that referenced this pull request Nov 8, 2018
This change adds the Invalidate Token API
(DELETE /_xpack/security/oauth2/token) to the Elasticsearch
High Level Rest Client.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants