url: conform to origin getter spec changes

PR-URL: #48319 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
nodejs · Jul 3, 2023 · fafe651 · fafe651
1 parent 764119b
commit fafe651
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -840,7 +840,9 @@ class URL {
       if (path.length > 0) {
         try {
           const out = new URL(path);
-          if (out.#context.scheme_type !== 1) {
+          // Only return origin of scheme is `http` or `https`
+          // Otherwise return a new opaque origin (null).
+          if (out.#context.scheme_type === 0 || out.#context.scheme_type === 2) {
             return `${out.protocol}//${out.host}`;
           }
         } catch {

diff --git a/test/wpt/status/url.json b/test/wpt/status/url.json
@@ -17,8 +17,5 @@
   },
   "javascript-urls.window.js": {
     "skip": "requires document.body reference"
-  },
-  "url-origin.any.js": {
-    "skip": "implementation not up to date"
   }
 }