Backout temporary fix allowing Thunderbird to use % in URLs from bug 1527236, rev. aa39bdeb511f - Make Thunderbird internal URL escaping standards compliant
Categories
(Core :: Networking, enhancement, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox67 | --- | affected |
People
(Reporter: jorgk-bmo, Unassigned)
References
Details
(Whiteboard: [necko-triaged])
Attachments
(1 obsolete file)
In bug 1527236 a patch was landed that allowed TB to use % in its mailnews: URLs.
Once TB has reworked that area, the patch can be backed out:
https://hg.mozilla.org/integration/mozilla-inbound/rev/aa39bdeb511f
https://hg.mozilla.org/mozilla-central/rev/aa39bdeb511f
Updated•5 years ago
|
Reporter | ||
Comment 1•5 years ago
|
||
Also see bug 1527236 comment #27 for some comments.
Updated•2 years ago
|
Comment hidden (spam) |
Updated•2 months ago
|
Comment 3•11 days ago
|
||
The "temporary" fix is problematic when pushing the check for WHATWG forbidden domain code point deeper into the dependencies (all the way to the idna
crate).
mkmelin, how feasible would it be to implement option 2 or 3 from bug 1527236 comment #27?
Comment 4•11 days ago
|
||
It should be feasible, but does require a fair bit of work.
We also do an annoying non-standards compliant re-escaping on the username. Should fix that at the same time: https://searchfox.org/comm-central/rev/0a3cadbb953e3caeda7f2d4a9b349a60d376ba25/mailnews/imap/src/nsImapUtils.cpp#65
Description
•