Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace uses of deprecated TransportClient #1578

Closed
davidlago opened this issue Jan 19, 2022 · 0 comments
Closed

Replace uses of deprecated TransportClient #1578

davidlago opened this issue Jan 19, 2022 · 0 comments
Assignees
@DarshitChanpura
Labels
v2.0.0

Comments

@davidlago
Copy link

davidlago commented Jan 19, 2022 •
edited
Loading

Describe the bug
After the deprecated TransportClient was removed in 2.0, we need to go through the uses in this repo and update the to use a REST client instead.

Current behavior
Building the security plugin against OpenSearch 2.0 gives the following error:

Error:  /home/runner/work/security/security/src/main/java/org/opensearch/security/tools/SecurityAdmin.java:[92,39] cannot find symbol
Error:    symbol:   class TransportClient
Error:    location: package org.opensearch.client.transport

Acceptance criteria
Security plugin builds against OpenSearch 2.0 with no errors, test pass.
@davidlago davidlago mentioned this issue Jan 19, 2022
Closed
3 tasks
@dblock dblock mentioned this issue Feb 9, 2022
Closed
19 tasks
@DarshitChanpura DarshitChanpura self-assigned this Feb 16, 2022
@rs-eliatra rs-eliatra mentioned this issue Feb 23, 2022
Merged
This was referenced Mar 2, 2022
Closed
Merged
@davidlago davidlago mentioned this issue Mar 23, 2022
Closed
jochenkressin added a commit to jochenkressin/security that referenced this issue Mar 23, 2022
@jochenkressin
Remove code related to TransportClient auth/auth (opensearch-project#…
5c11264 
…1578)

Signed-off-by: Jochen Kressin <[email protected]>
jochenkressin added a commit to jochenkressin/security that referenced this issue Mar 23, 2022
@jochenkressin
Remove Transport Auth auditlog categories (opensearch-project#1578)
30d7315 
Signed-off-by: Jochen Kressin <[email protected]>
@jochenkressin jochenkressin mentioned this issue Mar 23, 2022
Merged
3 tasks
peternied pushed a commit to peternied/security that referenced this issue Apr 7, 2022
@jochenkressin @peternied
Remove code related to TransportClient auth/auth (opensearch-project#…
779c09f 
…1578)

Signed-off-by: Jochen Kressin <[email protected]>
peternied pushed a commit to peternied/security that referenced this issue Apr 7, 2022
@jochenkressin @peternied
Remove Transport Auth auditlog categories (opensearch-project#1578)
600f129 
Signed-off-by: Jochen Kressin <[email protected]>
jochenkressin added a commit to jochenkressin/security that referenced this issue Apr 10, 2022
@jochenkressin
Enforce privileges evaluation for injected user (opensearch-project#1578
bbd7636 
)

While we do not need any transport client auth/auth code anymore,
we still need to check the privileges of an injected user on
transport layer. This functionality was removed by mistake and
is reenabled by this commit.

Signed-off-by: Jochen Kressin <[email protected]>
davidlago pushed a commit that referenced this issue Apr 12, 2022
@jochenkressin
Remove TransportClient auth/auth (#1701)
ef0d40a 
* Remove code related to TransportClient auth/auth (#1578)

Signed-off-by: Jochen Kressin <[email protected]>
@prudhvigodithi prudhvigodithi mentioned this issue Jun 20, 2022
Closed
wuychn pushed a commit to ochprince/security that referenced this issue Mar 16, 2023
@jochenkressin
Remove TransportClient auth/auth (opensearch-project#1701)
29cfb85 
* Remove code related to TransportClient auth/auth (opensearch-project#1578)

Signed-off-by: Jochen Kressin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DarshitChanpura DarshitChanpura

Labels
v2.0.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@davidlago @cliu123 @DarshitChanpura