feat: end early if format ends early

kevinoid · kevinoid · commit b460af3cf25e · 2020-10-27T15:46:22.000-06:00
Signal the end of data when format emits 'end' so that early 'end' (e.g. due to data after compressed data ends since nodejs/node#26363) so that InflateAuto behaves the same as the zlib streams. Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
diff --git a/index.js b/index.js
@@ -630,6 +630,13 @@ InflateAuto.prototype.setFormat = function setFormat(Format) {
   }
 
   format.on('data', (chunk) => this.push(chunk));
+  format.once('end', (chunk) => {
+    // format may emit 'end' before 'finish' (and before .end() is called)
+    // when there is data after the end of compressed input.
+    // https://github.com/nodejs/node/pull/26363
+    // Call push(null) to end this stream when the format has ended.
+    this.push(null);  // eslint-disable-line unicorn/no-null
+  });
   // Note: Readable.wrap proxies 'destroy' event.  No current use is known, but
   // we proxy it here for compatibility with non-Zlib formats.
   format.on('destroy', (...args) => this.emit('destroy', ...args));
