foster/denoland-deno
0
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-10-29 08:58:01 -04:00
Code Issues Wiki Activity

fix(op_crates/web/url): apply backslash replacement to the pathname setter (#7937)

Browse source
This commit is contained in:
Nayeem Rahman 2020-10-13 16:16:10 +01:00 committed by GitHub
parent bbf7b2ee72
commit d0c2714c03
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
cli/tests/unit/url_test.ts
View file

@ -192,6 +192,8 @@ unitTest(function urlModifyPathname(): void {
// deno-lint-ignore no-self-assign
url.pathname = url.pathname;
assertEquals(url.pathname, "/baz%23qat%20qux");
url.pathname = "\\a\\b\\c";
assertEquals(url.pathname, "/a/b/c");
});
unitTest(function urlModifyHash(): void {

6
op_crates/web/11_url.js
View file

@ -378,7 +378,7 @@
return null;
}
[parts.path, restUrl] = takePattern(restUrl, /^([^?#]*)/);
parts.path = encodePathname(parts.path.replace(/\\/g, "/"));
parts.path = encodePathname(parts.path);
if (usedNonBase) {
parts.path = normalizePath(parts.path, parts.protocol == "file");
} else {
@ -870,7 +870,9 @@
}
function encodePathname(s) {
return [...s].map((c) => (charInPathSet(c) ? encodeChar(c) : c)).join("");
return [...s.replace(/\\/g, "/")].map((
c,
) => (charInPathSet(c) ? encodeChar(c) : c)).join("");
}
function encodeSearch(s, isSpecial) {