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

Fix JWT unauthorized/expired issue #73

Merged
merged 1 commit into from
Jan 16, 2025
Merged

Fix JWT unauthorized/expired issue #73

merged 1 commit into from
Jan 16, 2025

Conversation

octref
Copy link
Member

@octref octref commented Jan 16, 2025

Description

For an expired JWT token, we used to only show this error message

image

Now we do this:

image

Also added some overall error handling for our fetch

Checklist

  • Code compiles correctly and linting passes locally
  • Tests added for new functionality, or regression tests for bug fixes added as applicable
  • For new features, etc., PR on docs repo staged and
    linked here

Instructions

  • The PR title should follow the Conventional Commits
    syntax, leading with fix:, feat:, chore:, ci:, etc.
  • The description should briefly explain what the PR is about. In the case of a bugfix, describe or
    link to the bug.
  • In the checklist section, check the boxes in that are applicable, using [x] syntax.
    • If not applicable, remove the entire line. Only leave the box unchecked if you intend to come
      back and check the box later.
  • Delete the Instructions line and everything below it, to indicate you have read and are
    following these instructions. 🙂

Thank you for your contribution to the Modus project!

@octref octref requested a review from a team as a code owner January 16, 2025 17:36
@octref octref enabled auto-merge (squash) January 16, 2025 17:37
@octref octref merged commit ee47e9f into main Jan 16, 2025
5 checks passed
@octref octref deleted the pine/HYP-2745 branch January 16, 2025 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants