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(java-sdk): fix token validity check for expiry #282

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

jimmyjames
Copy link
Contributor

@jimmyjames jimmyjames commented Jan 24, 2024

Fixes bug in AccessToken#isValid() that caused it to return false when the token is valid, resulting in extraneous calls to fetch new tokens.

Description

Fixes the logic used to check if the token is valid based on its expiry.

References

openfga/java-sdk#48

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

@jimmyjames jimmyjames requested a review from a team as a code owner January 24, 2024 23:08
@jimmyjames jimmyjames force-pushed the fix/java-sdk-token-exp-check branch from f7dbd5a to e03a16d Compare January 24, 2024 23:18
@rhamzeh rhamzeh added this pull request to the merge queue Jan 25, 2024
Merged via the queue into main with commit 7a066a1 Jan 25, 2024
12 checks passed
@rhamzeh rhamzeh deleted the fix/java-sdk-token-exp-check branch January 25, 2024 14:33
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