Enable IDN Display for .срб

RESOLVED FIXED in Firefox 11

Status

()

Core
Networking: Domain Lists
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: RNIDS, Assigned: gerv)

Tracking

unspecified
mozilla13
Points:
---

Firefox Tracking Flags

(firefox11 fixed, firefox12 fixed)

Details

(Whiteboard: [qa-])

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 575829 [details]
Enable IDN display for.pdf

User Agent: Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; Trident/5.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C)

Steps to reproduce:

Enter IDN domain names .срб ( xn--90a3ac )


Actual results:

Firefox display punicode, only.


Expected results:

Firefox display IDN domain name.

Comment 1

6 years ago
Gerv, do you handle these, or someone else?
Status: UNCONFIRMED → NEW
Component: General → Networking
Ever confirmed: true
QA Contact: general → networking
(Assignee)

Comment 2

6 years ago
bz: jothan is first point of contact these days.

Gerv
Component: Networking → Networking: Domain Lists
OS: Windows 7 → All
QA Contact: networking → networking.domain-lists
Hardware: x86 → All

Comment 3

6 years ago
While Mozilla uses the whitelist TLD process for mixed script security on cookies and location bar rendering, the presence of Cyrillic and Latin combined in a mixed script string is problematic.

In reviewing the policy and codepoints, I note that there is not any policy about mixed script registration, and there are present a number of visually identical characters in particular between Cyrillic and Latin (LDH) characters.

Example : 
The example of paypal in mixed script confusable is www.pаypal.срб [www.xn--pypal-4ve.xn--90a3ac] seems possible under currently defined policy at registry.
(Reporter)

Comment 4

6 years ago
Mixed script is not allowed in .срб. Only Serbian Cyrillic characters are alowed. The princip is the same as .рф.
[www.xn--pypal-4ve.xn--90a3ac] is not possible. There is no l in Serbian Cyrillic.
p, a, y are visually identical characters, but different characters with different codes. Same as Russian .рф.

Comment 5

6 years ago
Thank you for clarifying that there is NO MIXED script registration under .срб. Tomislav.  I apparently misread LDH as being also allowed, I apologize for any grief.

While caxap or other whole-string situations where a string could be composed of characters that could spell something appearing to look like Latin, it is entirely unlikely in the presence of a Cyrillic only policy that these would be mistaken, and “рачра1” would not be so visually similar it would confuse anyone, IMHO.

At this time, I propose it be added, and I will work on the patch file(s) required. 

In following our process, could I please have two votes of approval from within our development community?
Status: NEW → ASSIGNED
(Assignee)

Comment 6

6 years ago
If only Serbian Cyrillic is allowed, then that's fine.

Gerv
(Assignee)

Updated

6 years ago
Duplicate of this bug: 721720
(Reporter)

Comment 8

6 years ago
Any news about this? When do we can expect the patch?

Comment 9

6 years ago
Guys, PLEASE fix this as soon as possible - .срб TLD went public today!! http://www.rnids.rs/en/what-s-new/srb-domain-registration-begins/id/3428

Here's the info requested here http://www.mozilla.org/projects/security/tld-idn-policy-list.html for .срб TLD:

- TLD: .срб (.xn--90a3ac)
- Registry (RNIDS) homepage: http://rnids.rs
- Rulebook on .срб sunrise: http://rnids.rs/data/DOKUMENTI/Rules-commencement-SRB.pdf
- Rulebook on registration of .срб domains: http://rnids.rs/data/DOKUMENTI/RegistrationSRB.pdf (permitted character set defined in Article 2(3), also on IANA site http://www.iana.org/domains/idn-tables/tables/rs_sr-rs_1.0.pdf)

You can contact Mr. Tomislav Ciganovic (tomislav.ciganovic@rnids.rs), CTO of RNIDS, for any further clarification.

Thanks,
Sloba
(Assignee)

Comment 10

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/3a3a2fc1c1ac

Gerv
https://hg.mozilla.org/mozilla-central/rev/3a3a2fc1c1ac
Assignee: nobody → gerv
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
(Assignee)

Comment 12

6 years ago
Created attachment 600872 [details] [diff] [review]
Patch v.1
(Assignee)

Comment 13

6 years ago
Comment on attachment 600872 [details] [diff] [review]
Patch v.1

[Approval Request Comment]
User impact if declined: IDNs not working (displaying as gibberish) in relevant domain.
Testing completed (on m-c, etc.): Policy rather than code change; but patch has baked over the weekend.
Risk to taking this patch (and alternatives if risky): Very low.
String changes made by this patch: None.

Gerv
Attachment #600872 - Flags: approval-mozilla-beta?
Attachment #600872 - Flags: approval-mozilla-aurora?
Comment on attachment 600872 [details] [diff] [review]
Patch v.1

[Triage Comment]
Assuming this has gone through all the proper processes, approving for Aurora 12 and Beta 11. We do not expect to see any regressions caused by this patch which, as Gerv notes, is really just policy change.
Attachment #600872 - Flags: approval-mozilla-beta?
Attachment #600872 - Flags: approval-mozilla-beta+
Attachment #600872 - Flags: approval-mozilla-aurora?
Attachment #600872 - Flags: approval-mozilla-aurora+
(Assignee)

Comment 15

6 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/c039ae260920
https://hg.mozilla.org/releases/mozilla-beta/rev/752f3244405a

Gerv
status-firefox11: --- → fixed
status-firefox12: --- → fixed
Is there anything QA can do to verify this fix?
Whiteboard: [qa?]
Whiteboard: [qa?] → [qa-]
You need to log in before you can comment on or make changes to this bug.