From ef62e5a59e67f07fa1f5c94790e1cb45df997797 Mon Sep 17 00:00:00 2001
From: Khafra <maitken033380023@gmail.com>
Date: Wed, 22 Mar 2023 15:44:44 -0400
Subject: [PATCH] url: implement URL.canParse

PR-URL: https://github.com/nodejs/node/pull/47179
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
---
 benchmark/url/whatwgurl-canParse.js           | 14 +++++++
 doc/api/url.md                                | 21 ++++++++++
 lib/internal/url.js                           | 17 ++++++++
 src/node_url.cc                               | 26 ++++++++++++
 test/fixtures/wpt/README.md                   |  2 +-
 .../wpt/url/url-statics-canparse.any.js       | 42 +++++++++++++++++++
 test/fixtures/wpt/versions.json               |  2 +-
 7 files changed, 122 insertions(+), 2 deletions(-)
 create mode 100644 benchmark/url/whatwgurl-canParse.js
 create mode 100644 test/fixtures/wpt/url/url-statics-canparse.any.js

diff --git a/benchmark/url/whatwgurl-canParse.js b/benchmark/url/whatwgurl-canParse.js
new file mode 100644
index 00000000000000..65741d9884f106
--- /dev/null
+++ b/benchmark/url/whatwgurl-canParse.js
@@ -0,0 +1,14 @@
+'use strict';
+const common = require('../common.js');
+
+const bench = common.createBenchmark(main, {
+  type: Object.keys(common.urls),
+  n: [25e6],
+});
+
+function main({ type, n }) {
+  bench.start();
+  for (let i = 0; i < n; i += 1)
+    URL.canParse(common.urls[type]);
+  bench.end(n);
+}
diff --git a/doc/api/url.md b/doc/api/url.md
index ab8b4154ff023e..de4a4cb85bce37 100644
--- a/doc/api/url.md
+++ b/doc/api/url.md
@@ -662,6 +662,27 @@ added: v16.7.0
 Removes the stored {Blob} identified by the given ID. Attempting to revoke a
 ID that isn't registered will silently fail.
 
+#### `URL.canParse(input[, base])`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+* `input` {string} The absolute or relative input URL to parse. If `input`
+  is relative, then `base` is required. If `input` is absolute, the `base`
+  is ignored. If `input` is not a string, it is [converted to a string][] first.
+* `base` {string} The base URL to resolve against if the `input` is not
+  absolute. If `base` is not a string, it is [converted to a string][] first.
+* Returns: {boolean}
+
+Checks if an `input` relative to the `base` can be parsed to a `URL`.
+
+```js
+const isValid = URL.canParse('/foo', 'https://example.org/'); // true
+
+const isNotValid = URL.canParse('/foo'); // false
+```
+
 ### Class: `URLSearchParams`
 
 <!-- YAML
diff --git a/lib/internal/url.js b/lib/internal/url.js
index 89a7ea4c31b2dd..147ab7f0164400 100644
--- a/lib/internal/url.js
+++ b/lib/internal/url.js
@@ -88,6 +88,7 @@ const {
   domainToASCII: _domainToASCII,
   domainToUnicode: _domainToUnicode,
   parse,
+  canParse: _canParse,
   updateUrl,
 } = internalBinding('url');
 
@@ -789,6 +790,16 @@ class URL {
       // If there's an error, it's ignored.
     }
   }
