469866 - Update Public Suffix (effective TLD) list (4)

Status: RESOLVED FIXED

(Core :: Networking, defect)

Description

[David Triendl]

Attachment: Update for public suffix list

This is the 4th round of updating the public suffix list (network/dns/src/effective_tld_names.dat).

Changes:
* Update .vi
* Update .iq
* Update .cr
* Update .ru (thanks to Zac)
* Update .jp as requested by registry (thanks to Yoshiro Yoneya)

As I updated .ru to the official registry list, I'll make bug 430971 a duplicate of this bug.

Comment 2

[Gervase Markham [:gerv]]

Comment on attachment 353265 [details] [diff] [review]
Update for public suffix list

Removing names from the list is obviously more controversial than adding new ones. Can you explain how you know that it's OK to remove those .ru and .vi entries?

Gerv

Comment 3

[Gervase Markham [:gerv]]

We dropped this one on the floor :-( Some of these changes have been made, but not all. The patch needs refreshing.

Gerv

Comment 4

[Gervase Markham [:gerv]]

Peter, Adam: are either of you able to take this on?

Gerv

Comment 5

[Peter Kasting]

I can look up Wikipedia entries and registry homepages for the changes here if you want.  I'm not sure whether that's sufficient for you; what would you like to see from someone who "takes this on"?

Comment 6

[Gervase Markham [:gerv]]

What I was hoping for was for someone to look through David's patch, work out which of the changes he made have not yet been included in the current list, confirm those changes are still valid (which is fairly likely) and produce a new patch against the trunk for me to review :-)

Gerv

Comment 7

[Peter Kasting]

I'll try to do this.

Comment 8

[Peter Kasting]

Attachment: Updated patch

I double-checked all the old changes and added a few more.

Here are the differences from the old patch:
* .cr changes had already been applied, but with mismatched line endings; fixed those
* Modified comments slightly for clarity/accuracy
* Attempted to update .pa/.sa, about which we've debated in the past; the current .sa rules are quite clear, while the .pa rules are slightly less so (all public documentation agrees, but what it agrees on doesn't quite cover all of reality)
* Avoided collapsing Russian geographic rules from two sections to one, since the registrar didn't, and this makes both the patch diff and the diff of our list against the registrar slightly saner.  I did reorder a few rules in the name of alphabetization.
* Removed some inaccurate comments in the .us section and tried to replace with an accurate one

Comment 9

[Gervase Markham [:gerv]]

Comment on attachment 401265 [details] [diff] [review]
Updated patch

I think that, in the name of future-proofing the list, if edu.vi and gov.vi are reserved, then we should add them. The only thing that could go wrong is if someone set up a website at http://edu.vi. And if they are reserved, that's not going to happen. 

What happened to tagil.ru?

Gerv

Comment 10

[Peter Kasting]

(In reply to comment #9)
> (From update of attachment 401265 [details] [diff] [review])
> I think that, in the name of future-proofing the list, if edu.vi and gov.vi are
> reserved, then we should add them.

The problem with this is that the .vi registrar also lists thousands of other domains that are "reserved".  I feel like we should be consistent, and I don't see any benefit to getting the others.  If you can't purchase .edu.vi, and it's never been available in years, I'd prefer the simpler and more accurate file for now.

> What happened to tagil.ru?

It is not listed on the registrar page.  Considering that it was the _only_ existing domain not listed, I assumed I wasn't just looking at a partial list.

Comment 11

[Gervase Markham [:gerv]]

Comment on attachment 401265 [details] [diff] [review]
Updated patch

r=gerv.

Gerv

Comment 12

[Gervase Markham [:gerv]]

Let's get this on the trunk to start with, and then think about branches. Peter: do you need me to check it in?

Gerv

Comment 13

[Peter Kasting]

(In reply to comment #12)
> Peter: do you need me to check it in?

Sure, go ahead.  Any chance you could also r+/checkin bug 513834?  We (Chromium folks) are still interested in that.

Comment 14

[Gervase Markham [:gerv]]

http://hg.mozilla.org/mozilla-central/rev/cac6c26d9e5a

After a couple of days, we'll nominate this for the two branches.

Gerv

Comment 15

[Gervase Markham [:gerv]]

Nominating for branches. Companion bug 513834. Justification: bringing public suffix list closer to reality.

Gerv

Comment 16

[Daniel Veditz [:dveditz]]

Comment on attachment 401265 [details] [diff] [review]
Updated patch

Approved for 1.9.1.5 and 1.9.0.16, a=dveditz for release-drivers

Comment 17

[Gervase Markham [:gerv]]

Checked in on all branches.

Checking in netwerk/dns/src/effective_tld_names.dat;
/cvsroot/mozilla/netwerk/dns/src/effective_tld_names.dat,v  <--  effective_tld_names.dat
new revision: 1.11; previous revision: 1.10

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/3aa22391ebe7d5aac6d56813418092b329d92cac

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/22e923bf460b4de772063d121d42ff40a5ce1948

Gerv

Comment 18

[Al Billings [:abillings - ex-MoCo]]

Verified for 1.9.0.16 and 1.9.1.6 in source. Nothing else for QA here.