delta/node-new.git/src/node_url.cc, branch stabilize-modules

url: fix file url reparse

2020-10-19T06:52:45+00:00

Fixes: https://github.com/nodejs/node/issues/35571
Refs: https://github.com/whatwg/url/pull/550
Refs: https://github.com/web-platform-tests/wpt/pull/25989

PR-URL: https://github.com/nodejs/node/pull/35671
Reviewed-By: Rich Trott 
Reviewed-By: Michaël Zasso 
Reviewed-By: Franziska Hinkelmann

url: file URL path normalization

2020-10-12T06:06:55+00:00

Refs: https://github.com/whatwg/url/pull/544
Refs: https://github.com/web-platform-tests/wpt/pull/25716

PR-URL: https://github.com/nodejs/node/pull/35477
Fixes: https://github.com/nodejs/node/issues/35429
Reviewed-By: Guy Bedford 
Reviewed-By: James M Snell 
Reviewed-By: David Carlier 
Reviewed-By: Bradley Farias 
Reviewed-By: Michaël Zasso

url: verify domain is not empty after "ToASCII"

2020-09-16T06:23:41+00:00

Port of https://github.com/whatwg/url/pull/497

PR-URL: https://github.com/nodejs/node/pull/33770
Reviewed-By: Daijiro Wachi 
Reviewed-By: Shingo Inoue 
Reviewed-By: Rich Trott 
Reviewed-By: Joyee Cheung

url: remove U+0000 case in the fragment state

2020-09-16T06:23:40+00:00

Port of https://github.com/whatwg/url/pull/486

PR-URL: https://github.com/nodejs/node/pull/33770
Reviewed-By: Daijiro Wachi 
Reviewed-By: Shingo Inoue 
Reviewed-By: Rich Trott 
Reviewed-By: Joyee Cheung

src: prefer C++ empty() in boolean expressions

2020-08-11T20:22:04+00:00

PR-URL: https://github.com/nodejs/node/pull/34432
Reviewed-By: Richard Lau 
Reviewed-By: Michaël Zasso 
Reviewed-By: Jiawen Geng 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Gus Caplan 
Reviewed-By: David Carlier 
Reviewed-By: James M Snell 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Franziska Hinkelmann

src: snapshot node

2020-07-18T03:23:21+00:00

This runs `lib/internal/bootstrap/node.js` before creating
the builtin snapshot and deserialize the loaders from the
snapshot in deserialization mode.

PR-URL: https://github.com/nodejs/node/pull/32984
Reviewed-By: Anna Henningsen 
Reviewed-By: Daniel Bevenius

src,doc,test: remove String::New default parameter

2020-07-14T13:13:34+00:00

`kNormal` has been the implicit default for a while now (since V8 7.6).

Refs: https://github.com/v8/v8/commit/e0d7f816990ada28ebe1281ca9431236ef8c6e4f

PR-URL: https://github.com/nodejs/node/pull/34248
Reviewed-By: Ben Noordhuis 
Reviewed-By: Tobias Nießen 
Reviewed-By: James M Snell

url: remove gopher from special schemes

2020-06-06T09:11:02+00:00

Refs: https://github.com/nodejs/node/issues/33315
Refs: https://github.com/whatwg/url/commit/d589670451a4da9717bad69ed5d5364f93fede33
Refs: https://github.com/whatwg/url/commit/7ae1c691c96f0d82fafa24c33aa1e8df9ffbf2bc

PR-URL: https://github.com/nodejs/node/pull/33325
Reviewed-By: Colin Ihrig 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: James M Snell 
Reviewed-By: Tobias Nießen

src: avoid OOB read in URL parser

2020-05-30T17:26:24+00:00

This is not a big concern, because right now, all (non-test) inputs
to the parser are `'\0'`-terminated, but we should be future-proof
here and not perform these OOB reads.

PR-URL: https://github.com/nodejs/node/pull/33640
Reviewed-By: Sam Roberts 
Reviewed-By: Gus Caplan 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: James M Snell 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Colin Ihrig

url: forbid lt and gt in url host code point

2020-05-23T19:36:51+00:00

As per the recent changes in whatwg/url spec. lt and gt are also added
in the list of forbidden hostCodePoint list.

PR-URL: https://github.com/nodejs/node/pull/33328
Refs: https://github.com/whatwg/url/pull/459
Refs: https://github.com/nodejs/node/issues/33315
Reviewed-By: Anna Henningsen 
Reviewed-By: Yongsheng Zhang 
Reviewed-By: Ruben Bridgewater