Skip to content

Comments

Update style of all java files under **/auth#2822

Closed
stephen-crawford wants to merge 3 commits intoopensearch-project:mainfrom
stephen-crawford:updateStyleAuth
Closed

Update style of all java files under **/auth#2822
stephen-crawford wants to merge 3 commits intoopensearch-project:mainfrom
stephen-crawford:updateStyleAuth

Conversation

@stephen-crawford
Copy link
Contributor

Description

Update style of all java files under **/auth follows #2821

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.

Signed-off-by: Stephen Crawford <steecraw@amazon.com>
Signed-off-by: Stephen Crawford <steecraw@amazon.com>
@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Merging #2822 (c4618ea) into main (5f9c90b) will decrease coverage by 0.01%.
The diff coverage is 57.84%.

@@             Coverage Diff              @@
##               main    #2822      +/-   ##
============================================
- Coverage     61.61%   61.60%   -0.01%     
- Complexity     3413     3414       +1     
============================================
  Files           266      266              
  Lines         18893    18908      +15     
  Branches       3305     3303       -2     
============================================
+ Hits          11640    11648       +8     
- Misses         5656     5666      +10     
+ Partials       1597     1594       -3     
Impacted Files Coverage Δ
...ic/auth/http/jwt/AbstractHTTPJwtAuthenticator.java 58.42% <0.00%> (ø)
...ic/auth/http/kerberos/HTTPSpnegoAuthenticator.java 0.00% <0.00%> (ø)
...azon/dlic/auth/http/kerberos/util/JaasKrbUtil.java 0.00% <0.00%> (ø)
...zon/dlic/auth/http/kerberos/util/KrbConstants.java 0.00% <0.00%> (ø)
.../dlic/auth/http/saml/SamlHTTPMetadataResolver.java 62.96% <ø> (ø)
.../main/java/com/amazon/dlic/auth/ldap/LdapUser.java 85.71% <ø> (ø)
...a/org/opensearch/security/DefaultObjectMapper.java 64.55% <ø> (ø)
.../action/configupdate/ConfigUpdateNodeResponse.java 77.27% <ø> (ø)
...tion/configupdate/TransportConfigUpdateAction.java 100.00% <ø> (ø)
.../security/action/whoami/TransportWhoAmIAction.java 33.33% <0.00%> (ø)
... and 122 more

Signed-off-by: Stephen Crawford <steecraw@amazon.com>
@stephen-crawford
Copy link
Contributor Author

Closing since #2823 got merged and contains these changes.

@stephen-crawford stephen-crawford deleted the updateStyleAuth branch June 5, 2023 14:52
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.

2 participants