buffer: correctly apply prototype to cloned File / Blob

PR-URL: nodejs#55138 Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: James M Snell <[email protected]>
louwers · Nov 2, 2024 · 06aa440 · 06aa440
1 parent 24c4647
commit 06aa440
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/lib/internal/blob.js b/lib/internal/blob.js
@@ -386,6 +386,7 @@ class Blob {
 }
 
 function TransferableBlob(handle, length, type = '') {
+  ObjectSetPrototypeOf(this, Blob.prototype);
   markTransferMode(this, true, false);
   this[kHandle] = handle;
   this[kType] = type;

diff --git a/lib/internal/file.js b/lib/internal/file.js
@@ -130,6 +130,7 @@ class File extends Blob {
 
 function TransferableFile(handle, length, type = '') {
   FunctionPrototypeApply(TransferableBlob, this, [handle, length, type]);
+  ObjectSetPrototypeOf(this, File.prototype);
 }
 
 ObjectSetPrototypeOf(TransferableFile.prototype, File.prototype);

diff --git a/test/parallel/test-structuredClone-global.js b/test/parallel/test-structuredClone-global.js
@@ -30,6 +30,19 @@ for (const StreamClass of [ReadableStream, WritableStream, TransformStream]) {
   assert.ok(extendedTransfer instanceof StreamClass);
 }
 
+for (const Transferrable of [File, Blob]) {
+  const a2 = Transferrable === File ? '' : {};
+  const original = new Transferrable([], a2);
+  const transfer = structuredClone(original);
+  assert.strictEqual(Object.getPrototypeOf(transfer), Transferrable.prototype);
+  assert.ok(transfer instanceof Transferrable);
+
+  const extendedOriginal = new (class extends Transferrable {})([], a2);
+  const extendedTransfer = structuredClone(extendedOriginal);
+  assert.strictEqual(Object.getPrototypeOf(extendedTransfer), Transferrable.prototype);
+  assert.ok(extendedTransfer instanceof Transferrable);
+}
+
 {
   // See: https://github.com/nodejs/node/issues/49940
   const cloned = structuredClone({}, {