Skip to content

Support "source" parameter in _has_privileges#32310

Merged
tvernum merged 1 commit intoelastic:security-app-privsfrom
tvernum:has-priv-source-param
Jul 24, 2018
Merged

Support "source" parameter in _has_privileges#32310
tvernum merged 1 commit intoelastic:security-app-privsfrom
tvernum:has-priv-source-param

Conversation

@tvernum
Copy link
Copy Markdown
Contributor

@tvernum tvernum commented Jul 24, 2018

The rest tests require that any request that supports GET with body
must also support GET with source="..."

This changes the _has_privileges rest action to support the source
parameter as an alternative to reading a body

The rest tests require that any request that supports GET with body
must also support GET with source="..."

This changes the _has_privileges rest action to support the source
parameter as an alternative to reading a body
@tvernum tvernum added >bug :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC labels Jul 24, 2018
@tvernum tvernum requested a review from jaymode July 24, 2018 02:09
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-security

@tvernum
Copy link
Copy Markdown
Contributor Author

tvernum commented Jul 24, 2018

Prior to this fix, the following test would fail:

/gradlew :x-pack:plugin:integTestRunner -Dtests.seed=77D10D9E252FE2BB -Dtests.class=org.elasticsearch.xpack.test.rest.XPackRestIT -Dtests.method="test {p0=privileges/30_superuser*"

Following this change, that test/seed now passes.

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 32e4f62 into elastic:security-app-privs Jul 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants