Add early expiration of Access Token in cache (#233)

* Add early expiration of Access Token in cache * Add tests and early expire the token after 80%
auth0 · Oct 3, 2019 · a09dd1c · a09dd1c
1 parent fd21bc5
commit a09dd1c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/__tests__/cache.test.ts b/__tests__/cache.test.ts
@@ -54,7 +54,9 @@ describe('cache', () => {
         user: { name: 'Test' }
       }
     });
-    jest.advanceTimersByTime(1001);
+    jest.advanceTimersByTime(799);
+    expect(Object.keys(cache.cache).length).toBe(1);
+    jest.advanceTimersByTime(1);
     expect(Object.keys(cache.cache).length).toBe(0);
   });
   it('expires after `user.exp` when `user.exp` < `expires_in`', () => {
@@ -73,7 +75,9 @@ describe('cache', () => {
         user: { name: 'Test' }
       }
     });
-    jest.advanceTimersByTime(1001);
+    jest.advanceTimersByTime(799);
+    expect(Object.keys(cache.cache).length).toBe(1);
+    jest.advanceTimersByTime(1);
     expect(Object.keys(cache.cache).length).toBe(0);
   });
 });
diff --git a/src/cache.ts b/src/cache.ts
@@ -24,7 +24,7 @@ const createKey = (e: CacheKeyData) => `${e.audience}::${e.scope}`;
 const getExpirationTimeoutInMilliseconds = (expiresIn: number, exp: number) => {
   const expTime =
     (new Date(exp * 1000).getTime() - new Date().getTime()) / 1000;
-  return Math.min(expiresIn, expTime) * 1000;
+  return Math.min(expiresIn, expTime) * 1000 * 0.8;
 };
 
 export default class Cache {