Closed Bug 1415870 Opened 4 years ago Closed 4 years ago not in public suffix list


(Core :: Networking, defect, P5)




Tracking Status
firefox-esr52 --- wontfix
firefox57 --- wontfix
firefox58 + fixed
firefox59 --- fixed


(Reporter: daleharvey, Assigned: daleharvey)



(Whiteboard: [necko-triaged])


(1 file)

A friend asked me to look into this, hasnt been updated since 2015 and doesnt contain many recent additions to the public suffix list. 

Looking at not 100% certain its actually being used anyway
Gerv as the last committer is this something you know about, should we do another manual update from github?

Assignee: nobody → dharvey
Flags: needinfo?(gerv)
So, this is a bit of a whoopsie :-( We should have had a process for importing this regularly :-(( Yes, please import from Github.

I'm pretty sure it's being used, though - us not using it would be very odd given how many web platform features depend on it.

Flags: needinfo?(gerv)
effective_tld_names.dat is by some python that generates, which is used in
Priority: -- → P5
Whiteboard: [necko-triaged]
See Also: → 1416247
Comment on attachment 8926868 [details]
Bug 1415870 - Update psl list and related tests.

Looks fine, but of course I can't check each individual change. If we are importing the new file entirely, that's right.

Attachment #8926868 - Flags: review?(gerv) → review+
Yup it was just a full replacement, cheers
[Tracking Requested - why for this release]:

We'll want to make sure this makes 58, I think, to ensure we get some testing on it before shipping the updates with 59 esr.
Gerv any chance you could r+ this in mozreview? easier to autoland than to push directly
Pushed by
Update PSL to latest version from github. r=gerv
Backed out for failing XPCshell's netwerk/test/unit/test_psl.js:

Push with failures:
Failure log:

[task 2017-11-14T16:16:51.186Z] 16:16:51     INFO -  TEST-PASS | netwerk/test/unit/test_psl.js | checkPublicSuffix - [checkPublicSuffix : 35] "" == ""
[task 2017-11-14T16:16:51.187Z] 16:16:51     INFO -  TEST-PASS | netwerk/test/unit/test_psl.js | checkPublicSuffix - [checkPublicSuffix : 35] null == null
[task 2017-11-14T16:16:51.195Z] 16:16:51  WARNING -  TEST-UNEXPECTED-FAIL | netwerk/test/unit/test_psl.js | checkPublicSuffix - [checkPublicSuffix : 35] "" == null
[task 2017-11-14T16:16:51.196Z] 16:16:51     INFO -  /builds/worker/workspace/build/tests/xpcshell/tests/netwerk/test/unit/test_psl.js:checkPublicSuffix:35
[task 2017-11-14T16:16:51.197Z] 16:16:51     INFO -  /builds/worker/workspace/build/tests/xpcshell/tests/netwerk/test/unit/test_psl.js -> file:///builds/worker/workspace/build/tests/xpcshell/tests/netwerk/test/unit/data/test_psl.txt:null:42
[task 2017-11-14T16:16:51.198Z] 16:16:51     INFO -  /builds/worker/workspace/build/tests/xpcshell/tests/netwerk/test/unit/test_psl.js:run_test:18
[task 2017-11-14T16:16:51.198Z] 16:16:51     INFO -  /builds/worker/workspace/build/tests/xpcshell/head.js:_execute_test:540
[task 2017-11-14T16:16:51.199Z] 16:16:51     INFO -  -e:null:1
[task 2017-11-14T16:16:51.200Z] 16:16:51     INFO -  exiting test
[task 2017-11-14T16:16:51.201Z] 16:16:51     INFO -  "CONSOLE_MESSAGE: (info) No chrome package registered for chrome://branding/locale/"
[task 2017-11-14T16:16:51.201Z] 16:16:51     INFO -  <<<<<<<
Flags: needinfo?(dharvey)
Ah ok, the tests depended on some rules that have since changes (*.il) is no longer a wildcard, updated to an equivalent test and giving it a try run @
Flags: needinfo?(dharvey)
That test seems fine now but seeing failures with 

> FAIL | netwerk/test/unit/test_bug414122.js | checkPublicSuffix
> [checkPublicSuffix : 57] "ar" == "" was a change from the previous list, but having a hard time figuring out exactly what test_bug414122.js is trying to test and why this (and a few other changes) in particular fail
Failures may be due to artifact builds, running another full build test @
Yup tests are looking good with a full rebuild, relanding
Pushed by
Update psl list and related tests. r=gerv
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Please request Beta approval on this when you get a chance.
Flags: needinfo?(dharvey)
Comment on attachment 8926868 [details]
Bug 1415870 - Update psl list and related tests.

Approval Request Comment
[Feature/Bug causing the regression]: Outdated external list
[User impact if declined]: Domains isolation not working as expected
[Is this code covered by automated tests?]: yes
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: 
[List of other uplifts needed for the feature/fix]:
[Is the change risky?]: nope
[Why is the change risky/not risky?]: routine import of data from github
[String changes made/needed]:
Flags: needinfo?(dharvey)
Attachment #8926868 - Flags: approval-mozilla-beta?
Comment on attachment 8926868 [details]
Bug 1415870 - Update psl list and related tests.

Update the public suffix list. Beta58+.
Attachment #8926868 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.