Browse Source

url: handle windows drive letter in the file state

`C|` should not satisfy the condition to not copy
the base's path. It also synchronises WPT url test data
to verify the update in upstream.

PR-URL: https://github.com/nodejs/node/pull/12808
Refs: https://github.com/whatwg/url/pull/305
Refs: https://github.com/w3c/web-platform-tests/pull/5754
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
v6
Daijiro Wachi 8 years ago
committed by Timothy Gu
parent
commit
943dd5f9ed
No known key found for this signature in database GPG Key ID: 7FE6B095B582B0D4
  1. 8
      src/node_url.cc
  2. 101
      test/fixtures/url-tests.js

8
src/node_url.cc

@ -1169,6 +1169,7 @@ void URL::Parse(const char* input,
while (p <= end) { while (p <= end) {
const char ch = p < end ? p[0] : kEOL; const char ch = p < end ? p[0] : kEOL;
const size_t remaining = end == p ? 0 : (end - p - 1);
if (IsASCIITabOrNewline(ch)) { if (IsASCIITabOrNewline(ch)) {
if (state == kAuthority) { if (state == kAuthority) {
@ -1653,9 +1654,10 @@ void URL::Parse(const char* input,
state = kFragment; state = kFragment;
break; break;
default: default:
if ((!IsWindowsDriveLetter(ch, p[1]) || if ((remaining == 0 ||
end - p == 1 || !IsWindowsDriveLetter(ch, p[1]) ||
(p[2] != '/' && (remaining >= 2 &&
p[2] != '/' &&
p[2] != '\\' && p[2] != '\\' &&
p[2] != '?' && p[2] != '?' &&
p[2] != '#'))) { p[2] != '#'))) {

101
test/fixtures/url-tests.js

@ -1,7 +1,7 @@
'use strict'; 'use strict';
/* WPT Refs: /* WPT Refs:
https://github.com/w3c/web-platform-tests/blob/3afae94/url/urltestdata.json https://github.com/w3c/web-platform-tests/blob/28541bb/url/urltestdata.json
License: http://www.w3.org/Consortium/Legal/2008/04-testsuite-copyright.html License: http://www.w3.org/Consortium/Legal/2008/04-testsuite-copyright.html
*/ */
module.exports = module.exports =
@ -5430,6 +5430,105 @@ module.exports =
"search": "?chai", "search": "?chai",
"hash": "" "hash": ""
}, },
"# Windows drive letter handling with the 'file:' base URL",
{
"input": "C|",
"base": "file://host/dir/file",
"href": "file:///C:",
"protocol": "file:",
"username": "",
"password": "",
"host": "",
"hostname": "",
"port": "",
"pathname": "/C:",
"search": "",
"hash": ""
},
{
"input": "C|#",
"base": "file://host/dir/file",
"href": "file:///C:#",
"protocol": "file:",
"username": "",
"password": "",
"host": "",
"hostname": "",
"port": "",
"pathname": "/C:",
"search": "",
"hash": ""
},
{
"input": "C|?",
"base": "file://host/dir/file",
"href": "file:///C:?",
"protocol": "file:",
"username": "",
"password": "",
"host": "",
"hostname": "",
"port": "",
"pathname": "/C:",
"search": "",
"hash": ""
},
{
"input": "C|/",
"base": "file://host/dir/file",
"href": "file:///C:/",
"protocol": "file:",
"username": "",
"password": "",
"host": "",
"hostname": "",
"port": "",
"pathname": "/C:/",
"search": "",
"hash": ""
},
{
"input": "C|\\",
"base": "file://host/dir/file",
"href": "file:///C:/",
"protocol": "file:",
"username": "",
"password": "",
"host": "",
"hostname": "",
"port": "",
"pathname": "/C:/",
"search": "",
"hash": ""
},
{
"input": "C",
"base": "file://host/dir/file",
"href": "file://host/dir/C",
"protocol": "file:",
"username": "",
"password": "",
"host": "host",
"hostname": "host",
"port": "",
"pathname": "/dir/C",
"search": "",
"hash": ""
},
{
"input": "C|a",
"base": "file://host/dir/file",
"href": "file://host/dir/C|a",
"protocol": "file:",
"username": "",
"password": "",
"host": "host",
"hostname": "host",
"port": "",
"pathname": "/dir/C|a",
"search": "",
"hash": ""
},
"# Windows drive letter quirk with not empty host", "# Windows drive letter quirk with not empty host",
{ {
"input": "file://example.net/C:/", "input": "file://example.net/C:/",

Loading…
Cancel
Save