Last Comment Bug 632595 - Update public suffix list for changes in .eg and .uk
: Update public suffix list for changes in .eg and .uk
: verified1.9.2
Product: Core
Classification: Components
Component: Networking: Domain Lists (show other bugs)
: unspecified
: x86 Windows Vista
-- normal (vote)
: ---
Assigned To: Gervase Markham [:gerv]
: Patrick McManus [:mcmanus]
Depends on:
  Show dependency treegraph
Reported: 2011-02-08 15:42 PST by Peter Kasting
Modified: 2011-06-16 17:17 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch v.1 (762 bytes, patch)
2011-03-15 06:51 PDT, Gervase Markham [:gerv]
pkasting: review+
dveditz: approval2.0+
dveditz: approval1.9.2.18+
dveditz: approval1.9.1.20+
Details | Diff | Splinter Review

Description User image Peter Kasting 2011-02-08 15:42:27 PST
Recently two Chromium bugs ( and ) have pointed out websites that run afoul of the current public suffix rules: and, respectively.

Looks like we need to take a look at these registrars' current policies and update.
Comment 1 User image Gervase Markham [:gerv] 2011-03-15 06:51:34 PDT
Created attachment 519395 [details] [diff] [review]
Patch v.1

I checked a few sources and can't find any more omissions from the .uk list apart from The .eg stuff is straight from Wikipedia.

Comment 2 User image Peter Kasting 2011-04-01 16:31:50 PDT
Comment on attachment 519395 [details] [diff] [review]
Patch v.1

I'm sorry, this slipped through my radar somehow!
Comment 3 User image Gervase Markham [:gerv] 2011-05-20 06:16:47 PDT

Comment 4 User image Mounir Lamouri (:mounir) 2011-05-20 07:57:39 PDT
Backed out due to xpcshell failure on all platforms.
Comment 5 User image Gervase Markham [:gerv] 2011-05-20 08:58:09 PDT
The cause of the failure was this bug; the new TLDs for .pk contain what Firefox things is a non-permitted codepoint (200E). And the Verisign IDN checker agrees:

I will check this all back in without that change, and we can open another bug to fix the problem.

Comment 6 User image Gervase Markham [:gerv] 2011-05-20 09:03:29 PDT
Sorry, comment 5 was in the wrong bug (see bug 658084).

Comment 7 User image Gervase Markham [:gerv] 2011-05-20 09:14:11 PDT
Attempt 2:

Comment 8 User image Daniel Veditz [:dveditz] 2011-05-23 10:25:28 PDT
Comment on attachment 519395 [details] [diff] [review]
Patch v.1

Approved for and, a=dveditz for release-drivers
Approved for the mozilla2.0 repository, a=dveditz for release-drivers

Mozilla only has plans to build 1.9.2 releases, but you're approved to land on the other two repos if you'd like.
Comment 10 User image Al Billings [:abillings] 2011-06-16 17:17:03 PDT
Verified for in source.

Note You need to log in before you can comment on or make changes to this bug.