Merge branch 'bugfix/HD-3340' into q/1.1

scality · Oct 18, 2023 · c7742c0 · c7742c0
2 parents 3fb336d + f9b36ad
commit c7742c0
Show file tree

Hide file tree

Showing 5 changed files with 390 additions and 4 deletions.
diff --git a/lib/hdcontroller.js b/lib/hdcontroller.js
@@ -29,6 +29,7 @@ function _createRequest(req, log, callback) {
             error.code = response.statusCode;
             error.isExpected = true;
             log.debug('got expected response code:', { statusCode: response.statusCode });
+            response.resume(); // Drain the response stream
             return callback(error);
         }
         return callback(undefined, response);
@@ -267,14 +268,16 @@ class HDProxydClient {
     put(stream, size, params, reqUids, callback) {
         const log = this.createLogger(reqUids);
         this._failover('POST', stream, size, '', 0, log, (err, response) => {
+            if (response) {
+                response.resume();
+            }
             if (err || !response) {
                 return callback(err);
             }
             if (!response.headers['scal-key']) {
                 return callback(new HDProxydError('no key returned'));
             }
             const key = response.headers['scal-key'];
-            response.resume(); // drain the stream
             response.on('end', () => {
                 return callback(undefined, key);
             });