Do not throw from connection.send in Node.js

.changeset/curvy-peaches-deliver.md

@@ -0,0 +1,5 @@
+---
+"@firebase/storage": patch
+---
+
+Do not throw from `connection.send` to enable retries in Node.js.

packages/storage/src/implementation/connection.ts

@@ -23,6 +23,10 @@ export type Headers = Record<string, string>;
  * goog.net.XhrIo-like interface.
  */
 export interface Connection {
+  /**
+   * This method should never reject. In case of encountering an error, set an error code internally which can be accessed
+   * by calling getErrorCode() by callers. 
+   */
   send(
     url: string,
     method: string,

packages/storage/src/implementation/request.ts

@@ -106,6 +106,7 @@ class NetworkRequest<T> implements Request<T> {
         connection.addUploadProgressListener(progressListener);
       }
 
+      // connection.send() never rejects, so we don't need to have a error handler or use catch on the returned promise.
       // eslint-disable-next-line @typescript-eslint/no-floating-promises
       connection
         .send(this.url_, this.method_, this.body_, this.headers_)

packages/storage/src/platform/node/connection.ts

@@ -17,7 +17,7 @@
 
 import { ErrorCode, Connection } from '../../implementation/connection';
 import { internalError } from '../../implementation/error';
-import nodeFetch from 'node-fetch';
+import nodeFetch, { FetchError } from 'node-fetch';
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const fetch: typeof window.fetch = nodeFetch as any;
@@ -58,10 +58,13 @@ export class FetchConnection implements Connection {
       .then(resp => {
         this.headers_ = resp.headers;
         this.statusCode_ = resp.status;
-        return resp.text();
-      })
-      .then(body => {
-        this.body_ = body;
+        return resp.text().then(body => {
+          this.body_ = body;
+        });
+      }, (_err: FetchError) => {
+        this.errorCode_ = ErrorCode.NETWORK_ERROR;
+        // emulate XHR which sets status to 0 when encountering a network error
+        this.statusCode_ = 0;
       });
   }