Status

()

Core
Networking: Domain Lists
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: gerv, Assigned: gerv)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(status1.9.2 .11-fixed, status1.9.1 .14-fixed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

7 years ago
There are a load which ICANN has approved here:
http://www.icann.org/en/topics/idn/fast-track/string-evaluation-completion-en.htm

They all need to be added to the PSL, so Chrome can navigate to them (isn't that right?), even if they don't have any substructure.

Gerv

Comment 1

7 years ago
Created attachment 465139 [details] [diff] [review]
Captures all IDN ccTLD approved by ICANN Board to date; Updates a number of ccTLDs and stubs, adds contributor

Patch covers bugs 580147, 580148, 580152, 564213, 575610, 576508, 576827, 580132, 580161, 580162, 584989, 584991, 584992
Attachment #465139 - Flags: review?

Comment 2

7 years ago
Created attachment 465142 [details] [diff] [review]
all.js / IDNTLD Updates - Captures all IDN ccTLD approved by ICANN Board to date; Updates a number of ccTLDs and stubs, adds contributor

This updates the IDN TLD list to catch it up to date with the Currently Approved By Board List of IDN TLDs.  It also adds .CC which follows exactly the .COM/NET IDN.

Addresses bugs: 580147, 580148, 580152, 564213, 575610, 576508
576827, 580132, 580161, 580162, 584989, 584991, 584992
(Assignee)

Comment 3

7 years ago
Hi Jothan, 

This is great work! 

One tip: if you put the word "bug" in front of every bug number in a comment, Bugzilla will autolinkify them all, which makes looking at them much easier. I've now added them to the Blocks list, so there will also be links to them at the top.

For the IDN TLD whitelist updates, we need documentation of the registry character list and anti-spoofing policy for each. I know you are working on obtaining this information. It would probably be easiest if you did this documentation by also providing a patch to this page:
http://www.mozilla.org/projects/security/tld-idn-policy-list.html
giving the relevant URLs.
That page is stored in a Mozilla Subversion repository.
http://viewvc.svn.mozilla.org/vc/projects/mozilla.org/trunk/

Gerv
(Assignee)

Comment 4

7 years ago
Comment on attachment 465139 [details] [diff] [review]
Captures all IDN ccTLD approved by ICANN Board to date; Updates a number of ccTLDs and stubs, adds contributor

r=gerv with the exception of the addition of com.lu, which doesn't seem to have either a bug number or an explanation.

Gerv
Attachment #465139 - Flags: review? → review+

Updated

7 years ago
Duplicate of this bug: 589751
(Assignee)

Comment 6

7 years ago
Hey chaps,

What's the status here? For the PSL, we need a new patch without the change I review minused. For the all.js, we need the docs I mentioned.

Gerv
(Assignee)

Comment 7

7 years ago
Comment on attachment 465142 [details] [diff] [review]
all.js / IDNTLD Updates - Captures all IDN ccTLD approved by ICANN Board to date; Updates a number of ccTLDs and stubs, adds contributor

Let's just focus on PSL stuff in this bug.

Gerv
Attachment #465142 - Attachment is obsolete: true
(Assignee)

Updated

7 years ago
No longer blocks: 564213, 575610, 580147, 580148, 580161, 580162, 584989, 584991
(Assignee)

Updated

7 years ago
(Assignee)

Comment 8

7 years ago
Created attachment 474690 [details] [diff] [review]
Patch v.2

Roll-up of all current PSL entry bugs (as in the dependencies).

Gerv
Assignee: nobody → gerv
Attachment #465139 - Attachment is obsolete: true
Status: NEW → ASSIGNED
(Assignee)

Comment 9

7 years ago
Comment on attachment 474690 [details] [diff] [review]
Patch v.2

Requesting approval-2.0, as the tree is currently metered. This patch is a rollup of all the pending changes to the Public Suffix List over the past few months.

Gerv
Attachment #474690 - Flags: approval2.0?
(Assignee)

Updated

7 years ago
Summary: Add more new IDN TLDs to the PSL → PSL update rollup
Comment on attachment 474690 [details] [diff] [review]
Patch v.2

a=beltzner
Attachment #474690 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 11

7 years ago
Fixed on trunk.
http://hg.mozilla.org/mozilla-central/rev/14dc66dd53c1

Gerv
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 12

7 years ago
This got backed out due to a failing test.

Gerv
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 13

7 years ago
Really fixed this time.
http://hg.mozilla.org/mozilla-central/rev/248ea690d671

Gerv
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 14

7 years ago
Comment on attachment 474690 [details] [diff] [review]
Patch v.2

Requesting branch approvals.

Gerv
Attachment #474690 - Flags: approval1.9.2.11?
Attachment #474690 - Flags: approval1.9.1.14?
Comment on attachment 474690 [details] [diff] [review]
Patch v.2

Approved for 1.9.2.11 and 1.9.1.14, a=dveditz
Attachment #474690 - Flags: approval1.9.2.11?
Attachment #474690 - Flags: approval1.9.2.11+
Attachment #474690 - Flags: approval1.9.1.14?
Attachment #474690 - Flags: approval1.9.1.14+

Comment 16

7 years ago
Eh ... part of this patch (blogspot.com) was invalidated by bug 598911
(Assignee)

Comment 17

7 years ago
1.9.1: http://hg.mozilla.org/releases/mozilla-1.9.1/rev/35450067aac0
1.9.2: http://hg.mozilla.org/releases/mozilla-1.9.2/rev/5d8b8a4a366a

Gerv
status1.9.1: --- → .14-fixed
status1.9.2: --- → .11-fixed
(Assignee)

Comment 18

7 years ago
(And yes, I removed blogspot.com before committing.)

Gerv
Depends on: 614565
Depends on: 598911
Broke the Hawaii DOE, bug 614565
Depends on: 608362
You need to log in before you can comment on or make changes to this bug.