lib: fix unhandled errors in webstream adapters

WebStream's Readable controller does not tolerate `.close()` being
called after an `error`. However, when wrapping a Node's Readable stream
it is possible that the sequence of events leads to `finished()`'s
callback being invoked after such `error`.

In order to handle this, in this change we call the `finished()` handler
earlier when controller is canceled, and always handle this as an error
case.

Fix: #54205

Author: indutny

lib/internal/webstreams/adapters.js

@@ -67,6 +67,7 @@ const {
   createDeferredPromise,
   kEmptyObject,
   normalizeEncoding,
+  once,
 } = require('internal/util');
 
 const {
@@ -471,7 +472,7 @@ function newReadableStreamFromStreamReadable(streamReadable, options = kEmptyObj
 
   streamReadable.pause();
 
-  const cleanup = finished(streamReadable, (error) => {
+  const onFinished = once((error) => {
     error = handleKnownInternalErrors(error);
 
     cleanup();
@@ -482,6 +483,7 @@ function newReadableStreamFromStreamReadable(streamReadable, options = kEmptyObj
       return controller.error(error);
     controller.close();
   });
+  const cleanup = finished(streamReadable, onFinished);
 
   streamReadable.on('data', onData);
 
@@ -491,7 +493,9 @@ function newReadableStreamFromStreamReadable(streamReadable, options = kEmptyObj
     pull() { streamReadable.resume(); },
 
     cancel(reason) {
-      destroy(streamReadable, reason);
+      const error = reason || new ERR_STREAM_PREMATURE_CLOSE();
+      onFinished(error);
+      destroy(streamReadable, error);
     },
   }, strategy);
 }

test/parallel/test-stream-readable-from-web-termination.js

@@ -0,0 +1,15 @@
+'use strict';
+require('../common');
+const { Readable } = require('stream');
+
+{
+  const r = Readable.from(['data']);
+
+  const wrapper = Readable.fromWeb(Readable.toWeb(r));
+
+  wrapper.on('data', () => {
+    // Destroying wrapper while emitting data should not cause uncaught
+    // exceptions
+    wrapper.destroy();
+  });
+}

test/parallel/test-stream-readable-to-web-termination.js

@@ -0,0 +1,12 @@
+'use strict';
+require('../common');
+const { Readable } = require('stream');
+
+{
+  const r = Readable.from([]);
+  // Cancelling reader while closing should not cause uncaught exceptions
+  r.on('close', () => reader.cancel());
+
+  const reader = Readable.toWeb(r).getReader();
+  reader.read();
+}