github/directus
1
1
Fork 0
mirror of https://github.com/directus/directus.git synced 2026-04-25 03:00:53 -04:00
Code Issues Packages Projects Releases Wiki Activity

Don't try auto-refreshing on access token expiry times > 32bit integers (#4102)

Browse Source 
Fixes #4054
This commit is contained in:
Rijk van Zanten
2021-02-16 15:25:16 -05:00
committed by GitHub
parent 7830453cfc
commit 602be1fd48
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
app/src/auth.ts
View File

@@ -24,7 +24,12 @@ export async function login(credentials: LoginCredentials) {
// Refresh the token 10 seconds before the access token expires. This means the user will stay
// logged in without any noticable hickups or delays
setTimeout(() => refresh(), response.data.data.expires - 10000);
// setTimeout breaks with numbers bigger than 32bits. This ensures that we don't try refreshing
// for tokens that last > 24 days. Ref #4054
if (response.data.data.expires <= 2100000000) {
setTimeout(() => refresh(), response.data.data.expires - 10000);
}
appStore.state.authenticated = true;
@@ -47,7 +52,12 @@ export async function refresh({ navigate }: LogoutOptions = { navigate: true })
// Refresh the token 10 seconds before the access token expires. This means the user will stay
// logged in without any noticable hickups or delays
if (refreshTimeout) clearTimeout(refreshTimeout);
refreshTimeout = setTimeout(() => refresh(), response.data.data.expires - 10000);
// setTimeout breaks with numbers bigger than 32bits. This ensures that we don't try refreshing
// for tokens that last > 24 days. Ref #4054
if (response.data.data.expires <= 2100000000) {
refreshTimeout = setTimeout(() => refresh(), response.data.data.expires - 10000);
}
appStore.state.authenticated = true;
return accessToken;