+
+  static canParse(url, base = undefined) {
+    url = `${url}`;
+
+    if (base !== undefined) {
+      base = `${base}`;
+    }
+
+    return _canParse(url, base);
+  }
 }
 
 ObjectDefineProperties(URL.prototype, {
@@ -812,6 +823,12 @@ ObjectDefineProperties(URL.prototype, {
 ObjectDefineProperties(URL, {
   createObjectURL: kEnumerableProperty,
   revokeObjectURL: kEnumerableProperty,
+  canParse: {
+    __proto__: null,
+    configurable: true,
+    writable: true,
+    enumerable: true,
+  },
 });
 
 // application/x-www-form-urlencoded parser
diff --git a/src/node_url.cc b/src/node_url.cc
index 27e8448517a3a4..4089552fe03643 100644
--- a/src/node_url.cc
+++ b/src/node_url.cc
@@ -93,6 +93,30 @@ void Parse(const FunctionCallbackInfo<Value>& args) {
   args.GetReturnValue().Set(true);
 }
 
+void CanParse(const FunctionCallbackInfo<Value>& args) {
+  CHECK_GE(args.Length(), 2);
+  CHECK(args[0]->IsString());  // input
+  // args[1] // base url
+
+  Environment* env = Environment::GetCurrent(args);
+  HandleScope handle_scope(env->isolate());
+  Context::Scope context_scope(env->context());
+
+  Utf8Value input(env->isolate(), args[0]);
+  ada::result base;
+  ada::url* base_pointer = nullptr;
+  if (args[1]->IsString()) {
+    base = ada::parse(Utf8Value(env->isolate(), args[1]).ToString());
+    if (!base) {
+      return args.GetReturnValue().Set(false);
+    }
+    base_pointer = &base.value();
+  }
+  ada::result out = ada::parse(input.ToStringView(), base_pointer);
+
+  args.GetReturnValue().Set(out.has_value());
+}
+
 void DomainToASCII(const FunctionCallbackInfo<Value>& args) {
   Environment* env = Environment::GetCurrent(args);
   CHECK_GE(args.Length(), 1);
@@ -285,6 +309,7 @@ void Initialize(Local<Object> target,
                 void* priv) {
   SetMethod(context, target, "parse", Parse);
   SetMethod(context, target, "updateUrl", UpdateUrl);
+  SetMethodNoSideEffect(context, target, "canParse", CanParse);
   SetMethodNoSideEffect(context, target, "formatUrl", FormatUrl);
 
   SetMethodNoSideEffect(context, target, "domainToASCII", DomainToASCII);
@@ -294,6 +319,7 @@ void Initialize(Local<Object> target,
 
 void RegisterExternalReferences(ExternalReferenceRegistry* registry) {
   registry->Register(Parse);
+  registry->Register(CanParse);
   registry->Register(UpdateUrl);
   registry->Register(FormatUrl);
 
diff --git a/test/fixtures/wpt/README.md b/test/fixtures/wpt/README.md
index d767e533ec3e67..950f0cd0a966f3 100644
--- a/test/fixtures/wpt/README.md
+++ b/test/fixtures/wpt/README.md
@@ -27,7 +27,7 @@ Last update:
 - resource-timing: https://github.com/web-platform-tests/wpt/tree/22d38586d0/resource-timing
 - resources: https://github.com/web-platform-tests/wpt/tree/919874f84f/resources
 - streams: https://github.com/web-platform-tests/wpt/tree/51750bc8d7/streams
-- url: https://github.com/web-platform-tests/wpt/tree/84caeb6fbd/url
+- url: https://github.com/web-platform-tests/wpt/tree/7c5c3cc125/url
 - user-timing: https://github.com/web-platform-tests/wpt/tree/df24fb604e/user-timing
 - wasm/jsapi: https://github.com/web-platform-tests/wpt/tree/d8dbe6990b/wasm/jsapi
 - wasm/webapi: https://github.com/web-platform-tests/wpt/tree/fd1b23eeaa/wasm/webapi
diff --git a/test/fixtures/wpt/url/url-statics-canparse.any.js b/test/fixtures/wpt/url/url-statics-canparse.any.js
new file mode 100644
index 00000000000000..c87fcb4f56fcdd
--- /dev/null
+++ b/test/fixtures/wpt/url/url-statics-canparse.any.js
@@ -0,0 +1,42 @@
+// This intentionally does not use resources/urltestdata.json to preserve resources.
+[
+  {
+    "url": undefined,
+    "base": undefined,
+    "expected": false
+  },
+  {
+    "url": "a:b",
+    "base": undefined,
+    "expected": true
+  },
+  {
+    "url": undefined,
+    "base": "a:b",
+    "expected": false
+  },
+  {
+    "url": "a:/b",
+    "base": undefined,
+    "expected": true
+  },
+  {
+    "url": undefined,
+    "base": "a:/b",
+    "expected": true
+  },
+  {
+    "url": "https://test:test",
+    "base": undefined,
+    "expected": false
+  },
+  {
+    "url": "a",
+    "base": "https://b/",
+    "expected": true
+  }
+].forEach(({ url, base, expected }) => {
+  test(() => {
+    assert_equals(URL.canParse(url, base), expected);
+  }, `URL.canParse(${url}, ${base})`);
+});
diff --git a/test/fixtures/wpt/versions.json b/test/fixtures/wpt/versions.json
index 03f6b716210f4b..583e1f75bf585f 100644
--- a/test/fixtures/wpt/versions.json
+++ b/test/fixtures/wpt/versions.json
@@ -68,7 +68,7 @@
     "path": "streams"
   },
   "url": {
-    "commit": "84caeb6fbdf45129f57c67448e6113ee1ced9fb3",
+    "commit": "7c5c3cc125979b4768d414471e6ab655b473aae8",
     "path": "url"
   },
   "user-timing": {