Skip to content

[Backport 2.x] fix-expiration-of-token-from-actual-idtoken#1091

Merged
peternied merged 1 commit into2.xfrom
backport/backport-1040-to-2.x
Aug 31, 2022
Merged

[Backport 2.x] fix-expiration-of-token-from-actual-idtoken#1091
peternied merged 1 commit into2.xfrom
backport/backport-1040-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 057cbe4 from #1040

Use expiration of tokens from the id token

The like oidc token response from Gitlab expiresIn property is not the actual expiration of the idToken. idToken expires before the access_token does. This fix allows the security plugin used when integrating with Gitlab

Signed-off-by: Vasile Negru <vasile@eosfintek.com>
(cherry picked from commit 057cbe4)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team August 31, 2022 17:10
@codecov-commenter
Copy link

Codecov Report

Merging #1091 (057cbe4) into 2.x (3d43bdb) will not change coverage.
The diff coverage is n/a.

❗ Current head 057cbe4 differs from pull request most recent head 0aea5a4. Consider uploading reports for the commit 0aea5a4 to get more accurate results

@@           Coverage Diff           @@
##              2.x    #1091   +/-   ##
=======================================
  Coverage   72.27%   72.27%           
=======================================
  Files          87       87           
  Lines        1915     1915           
  Branches      244      244           
=======================================
  Hits         1384     1384           
  Misses        478      478           
  Partials       53       53           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@peternied peternied merged commit 7212d19 into 2.x Aug 31, 2022
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