diff --git a/patches/node/.patches b/patches/node/.patches index 70ce8c1dd15c1..838938ed07d05 100644 --- a/patches/node/.patches +++ b/patches/node/.patches @@ -51,3 +51,4 @@ crypto_update_root_certificates_to_nss_3_47.patch tools_update_certdata_txt_to_nss_3_53.patch crypto_update_root_certificates_to_nss_3_53.patch darwin_work_around_clock_jumping_back_in_time.patch +lib_use_non-symbols_in_isurlinstance_check.patch diff --git a/patches/node/lib_use_non-symbols_in_isurlinstance_check.patch b/patches/node/lib_use_non-symbols_in_isurlinstance_check.patch new file mode 100644 index 0000000000000..73d8adbbdb943 --- /dev/null +++ b/patches/node/lib_use_non-symbols_in_isurlinstance_check.patch @@ -0,0 +1,40 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Shelley Vohr +Date: Tue, 4 Aug 2020 09:17:06 -0700 +Subject: lib: use non-symbols in isURLInstance check + +This slightly changes the conditional used to determine whether or +not something is a URL instance. Since Node.js adds symbols to the URL +not specified by the WHATWG, those symbols are not present in other +implementations (like Blink's) and therefore can cause false negatives. + +This fixes that by slightly changing the check to properties present +in all URL instances as specified in the WHATWG spec. + +Upstreamed at: https://github.com/nodejs/node/pull/34622. + +diff --git a/lib/internal/url.js b/lib/internal/url.js +index 09fa9f2cf47d50d369fa3eae378f81767486e83e..e6a37dcd22c51b37838a577be44be198649eabad 100644 +--- a/lib/internal/url.js ++++ b/lib/internal/url.js +@@ -1348,8 +1348,7 @@ function getPathFromURLPosix(url) { + function fileURLToPath(path) { + if (typeof path === 'string') + path = new URL(path); +- else if (path == null || !path[searchParams] || +- !path[searchParams][searchParams]) ++ else if (path == null || !path['origin'] || !path['href']) + throw new ERR_INVALID_ARG_TYPE('path', ['string', 'URL'], path); + if (path.protocol !== 'file:') + throw new ERR_INVALID_URL_SCHEME('file'); +@@ -1397,8 +1396,7 @@ function pathToFileURL(filepath) { + } + + function toPathIfFileURL(fileURLOrPath) { +- if (fileURLOrPath == null || !fileURLOrPath[searchParams] || +- !fileURLOrPath[searchParams][searchParams]) ++ if (fileURLOrPath == null || !fileURLOrPath['origin'] || !fileURLOrPath['href']) + return fileURLOrPath; + return fileURLToPath(fileURLOrPath); + } + \ No newline at end of file diff --git a/spec/node-spec.js b/spec/node-spec.js index ab9e180d31238..75516b449ef85 100644 --- a/spec/node-spec.js +++ b/spec/node-spec.js @@ -191,6 +191,15 @@ describe('node feature', () => { }); }); + describe('URL handling in the renderer process', () => { + it('can successfully handle WHATWG URLs constructed by Blink', () => { + const url = new URL('file://' + path.resolve(fixtures, 'pages', 'base-page.html')); + expect(() => { + fs.createReadStream(url); + }).to.not.throw(); + }); + }); + describe('error thrown in main process node context', () => { it('gets emitted as a process uncaughtException event', () => { const error = ipcRenderer.sendSync('handle-uncaught-exception', 'hello');