Skip to content

Remove enable/disableUser methods from HLRC#83626

Merged
tvernum merged 1 commit intoelastic:masterfrom
tvernum:hlrc/remove-enable-user
Feb 11, 2022
Merged

Remove enable/disableUser methods from HLRC#83626
tvernum merged 1 commit intoelastic:masterfrom
tvernum:hlrc/remove-enable-user

Conversation

@tvernum
Copy link
Copy Markdown
Contributor

@tvernum tvernum commented Feb 8, 2022

Removes the enableUser and disableUser methods from the High Level
Rest Client's SecurityClient and replaces existing usage in tests
with a helper method.

Relates: #83423

Removes the `enableUser` and `disableUser` methods from the High Level
Rest Client's `SecurityClient` and replaces existing usage in tests
with a helper method.

Relates: elastic#83423
@elasticmachine elasticmachine added Team:Security Meta label for security team Team:Data Management (obsolete) DO NOT USE. This team no longer exists. labels Feb 8, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@tvernum
Copy link
Copy Markdown
Contributor Author

tvernum commented Feb 8, 2022

Sorry Albert, this one is only +29 -175

@tvernum
Copy link
Copy Markdown
Contributor Author

tvernum commented Feb 10, 2022

Ping @albertzaharovits

Copy link
Copy Markdown
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

@tvernum tvernum merged commit 2d79dcc into elastic:master Feb 11, 2022
@tvernum tvernum mentioned this pull request Feb 16, 2022
66 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>non-issue :Security/Security Security issues without another label Team:Data Management (obsolete) DO NOT USE. This team no longer exists. Team:Security Meta label for security team v8.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants