fix: don't reconnect on HEAD requests

fent · Oct 24, 2020 · 5538df8 · 5538df8
1 parent deeaa77
commit 5538df8
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/index.ts b/src/index.ts
@@ -85,7 +85,7 @@ function Miniget(url: string, options: Miniget.Options = {}): Miniget.Stream {
   };
 
   const reconnectIfEndedEarly = (err?: Error) => {
-    if (!downloadEnded() && reconnects++ < opts.maxReconnects) {
+    if (options.method != 'HEAD' && !downloadEnded() && reconnects++ < opts.maxReconnects) {
       reconnect(err);
       return true;
     }

diff --git a/test/request-test.ts b/test/request-test.ts
@@ -753,4 +753,19 @@ describe('Make a request', () => {
       });
     });
   });
+
+  describe('with `method = "HEAD"`', () => {
+    it('Emits `response`', (done) => {
+      const scope = nock('http://hello.net')
+        .head('/world')
+        .reply(200, '', { 'content-length': '10' });
+      const req = miniget('http://hello.net/world', { method: 'HEAD' });
+      req.on('error', done);
+      req.on('response', res => {
+        scope.done();
+        assert.equal(res.headers['content-length'], '10');
+        done();
+      });
+    });
+  });
 });