Closed
Bug 1197123
Opened 10 years ago
Closed 8 years ago
percent encoding english alphabet characters in url are shown as english characters, but url doesn't work
Categories
(Core :: Networking, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: m.ostrowski, Assigned: CuveeHsu)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-next])
User Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:40.0) Gecko/20100101 Firefox/40.0
Build ID: 20150811082003
Steps to reproduce:
When you replace english characters with % encoding the url wont work.
For example:
replace "o" in https://www.google.com/ with "%6f" (its "o" in percent encoding)
you will get url like this: https://www.g%6f%6fgle.com/
paste it in url address, and go there
in url will be displayed https://www.google.com/, but you will get error "server not found".
Another example:
changed "k" into "%6b":
https://en.wikipedia.org/wiki/Percent-encoding
https://en.wikipedia.org/wi%6bi/Percent-encoding
wikipedia main page will appear instead of percent-encoding article.
When you change only last section of url, the wikipedia works fine:
https://en.wikipedia.org/wiki/Percent-enc%6fding
Chcecked how this works on IE, Chrome, Chrome on android, Opera and Firefox. Only Firefox have problems. Other browsers works fine.
Comment 1•10 years ago
|
||
(In reply to Michał from comment #0)
> https://www.g%6f%6fgle.com/
Bug 309671.
> https://en.wikipedia.org/wi%6bi/Percent-encoding
I'm not sure about this. Marking as blocking bug 906714 to bring it to the attention of the right people.
Updated•9 years ago
|
Whiteboard: [necko-backlog]
| Assignee | ||
Comment 2•9 years ago
|
||
I've tested this in Firefox/47 and this issue no longer exists.
I believe it's a dup of bug 412457. How do you think, Valentin?
Flags: needinfo?(valentin.gosu)
It's partially fixed.
https://www.g%6f%6fgle.com/ does work, but https://en.wikipedia.org/wi%6bi/Percent-encoding is leading to main page of wikipedia instead of https://en.wikipedia.org/wiki/Percent-encoding
| Assignee | ||
Comment 4•9 years ago
|
||
Okay, that's because Host is always un-escaped for IDN.
Need to know if we really need to unescaped English letters.
Comment 5•9 years ago
|
||
As far as I can tell, the URL spec doesn't instruct to do percent decoding in the path [1] or query, but Chrome and Edge do so.
Anne, what should we do here?
[1] https://url.spec.whatwg.org/#path-state
Flags: needinfo?(valentin.gosu) → needinfo?(annevk)
Comment 6•9 years ago
|
||
I'm not sure, Safari does the same as us (and the standard). We could change it I suppose, but we'd have to investigate exactly what Chrome and Edge implement here.
Flags: needinfo?(annevk)
| Assignee | ||
Comment 7•9 years ago
|
||
Any motivation or use case for percent encoding english alphabet in path/query...?
Flags: needinfo?(m.ostrowski)
(In reply to Junior [:junior] from comment #7)
> Any motivation or use case for percent encoding english alphabet in
> path/query...?
You ask me for a set of beer?=]
It's useful as walkaround for badly implemented censor on forums, for tricking bots etc.
I'm thinking if there is a nasty way of using this, like showing replaced page with not changed url or giving a link to a page that work only with encoded character, but with normal doesn't.
I found this while trying to fake forum censor with link like "expertsexchange.com" ("sex" in string). I edited link and replaced s with % encoded character. For some people link worked, but for others it didn't. Would be nice to act the same for everyone.
Flags: needinfo?(m.ostrowski)
Comment 9•9 years ago
|
||
I think we should actually change the behaviour, and unescape path&query too.
Opening file:///path/to/fil%65.txt actually works, because we do that just before opening the file path. It makes sense for us to do the same for regular http URLs.
Comment 10•9 years ago
|
||
It would make us different from Safari, but align with Chrome and Edge. That is probably okay and if you come up with a more detailed proposal (exactly which sequences we'd do this for) we might be able to convince Safari engineers to follow suit.
| Assignee | ||
Updated•9 years ago
|
Assignee: nobody → juhsu
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Whiteboard: [necko-backlog] → [necko-active]
| Assignee | ||
Updated•9 years ago
|
Whiteboard: [necko-active] → [necko-next]
Comment 11•8 years ago
|
||
Upon discussion in https://github.com/whatwg/url/issues/87 it seems that we don't want to do this after all.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•