226136 2021-05-21 22:10:08 -0700 URL path setter messes up UTF-8 encoding after ? 2021-05-26 10:35:18 -0700 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 timothygu99 achristensen achristensen benjamin cdumez cmarcelo ews-watchlist webkit-bug-importer oldest_to_newest 1762763 0 timothygu99 2021-05-21 22:10:08 -0700 With the following code: u = new URL('http://abc/'); u.pathname = "/?é"; console.log(u.pathname); The string "/%3F%C3%83%C2%A9" is printed. But this is wrong, since é in UTF-8 is 0xC3 0xA9. If we remove the ?, everything works fine: u = new URL('http://abc/'); u.pathname = "/é"; console.log(u.pathname); // prints "/%C3%A9" 1763160 1 429539 achristensen 2021-05-24 09:30:05 -0700 Created attachment 429539 Patch 1763161 2 achristensen 2021-05-24 09:30:29 -0700 Thanks for pointing this out, Timothy! 1763951 3 429539 cdumez 2021-05-26 10:07:32 -0700 Comment on attachment 429539 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=429539&action=review r=me > Source/WTF/ChangeLog:10 > + This matches Chrome, Firefox, and reason. :) 1763968 4 ews-feeder 2021-05-26 10:34:10 -0700 Committed r278091 (238166@main): <https://commits.webkit.org/238166@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 429539. 1763970 5 webkit-bug-importer 2021-05-26 10:35:18 -0700 <rdar://problem/78520238> 429539 2021-05-24 09:30:05 -0700 2021-05-26 10:34:11 -0700 Patch bug-226136-20210524093004.patch text/plain 5693 achristensen SW5kZXg6IFNvdXJjZS9XVEYvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XVEYvQ2hh bmdlTG9nCShyZXZpc2lvbiAyNzc4MzcpCisrKyBTb3VyY2UvV1RGL0NoYW5nZUxvZwkod29ya2lu ZyBjb3B5KQpAQCAtMSwzICsxLDE5IEBACisyMDIxLTA1LTI0ICBBbGV4IENocmlzdGVuc2VuICA8 YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgVVJMIHBhdGggc2V0dGVyIG1lc3Nl cyB1cCBVVEYtOCBlbmNvZGluZyBhZnRlciA/CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQu b3JnL3Nob3dfYnVnLmNnaT9pZD0yMjYxMzYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg KE9PUFMhKS4KKworICAgICAgICBlc2NhcGVQYXRoV2l0aG91dENvcHlpbmcgbmVlZHMgdG8gdGVs bCBwZXJjZW50RW5jb2RlQ2hhcmFjdGVycyB0byBwZXJjZW50IGVuY29kZSBhbGwgbm9uLUFTQ0lJ IGNoYXJhY3RlcnMgaW4gdGhlIG91dHB1dCBvZiBVVEYtOCBlbmNvZGluZworICAgICAgICBpbiBh ZGRpdGlvbiB0byA/IGFuZCAjLiAgT3RoZXJ3aXNlLCB3ZSB0YWtlIHRoZSBvdXRwdXQgb2YgVVRG LTggZW5jb2RpbmcgYW5kIGFwcGVuZCBlYWNoIGNvZGUgdW5pdCB0byB0aGUgaW5wdXQgb2YgVVJM IHBhcnNpbmcgYXMgaWYgaXQgd2VyZSBMYXRpbi0xLgorICAgICAgICBUaGlzIG1hdGNoZXMgQ2hy b21lLCBGaXJlZm94LCBhbmQgcmVhc29uLgorICAgICAgICBDb3ZlcmVkIGJ5IGEgbmV3bHkgcGFz c2luZyBXUFQgdGVzdC4KKyAgICAgICAgT3RoZXIgY2FsbGVycyBvZiBwZXJjZW50RW5jb2RlQ2hh cmFjdGVycyB1c2UgZnVuY3Rpb25zIHRoYXQgYWxyZWFkeSBlbmNvZGUgYWxsIG5vbi1BU0NJSSBj aGFyYWN0ZXJzLgorCisgICAgICAgICogd3RmL1VSTC5jcHA6CisgICAgICAgIChXVEY6OmVzY2Fw ZVBhdGhXaXRob3V0Q29weWluZyk6CisKIDIwMjEtMDUtMTkgIEFsZXggQ2hyaXN0ZW5zZW4gIDxh Y2hyaXN0ZW5zZW5Ad2Via2l0Lm9yZz4KIAogICAgICAgICBBZGQgc3VwcG9ydCBmb3IgTmF2aWdh dGlvbiBUaW1pbmcgTGV2ZWwgMgpJbmRleDogU291cmNlL1dURi93dGYvVVJMLmNwcAo9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBTb3VyY2UvV1RGL3d0Zi9VUkwuY3BwCShyZXZpc2lvbiAyNzc4MzcpCisrKyBTb3Vy Y2UvV1RGL3d0Zi9VUkwuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC02NjUsMTAgKzY2NSwxMCBAQCB2 b2lkIFVSTDo6c2V0UXVlcnkoU3RyaW5nVmlldyBuZXdRdWVyeSkKIAogc3RhdGljIFN0cmluZyBl c2NhcGVQYXRoV2l0aG91dENvcHlpbmcoU3RyaW5nVmlldyBwYXRoKQogewotICAgIGF1dG8gcXVl c3Rpb25NYXJrT3JOdW1iZXJTaWduID0gW10gKFVDaGFyIGNoYXJhY3RlcikgewotICAgICAgICBy ZXR1cm4gY2hhcmFjdGVyID09ICc/JyB8fCBjaGFyYWN0ZXIgPT0gJyMnOworICAgIGF1dG8gcXVl c3Rpb25NYXJrT3JOdW1iZXJTaWduT3JOb25BU0NJSSA9IFtdIChVQ2hhciBjaGFyYWN0ZXIpIHsK KyAgICAgICAgcmV0dXJuIGNoYXJhY3RlciA9PSAnPycgfHwgY2hhcmFjdGVyID09ICcjJyB8fCAh aXNBU0NJSShjaGFyYWN0ZXIpOwogICAgIH07Ci0gICAgcmV0dXJuIHBlcmNlbnRFbmNvZGVDaGFy YWN0ZXJzKHBhdGgudG9TdHJpbmdXaXRob3V0Q29weWluZygpLCBxdWVzdGlvbk1hcmtPck51bWJl clNpZ24pOworICAgIHJldHVybiBwZXJjZW50RW5jb2RlQ2hhcmFjdGVycyhwYXRoLnRvU3RyaW5n V2l0aG91dENvcHlpbmcoKSwgcXVlc3Rpb25NYXJrT3JOdW1iZXJTaWduT3JOb25BU0NJSSk7CiB9 CiAKIHZvaWQgVVJMOjpzZXRQYXRoKFN0cmluZ1ZpZXcgcGF0aCkKSW5kZXg6IExheW91dFRlc3Rz L2ltcG9ydGVkL3czYy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvaW1wb3J0 ZWQvdzNjL0NoYW5nZUxvZwkocmV2aXNpb24gMjc3OTU0KQorKysgTGF5b3V0VGVzdHMvaW1wb3J0 ZWQvdzNjL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDEyIEBACisyMDIxLTA1 LTI0ICBBbGV4IENocmlzdGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAg ICAgVVJMIHBhdGggc2V0dGVyIG1lc3NlcyB1cCBVVEYtOCBlbmNvZGluZyBhZnRlciA/CisgICAg ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMjYxMzYKKworICAg ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIHdlYi1wbGF0Zm9y bS10ZXN0cy91cmwvdXJsLXNldHRlcnMtZXhwZWN0ZWQudHh0OgorCiAyMDIxLTA1LTI0ICBJbWFu b2wgRmVybmFuZGV6ICA8aWZlcm5hbmRlekBpZ2FsaWEuY29tPgogCiAgICAgICAgIFVwZGF0ZSBX ZWJYUiBXUFQgdGVzdHMKSW5kZXg6IExheW91dFRlc3RzL2ltcG9ydGVkL3czYy93ZWItcGxhdGZv cm0tdGVzdHMvdXJsL3VybC1zZXR0ZXJzLWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlv dXRUZXN0cy9pbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL3VybC91cmwtc2V0dGVycy1l eHBlY3RlZC50eHQJKHJldmlzaW9uIDI3NzgzNykKKysrIExheW91dFRlc3RzL2ltcG9ydGVkL3cz Yy93ZWItcGxhdGZvcm0tdGVzdHMvdXJsL3VybC1zZXR0ZXJzLWV4cGVjdGVkLnR4dAkod29ya2lu ZyBjb3B5KQpAQCAtNTEzLDEyICs1MTMsMTIgQEAgUEFTUyA8YXJlYT46IFNldHRpbmcgPGh0dHA6 Ly9leGFtcGxlLm5ldAogUEFTUyBVUkw6IFNldHRpbmcgPHZpZXctc291cmNlK2h0dHA6Ly9leGFt cGxlLm5ldC9ob21lP2xhbmc9ZnIjbmF2Pi5wYXRobmFtZSA9ICdcYVwlMkVcYlwlMmUuXGMnIFwg aXMgKm5vdCogYSBzZWdtZW50IGRlbGltaXRlciBmb3Igbm9uLSdzcGVjaWFsJyBVUkxzCiBQQVNT IDxhPjogU2V0dGluZyA8dmlldy1zb3VyY2UraHR0cDovL2V4YW1wbGUubmV0L2hvbWU/bGFuZz1m ciNuYXY+LnBhdGhuYW1lID0gJ1xhXCUyRVxiXCUyZS5cYycgXCBpcyAqbm90KiBhIHNlZ21lbnQg ZGVsaW1pdGVyIGZvciBub24tJ3NwZWNpYWwnIFVSTHMKIFBBU1MgPGFyZWE+OiBTZXR0aW5nIDx2 aWV3LXNvdXJjZStodHRwOi8vZXhhbXBsZS5uZXQvaG9tZT9sYW5nPWZyI25hdj4ucGF0aG5hbWUg PSAnXGFcJTJFXGJcJTJlLlxjJyBcIGlzICpub3QqIGEgc2VnbWVudCBkZWxpbWl0ZXIgZm9yIG5v bi0nc3BlY2lhbCcgVVJMcwotRkFJTCBVUkw6IFNldHRpbmcgPGE6Lz4ucGF0aG5hbWUgPSAnXDAB CQotXHIfICEiIyQlJicoKSorLC0uLzA5Ojs8PT4/QEFaW1xdXl9gYXp7fH1+f8KAwoHDicOpJyBV VEYtOCBwZXJjZW50IGVuY29kaW5nIHdpdGggdGhlIGRlZmF1bHQgZW5jb2RlIHNldC4gVGFicyBh bmQgbmV3bGluZXMgYXJlIHJlbW92ZWQuIGFzc2VydF9lcXVhbHM6IGV4cGVjdGVkICJhOi8lMDAl MDElMUYlMjAhJTIyJTIzJCUmJygpKissLS4vMDk6OyUzQz0lM0UlM0ZAQVpbXFxdXl8lNjBheiU3 QnwlN0R+JTdGJUMyJTgwJUMyJTgxJUMzJTg5JUMzJUE5IiBidXQgZ290ICJhOi8lMDAlMDElMUYl MjAhJTIyJTIzJCUmJygpKissLS4vMDk6OyUzQz0lM0UlM0ZAQVpbXFxdXl8lNjBheiU3QnwlN0R+ JTdGJUMzJTgyJUMyJTgwJUMzJTgyJUMyJTgxJUMzJTgzJUMyJTg5JUMzJTgzJUMyJUE5IgotRkFJ TCA8YT46IFNldHRpbmcgPGE6Lz4ucGF0aG5hbWUgPSAnXDABCQotXHIfICEiIyQlJicoKSorLC0u LzA5Ojs8PT4/QEFaW1xdXl9gYXp7fH1+f8KAwoHDicOpJyBVVEYtOCBwZXJjZW50IGVuY29kaW5n IHdpdGggdGhlIGRlZmF1bHQgZW5jb2RlIHNldC4gVGFicyBhbmQgbmV3bGluZXMgYXJlIHJlbW92 ZWQuIGFzc2VydF9lcXVhbHM6IGV4cGVjdGVkICJhOi8lMDAlMDElMUYlMjAhJTIyJTIzJCUmJygp KissLS4vMDk6OyUzQz0lM0UlM0ZAQVpbXFxdXl8lNjBheiU3QnwlN0R+JTdGJUMyJTgwJUMyJTgx JUMzJTg5JUMzJUE5IiBidXQgZ290ICJhOi8lMDAlMDElMUYlMjAhJTIyJTIzJCUmJygpKissLS4v MDk6OyUzQz0lM0UlM0ZAQVpbXFxdXl8lNjBheiU3QnwlN0R+JTdGJUMzJTgyJUMyJTgwJUMzJTgy JUMyJTgxJUMzJTgzJUMyJTg5JUMzJTgzJUMyJUE5IgotRkFJTCA8YXJlYT46IFNldHRpbmcgPGE6 Lz4ucGF0aG5hbWUgPSAnXDABCQotXHIfICEiIyQlJicoKSorLC0uLzA5Ojs8PT4/QEFaW1xdXl9g YXp7fH1+f8KAwoHDicOpJyBVVEYtOCBwZXJjZW50IGVuY29kaW5nIHdpdGggdGhlIGRlZmF1bHQg ZW5jb2RlIHNldC4gVGFicyBhbmQgbmV3bGluZXMgYXJlIHJlbW92ZWQuIGFzc2VydF9lcXVhbHM6 IGV4cGVjdGVkICJhOi8lMDAlMDElMUYlMjAhJTIyJTIzJCUmJygpKissLS4vMDk6OyUzQz0lM0Ul M0ZAQVpbXFxdXl8lNjBheiU3QnwlN0R+JTdGJUMyJTgwJUMyJTgxJUMzJTg5JUMzJUE5IiBidXQg Z290ICJhOi8lMDAlMDElMUYlMjAhJTIyJTIzJCUmJygpKissLS4vMDk6OyUzQz0lM0UlM0ZAQVpb XFxdXl8lNjBheiU3QnwlN0R+JTdGJUMzJTgyJUMyJTgwJUMzJTgyJUMyJTgxJUMzJTgzJUMyJTg5 JUMzJTgzJUMyJUE5IgorUEFTUyBVUkw6IFNldHRpbmcgPGE6Lz4ucGF0aG5hbWUgPSAnXDABCQor XHIfICEiIyQlJicoKSorLC0uLzA5Ojs8PT4/QEFaW1xdXl9gYXp7fH1+f8KAwoHDicOpJyBVVEYt OCBwZXJjZW50IGVuY29kaW5nIHdpdGggdGhlIGRlZmF1bHQgZW5jb2RlIHNldC4gVGFicyBhbmQg bmV3bGluZXMgYXJlIHJlbW92ZWQuCitQQVNTIDxhPjogU2V0dGluZyA8YTovPi5wYXRobmFtZSA9 ICdcMAEJCitcch8gISIjJCUmJygpKissLS4vMDk6Ozw9Pj9AQVpbXF1eX2Bhent8fX5/woDCgcOJ w6knIFVURi04IHBlcmNlbnQgZW5jb2Rpbmcgd2l0aCB0aGUgZGVmYXVsdCBlbmNvZGUgc2V0LiBU YWJzIGFuZCBuZXdsaW5lcyBhcmUgcmVtb3ZlZC4KK1BBU1MgPGFyZWE+OiBTZXR0aW5nIDxhOi8+ LnBhdGhuYW1lID0gJ1wwAQkKK1xyHyAhIiMkJSYnKCkqKywtLi8wOTo7PD0+P0BBWltcXV5fYGF6 e3x9fn/CgMKBw4nDqScgVVRGLTggcGVyY2VudCBlbmNvZGluZyB3aXRoIHRoZSBkZWZhdWx0IGVu Y29kZSBzZXQuIFRhYnMgYW5kIG5ld2xpbmVzIGFyZSByZW1vdmVkLgogUEFTUyBVUkw6IFNldHRp bmcgPGh0dHA6Ly9leGFtcGxlLm5ldD4ucGF0aG5hbWUgPSAnJTJlJTJFJWMzJTg5dMOpJyBCeXRl cyBhbHJlYWR5IHBlcmNlbnQtZW5jb2RlZCBhcmUgbGVmdCBhcy1pcywgaW5jbHVkaW5nICUyRSBv dXRzaWRlIGRvdHRlZCBzZWdtZW50cy4KIFBBU1MgPGE+OiBTZXR0aW5nIDxodHRwOi8vZXhhbXBs ZS5uZXQ+LnBhdGhuYW1lID0gJyUyZSUyRSVjMyU4OXTDqScgQnl0ZXMgYWxyZWFkeSBwZXJjZW50 LWVuY29kZWQgYXJlIGxlZnQgYXMtaXMsIGluY2x1ZGluZyAlMkUgb3V0c2lkZSBkb3R0ZWQgc2Vn bWVudHMuCiBQQVNTIDxhcmVhPjogU2V0dGluZyA8aHR0cDovL2V4YW1wbGUubmV0Pi5wYXRobmFt ZSA9ICclMmUlMkUlYzMlODl0w6knIEJ5dGVzIGFscmVhZHkgcGVyY2VudC1lbmNvZGVkIGFyZSBs ZWZ0IGFzLWlzLCBpbmNsdWRpbmcgJTJFIG91dHNpZGUgZG90dGVkIHNlZ21lbnRzLgo=