Fixes http timeout issues in node 12.

firebase · Jul 8, 2020 · dc523db · dc523db
1 parent 258fcfc
commit dc523db
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/utils/api-request.ts b/src/utils/api-request.ts
@@ -496,11 +496,15 @@ class AsyncHttpCall {
     });
 
     const timeout: number | undefined = this.config.timeout;
+    const timeoutCallback = () => {
+      req.abort();
+      this.rejectWithError(`timeout of ${timeout}ms exceeded`, 'ETIMEDOUT', req);
+    };
     if (timeout) {
       // Listen to timeouts and throw an error.
-      req.setTimeout(timeout, () => {
-        req.abort();
-        this.rejectWithError(`timeout of ${timeout}ms exceeded`, 'ETIMEDOUT', req);
+      req.setTimeout(timeout, timeoutCallback);
+      req.on('socket', (socket) => {
+        socket.setTimeout(timeout, timeoutCallback);
       });
     }