url: conform structuredClone url serialization

nodejs · Mar 22, 2023 · 137d2de · 137d2de
1 parent 1a18b44
commit 137d2de
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 3 deletions.
diff --git a/lib/internal/structured_clone.js b/lib/internal/structured_clone.js
@@ -4,17 +4,36 @@ const {
   codes: { ERR_MISSING_ARGS },
 } = require('internal/errors');
 
+const {
+  lazyDOMException,
+} = require('internal/util');
+
 const {
   MessageChannel,
   receiveMessageOnPort,
 } = require('internal/worker/io');
 
+const {
+  isURL,
+  isURLSearchParams,
+} = require('internal/url');
+
 let channel;
 function structuredClone(value, options = undefined) {
   if (arguments.length === 0) {
     throw new ERR_MISSING_ARGS('value');
   }
 
+  if (isURL(value)) {
+    throw new lazyDOMException(
+      'URL: no structured serialize/deserialize support',
+      'DataCloneError');
+  } else if (isURLSearchParams(value)) {
+    throw new lazyDOMException(
+      'URLSearchParams: no structured serialize/deserialize support',
+      'DataCloneError');
+  }
+
   // TODO: Improve this with a more efficient solution that avoids
   // instantiating a MessageChannel
   channel ??= new MessageChannel();

diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -1279,4 +1279,5 @@ module.exports = {
   urlToHttpOptions,
   encodeStr,
   isURL,
+  isURLSearchParams,
 };
diff --git a/test/wpt/status/url.json b/test/wpt/status/url.json
@@ -11,9 +11,7 @@
     "fail": {
       "note": "We are faking location with a URL object for the sake of the testharness and it has searchParams.",
       "expected": [
-        "searchParams on location object",
-        "URL: no structured serialize/deserialize support",
-        "URLSearchParams: no structured serialize/deserialize support"
+        "searchParams on location object"
       ]
     }
   },