Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 4b38671 from #2808

* [FEATURE] usage of JWKS with JWT (w/o OpenID connect)

Signed-off-by: Sebastian Michalski <shekerama@gmail.com>

* Change issuer to audience

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Switch to OCT_2 and use String instead of var

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Only check required issuer and required audience if set

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Fix test cases

Signed-off-by: Craig Perkins <cwperx@amazon.com>

* Remove unused import

Signed-off-by: Craig Perkins <cwperx@amazon.com>

---------

Signed-off-by: Sebastian Michalski <shekerama@gmail.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Co-authored-by: Sebastian Michalski <shekerama@gmail.com>
(cherry picked from commit 4b38671)
@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Merging #2815 (beb441c) into 2.8 (2ebcfa7) will increase coverage by 0.15%.
The diff coverage is 91.42%.

@@             Coverage Diff              @@
##                2.8    #2815      +/-   ##
============================================
+ Coverage     61.38%   61.54%   +0.15%     
- Complexity     3370     3384      +14     
============================================
  Files           264      264              
  Lines         18631    18659      +28     
  Branches       3283     3286       +3     
============================================
+ Hits          11437    11483      +46     
+ Misses         5610     5591      -19     
- Partials       1584     1585       +1     
Impacted Files Coverage Δ
...ic/auth/http/jwt/AbstractHTTPJwtAuthenticator.java 58.42% <60.00%> (+3.13%) ⬆️
...byoidc/HTTPJwtKeyByOpenIdConnectAuthenticator.java 90.00% <80.00%> (-4.12%) ⬇️
...azon/dlic/auth/http/jwt/keybyoidc/JwtVerifier.java 88.67% <100.00%> (+2.63%) ⬆️
.../dlic/auth/http/jwt/keybyoidc/KeySetRetriever.java 79.56% <100.00%> (+2.74%) ⬆️

... and 4 files with indirect coverage changes

@RyanL1997 RyanL1997 merged commit ce5b295 into 2.8 May 30, 2023
@RyanL1997 RyanL1997 deleted the backport/backport-2808-to-2.8 branch May 30, 2023 22:46
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.

3 participants