1766511e34
* chore: bump node in DEPS to v18.18.0 * child_process: harden against prototype pollution https://github.com/nodejs/node/pull/48726 * deps: upgrade to libuv 1.46.0 https://github.com/nodejs/node/pull/49591 * module: reduce url invocations in esm/load.js https://github.com/nodejs/node/pull/48337 * Revert "test: remove test-crypto-keygen flaky designation" https://github.com/nodejs/node/pull/48652 * fix: FTBTFS in ada dep https://github.com/ada-url/ada/pull/464 https://github.com/ada-url/idna/pull/31 * fix: force_colors snapshot line number * chore: fixup patch indices * chore: update filenames.json --------- Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com> Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
71 lines
2.4 KiB
Diff
71 lines
2.4 KiB
Diff
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
From: Charles Kerr <charles@charleskerr.com>
|
|
Date: Thu, 20 Jul 2023 14:18:19 -0500
|
|
Subject: fix: isURL() implementation
|
|
|
|
Modify Node's lib/internal/url.js isURL() to return the correct value for
|
|
URLs created both inside and outside of Node.
|
|
|
|
The isURL() impl in `main` is Electron-safe but can't be backported in
|
|
isolation because it relies on other changes from 18 to main. But we can
|
|
safely get there by trying the 18 version first (catching Node URLs),
|
|
then the `main` version (catching Electron URLs).
|
|
|
|
More background w/upstream links at
|
|
https://github.com/electron/electron/pull/39154#issuecomment-1644433388
|
|
|
|
This patch can be removed when we update to Node 20.
|
|
|
|
diff --git a/lib/internal/url.js b/lib/internal/url.js
|
|
index b66a5bbde3e8c419d5385339805cbd94f630986d..228122a38776fe5436f2601efa16b7aa2b440468 100644
|
|
--- a/lib/internal/url.js
|
|
+++ b/lib/internal/url.js
|
|
@@ -618,7 +618,12 @@ ObjectDefineProperties(URLSearchParams.prototype, {
|
|
* @returns {self is URL}
|
|
*/
|
|
function isURL(self) {
|
|
- return self != null && ObjectPrototypeHasOwnProperty(self, context);
|
|
+ // if it has `context` it is a Node.js URL...
|
|
+ if (self != null && ObjectPrototypeHasOwnProperty(self, context))
|
|
+ return true;
|
|
+
|
|
+ // ...but also honor 3rd party URLs e.g. from Electron.
|
|
+ return Boolean(self?.href && self.protocol && self.auth === undefined && self.path === undefined);
|
|
}
|
|
|
|
class URL {
|
|
@@ -715,14 +720,10 @@ class URL {
|
|
}
|
|
|
|
get href() {
|
|
- if (!isURL(this))
|
|
- throw new ERR_INVALID_THIS('URL');
|
|
return this[context].href;
|
|
}
|
|
|
|
set href(value) {
|
|
- if (!isURL(this))
|
|
- throw new ERR_INVALID_THIS('URL');
|
|
value = `${value}`;
|
|
const href = bindingUrl.update(this[context].href, updateActions.kHref, value);
|
|
if (!href) { throw ERR_INVALID_URL(value); }
|
|
diff --git a/test/parallel/test-whatwg-url-invalidthis.js b/test/parallel/test-whatwg-url-invalidthis.js
|
|
index 790c28e37c13ed6763cb61b549ab4a983f384717..bddf48b8302632a275d996a53b09343938dc7dc9 100644
|
|
--- a/test/parallel/test-whatwg-url-invalidthis.js
|
|
+++ b/test/parallel/test-whatwg-url-invalidthis.js
|
|
@@ -15,7 +15,6 @@ const assert = require('assert');
|
|
});
|
|
|
|
[
|
|
- 'href',
|
|
'protocol',
|
|
'username',
|
|
'password',
|
|
@@ -36,7 +35,6 @@ const assert = require('assert');
|
|
});
|
|
|
|
[
|
|
- 'origin',
|
|
'searchParams',
|
|
].forEach((i) => {
|
|
assert.throws(() => Reflect.get(URL.prototype, i, {}), {
|