Skip to content

[HLRC] Remove SecurityClient #85983

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.

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

Merged
merged 3 commits into from
Apr 21, 2022

Conversation

tvernum
Copy link
Contributor

@tvernum tvernum commented Apr 19, 2022

The High Level Rest Client (HLRC) is deprecated and is no longer
shipped as a standalone artifact.
It can now be safely removed from the ES codebase.

This commit removes all remaining methods from the HLRC
SecurityClient, and removes the SecurityClient as well as all
Request and Response objects.

Test code that depended on these methods has been migrated
to use TestSecurityClient instead.

Relates: #83423

@tvernum tvernum added >test Issues or PRs that are addressing/adding tests :Clients/Java High Level REST Client :Security/Security Security issues without another label v8.3.0 labels Apr 19, 2022
@elasticmachine elasticmachine added Team:Security Meta label for security team Team:Clients Meta label for clients team labels Apr 19, 2022
@elasticmachine
Copy link
Collaborator

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

@elasticmachine
Copy link
Collaborator

Pinging @elastic/clients-team (Team:Clients)

@tvernum tvernum mentioned this pull request Apr 19, 2022
66 tasks
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

@tvernum tvernum merged commit 997317d into elastic:master Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Security/Security Security issues without another label Team:Clients Meta label for clients team Team:Security Meta label for security team >test Issues or PRs that are addressing/adding tests v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants