Skip to content

Retain refresh token expiry on refreshing #360

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.

Sign up for GitHub

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

Merged
merged 13 commits into from
May 17, 2022
Merged

Retain refresh token expiry on refreshing #360

Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
f095ebc
fix refresh token expiry lost during refresh
chrisirhc Apr 20, 2022
22a3c9d
add failing test
chrisirhc Apr 20, 2022
73efd80
revert previous changes to go with a different approach
chrisirhc Apr 28, 2022
9a80748
stab at expires at variables
chrisirhc Apr 28, 2022
d53fd30
fix test and typo
chrisirhc Apr 28, 2022
cbf79c5
fix typo
chrisirhc Apr 28, 2022
bdf2283
clean up code for easier review, minimize changes, follow previus pat…
chrisirhc Apr 28, 2022
f062572
add missing var
chrisirhc Apr 28, 2022
ba5c11a
fix canRefresh logic
chrisirhc May 17, 2022
e28902c
add setExpiresAt tests
chrisirhc May 17, 2022
355fb89
remove extra comment
chrisirhc May 17, 2022
a38cab9
add tests and fix typo in logic
chrisirhc May 17, 2022
defbfd1
Merge branch 'master' into fix-refresh-token-expires
sqrrrl May 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view

These merge commits were added into this branch cleanly.

There are no new changes to show.