fix(transporter): error for too high timeout values

fixes #1271 DX-1911
algolia · May 21, 2021 · 142d5e2 · 142d5e2
1 parent 84355a9
commit 142d5e2
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/packages/transporter/src/createTransporter.ts b/packages/transporter/src/createTransporter.ts
@@ -9,6 +9,9 @@ import {
 } from '.';
 import { retryableRequest } from './concerns/retryableRequest';
 
+// setTimeout is using uint32, so this is the maximum in ms and 5x is the max timeout multiplier
+const MAX_TIMEOUT_VALUE = 0x7fffffff / 5000;
+
 export function createTransporter(options: TransporterOptions): Transporter {
   const {
     hostsCache,
@@ -23,6 +26,16 @@ export function createTransporter(options: TransporterOptions): Transporter {
     headers,
   } = options;
 
+  if (
+    timeouts.connect > MAX_TIMEOUT_VALUE ||
+    timeouts.read > MAX_TIMEOUT_VALUE ||
+    timeouts.write > MAX_TIMEOUT_VALUE
+  ) {
+    throw new Error(
+      `Timeout values can be no higher than setTimeout accepts (in seconds), ${MAX_TIMEOUT_VALUE}`
+    );
+  }
+
   const transporter: Transporter = {
     hostsCache,
     logger,