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.

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

[Backport 2.x] Adds default roles for Snapshot Management plugin (#1897) #1916

Merged

Conversation

DarshitChanpura
Copy link
Member

@DarshitChanpura DarshitChanpura commented Jun 27, 2022

Signed-off-by: Clay Downs [email protected]
(cherry picked from commit 7f5ca53)

Backport 7f5ca53 from #1897

Check List

- [ ] New functionality includes testing
- [ ] New functionality has been documented

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@DarshitChanpura DarshitChanpura marked this pull request as ready for review June 27, 2022 17:31
@DarshitChanpura DarshitChanpura requested a review from a team June 27, 2022 17:31
@DarshitChanpura DarshitChanpura changed the title Adds default roles for Snapshot Management plugin (#1897) [Backport 2.x] Adds default roles for Snapshot Management plugin (#1897) Jun 27, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #1916 (b868a06) into 2.x (203ecb5) will increase coverage by 0.21%.
The diff coverage is 83.18%.

@@             Coverage Diff              @@
##                2.x    #1916      +/-   ##
============================================
+ Coverage     60.78%   61.00%   +0.21%     
- Complexity     3185     3233      +48     
============================================
  Files           253      256       +3     
  Lines         17931    18088     +157     
  Branches       3204     3224      +20     
============================================
+ Hits          10900    11034     +134     
- Misses         5454     5469      +15     
- Partials       1577     1585       +8     
Impacted Files Coverage Δ
...ic/auth/http/jwt/AbstractHTTPJwtAuthenticator.java 55.81% <ø> (ø)
...t/keybyoidc/AuthenticatorUnavailableException.java 0.00% <ø> (ø)
...th/http/jwt/keybyoidc/BadCredentialsException.java 40.00% <ø> (ø)
...byoidc/HTTPJwtKeyByOpenIdConnectAuthenticator.java 94.11% <ø> (ø)
...azon/dlic/auth/http/jwt/keybyoidc/JwtVerifier.java 85.71% <ø> (ø)
.../dlic/auth/http/jwt/keybyoidc/KeySetRetriever.java 76.82% <ø> (ø)
.../auth/http/jwt/keybyoidc/SelfRefreshingKeySet.java 59.85% <ø> (ø)
...ttp/jwt/oidc/json/OpenIdProviderConfiguration.java 100.00% <ø> (ø)
...ic/auth/http/kerberos/HTTPSpnegoAuthenticator.java 0.00% <ø> (ø)
...azon/dlic/auth/http/kerberos/util/JaasKrbUtil.java 0.00% <ø> (ø)
... and 242 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 028b5a0...b868a06. Read the comment docs.

@peternied peternied merged commit 043360c into opensearch-project:2.x Jun 27, 2022
stephen-crawford pushed a commit to stephen-crawford/security that referenced this pull request Nov 10, 2022
stephen-crawford pushed a commit to stephen-crawford/security that referenced this pull request Nov 10, 2022
…#1897) (opensearch-project#1916)

Signed-off-by: Clay Downs <[email protected]>
(cherry picked from commit 7f5ca53)

Co-authored-by: Clay Downs <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
stephen-crawford pushed a commit to stephen-crawford/security that referenced this pull request Nov 10, 2022
…#1897) (opensearch-project#1916)

Signed-off-by: Clay Downs <[email protected]>
(cherry picked from commit 7f5ca53)

Co-authored-by: Clay Downs <[email protected]>
Signed-off-by: Stephen Crawford <[email protected]>
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants