Closed
Bug 140472
Opened 23 years ago
Closed 23 years ago
DNS: No escape for non ASCII host name
Categories
(Core :: Networking, defect, P3)
Core
Networking
Tracking
()
RESOLVED
WONTFIX
Future
People
(Reporter: nhottanscp, Assigned: nhottanscp)
References
Details
(Keywords: intl, qawanted, Whiteboard: [adt3])
Attachments
(1 file)
561 bytes,
patch
|
dougt
:
review+
rpotts
:
superreview+
|
Details | Diff | Splinter Review |
Mozilla escapes entire host name if it contains non ASCII.
e.g.
www.á.com -> "%77%77%77%2E%E3%81%82%2E%63%6F%6D"
Encoding host name may be done at client resolver or by the server. That would
not work if Mozilla escapes the host name.
E.g. mdnkit by JPNIC
http://www.nic.ad.jp/ja/idn/mdnkit/download/documents/mdnkit-2.4-doc/en/spec/spec.html
I think this is important because Mozilla does not provide IDN encoding as
default and the user may need to depend on other solutions to enable IDN.
Comment 1•23 years ago
|
||
CC'ing edmon.
I want to nominate this for nsbeta1. We would like to
claim support for multilingal domain names for the next
client release. If we escape the domain name part of it,
our support only has an incomlete story.
Keywords: nsbeta1
Comment 2•23 years ago
|
||
sounds like a good plan to me.
Comment 3•23 years ago
|
||
no time for 1.0
Whiteboard: [RTM]
Target Milestone: mozilla1.0 → mozilla1.0.1
Updated•23 years ago
|
Target Milestone: mozilla1.0.1 → ---
Updated•23 years ago
|
Target Milestone: --- → mozilla1.0.1
Updated•23 years ago
|
Whiteboard: [RTM] → [adt2 RTM]
Comment 4•23 years ago
|
||
this patch doesn't solve the bug exactly... it just fixes the escaping so that
only non-ASCII characters are escaped. escaping all characters was never
intended.
Comment 5•23 years ago
|
||
Comment on attachment 84876 [details] [diff] [review]
patch
r=dougt
Attachment #84876 -
Flags: review+
Comment 6•23 years ago
|
||
Attachment #84876 -
Flags: superreview+
Comment 7•23 years ago
|
||
landed attachment 84876 [details] [diff] [review] on the trunk.
Comment 8•23 years ago
|
||
nhotta: i'm on vacation now... if this still needs to be fixed by RTM then it'll
need a new owner.
-> 1.2alpha
Target Milestone: mozilla1.0.1 → mozilla1.2alpha
Assignee | ||
Comment 9•23 years ago
|
||
Reassign to nhotta and i18n group will evaluate this for rtm.
Comment 10•23 years ago
|
||
I think the idns is still not mature enough and it is too risky to change in
this level of code right now. Let's nsbeta1- it and push for trunk .
Assignee | ||
Comment 11•23 years ago
|
||
Return this to darin.
Can we unescape non ASCII just before we send the host name to the server?
Assignee: nhotta → darin
Summary: No escape for non ASCII host name → DNS: No escape for non ASCII host name
Assignee | ||
Comment 12•23 years ago
|
||
*** Bug 161432 has been marked as a duplicate of this bug. ***
Comment 14•23 years ago
|
||
If this is IDN, can I get the IDN QA to take this bug from me?
Comment 15•23 years ago
|
||
For simple test cases, visit http://pi.cr.yp.to/
If this were working, you could visit http://π.cr.yp.to/ (i.e. with a greek Pi)
but instead Mozilla goes crazy.
This should be simple to fix: just stop escaping the domain name.
Comment 16•23 years ago
|
||
very important to fix for international, since non ascii characters are
increasingly being used in domain names; changed to nsbeta1+; assigned to nhotta
-dassi
Assignee | ||
Comment 18•23 years ago
|
||
IDN (Internatioanl Domain Name) is now official (bug 112979). That means sending
a domain name as raw UTF-8 is not legal. The current behavior to escape a domain
name when IDN is not enable is the right behavior.
Removing nsbeta1+. Shall I mark this as wontfix?
Comment 19•23 years ago
|
||
yeah, i think that makes sense. marking WONTFIX.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → WONTFIX
Comment 20•23 years ago
|
||
So, the initally described behavior, (escape entire hostname) is the correct
behavior if we are not in an IDN-enabled mode?
Assignee | ||
Comment 21•23 years ago
|
||
Yes, if IDN is not enable, apply escaping per RFC 2396.
You need to log in
before you can comment on or make changes to this bug.
Description
•