fix(authService): limit setTimeout to it's maximum value of about ca …

…24.85d fixes #375
SpoonX · Apr 23, 2018 · 3c51df6 · 3c51df6
1 parent 6d0f681
commit 3c51df6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/doc/api_authService.md b/doc/api_authService.md
@@ -95,6 +95,8 @@ Sets the login timeout.
 
 CAUTION: .authenticated and isAuthenticated() might get different results when set manually.
 
+WARNING: Maximum timeout is 2^31 - 1 ms (ca. 24.85d).
+
 #### Parameters
 
 | Parameter | Type     | Description        |

diff --git a/src/authService.js b/src/authService.js
@@ -148,6 +148,14 @@ export class AuthService {
    * @param  {number} ttl  Timeout time in ms
    */
   setTimeout(ttl: number) {
+    const maxTimeout = 2147483647; // maximum period in ms (ca. 24.85d) for windows.setTimeout
+
+    // limit timer ttl to max value allowed for windows.setTimeout function
+    if (ttl > maxTimeout) {
+      ttl = maxTimeout;
+      logger.warn('Token timeout limited to ', maxTimeout, ' ms (ca 24.85d).');
+    }
+
     this.clearTimeout();
 
     const expiredTokenHandler = () => {