Closed Bug 455587 Opened 16 years ago Closed 16 years ago

Fix Public Suffix List entry for .se

Categories

(Core :: Networking, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: gerv, Unassigned)

Details

In the recent set of public suffix list updates, there was an error transcribing the entry for .se which the registry submitted. I found the error when reviewing the branch patch. The root "se" entry has been omitted - all there is is a list of permitted second-level domains. 

I _think_ this isn't bad, because foo.se (for a random value of 'foo') will fall back to the default rule of permitting cookies for foo.se but not se, but it's something we should fix.

Gerv
Do we need to fix this before 3.0.2 is released? We can if you do it soon.
Sorry, I should have been more clear. I fixed it pre-checkin for 3.0.2, in bug 454816. This bug covers fixing it on the trunk, so the trunk matches the branch in this regard.

Gerv
Gerv: are you 100% sure that it's right on trunk? I'm beginning to think that this patch is riskier than I thought, and maybe it's just better to back it out and wait for 3.0.3.
Ah, yes, looks like I used the right patch.
beltzner: it's definitely right on trunk, and on the relbranch link above. Search for "se :". Note that the "se" entry has not been removed.

I'm also reasonably certain that even if it was wrong, the right thing would actually happen.
"If no rule matches, the first-level domain (the portion of the hostname after its last dot, or the entire hostname if it contains no dots) will be used, whether or not it appears in the file."
https://wiki.mozilla.org/Gecko:TLD_Service
So my analysis in comment 0 is correct.

Gavin: there's a relbranch? I checked with mconnor on IRC and he said there wasn't one... Sorry about that.

Gerv
Gerv's comment #6 is correct: "se" will be used as the default for any "foo.se" not covered by any other rule.  See http://mxr.mozilla.org/seamonkey/source/netwerk/dns/src/nsEffectiveTLDService.cpp#213  All the real TLDs in the file are good for clarity, but not required.
So this got fixed on the relbranch, and it seems right here:
http://mxr.mozilla.org/firefox/source/netwerk/dns/src/effective_tld_names.dat

So resolving this as FIXED.

Gerv
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.