Skip to content
This repository has been archived by the owner on Apr 11, 2024. It is now read-only.

Increase JWT validation tolerance #462

Merged
merged 1 commit into from
Aug 22, 2022
Merged

Increase JWT validation tolerance #462

merged 1 commit into from
Aug 22, 2022

Conversation

paulomarg
Copy link
Contributor

WHY are these changes introduced?

We currently have a 5s grace period for validating JWTs from session token, but there are cases where that is not long enough and we still end up failing to validate the token.

WHAT is this pull request doing?

Increase the period to 10s which seems to eliminate the problem for good.

Type of change

  • Patch: Bug (non-breaking change which fixes an issue)

Checklist

  • I have added a changelog entry, prefixed by the type of change noted above

@paulomarg paulomarg requested a review from a team as a code owner August 22, 2022 16:01
@paulomarg paulomarg force-pushed the increase_jwt_leeway branch from 9722ab5 to ffea7e5 Compare August 22, 2022 16:01
Copy link

@tatsuya tatsuya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@paulomarg paulomarg merged commit ee78e14 into main Aug 22, 2022
@paulomarg paulomarg deleted the increase_jwt_leeway branch August 22, 2022 16:18
@shopify-shipit shopify-shipit bot temporarily deployed to production September 19, 2022 19:28 Inactive
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants