Closed
Bug 1024964
Opened 10 years ago
Closed 10 years ago
Uplift recent PSL changes to Firefox 31 and 32
Categories
(Firefox :: General, defect)
Tracking
()
VERIFIED
FIXED
Firefox 31
People
(Reporter: gerv, Assigned: gerv)
Details
Attachments
(1 file)
13.89 KB,
patch
|
gerv
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
We recently made two large batches of changes to the Public Suffix List, clearing a several-month backlog. These changes included a change for .uk to reflect the fact that they have now gone live with top-level registration. .uk is a busy TLD, and errors here will get noticed. The impact if the uplift doesn't happen is that sites using the new wibble.uk domain names may not be able to set cookies across their multiple servers. We should uplift the current PSL file en masse to Firefox 31 (beta) and 32 (aurora). (Let me know if you need two separate bugs for this.) The file is known to pass tests on mozilla-central and so the risk is minimal. Here's the .uk change: https://hg.mozilla.org/mozilla-central/rev/8597415940ed although I'd want to take the file wholesale, including all the updates. Gerv
Assignee | ||
Updated•10 years ago
|
tracking-firefox31:
--- → ?
tracking-firefox32:
--- → ?
Comment 1•10 years ago
|
||
Why do you open a new bug for this? bug 1015214 is a better place for an uplift request.
Flags: needinfo?(gerv)
Assignee | ||
Comment 2•10 years ago
|
||
Bug 1015214 only covers the .uk changes. As I said, I want to upload all the recent changes, which are about 15 bugs. I will attach a patch or patches in a moment. Gerv
Flags: needinfo?(gerv)
Assignee | ||
Comment 3•10 years ago
|
||
This patch contains all the recent PSL changes, and applies to both mozilla-aurora and mozilla-beta. It makes the PSL the same on those branches as it is on mozilla-central. Gerv
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8439888 [details] [diff] [review] Patch v.1 [Approval Request Comment] Bug caused by (feature/regressing bug #): Rollup patch for 17 bugs, listed here: https://bugzilla.mozilla.org/buglist.cgi?resolution=FIXED&chfieldto=2014-06-13&chfield=bug_status&chfieldfrom=2014-06-08&component=Networking%3A%20Domain%20Lists User impact if declined: problems with PSL-related functionality, including cookie-setting, on various domains, including the new wibble.uk domains. If patch is not taken, problems will persist for 12 weeks longer than otherwise. Testing completed (on m-c, etc.): Passes all tests on m-c. Risk to taking this patch (and alternatives if risky): Very low. String or IDL/UUID changes made by this patch: none. Gerv
Attachment #8439888 -
Flags: approval-mozilla-beta?
Attachment #8439888 -
Flags: approval-mozilla-aurora?
Comment 5•10 years ago
|
||
Comment on attachment 8439888 [details] [diff] [review] Patch v.1 31 will be an ESR. Taking it for this release.
Attachment #8439888 -
Flags: approval-mozilla-beta?
Attachment #8439888 -
Flags: approval-mozilla-beta+
Attachment #8439888 -
Flags: approval-mozilla-aurora?
Attachment #8439888 -
Flags: approval-mozilla-aurora+
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/5301b6d459ae https://hg.mozilla.org/releases/mozilla-beta/rev/4f05af9f1d85
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
Updated•10 years ago
|
Keywords: checkin-needed
Flagging for verification to keep an eye out for potentially broken websites due to this change.
Keywords: verifyme
Comment 8•10 years ago
|
||
Should we uplift this to 24 seeing as we'll still be supporting it for a while after 31 gets out, if I understand our ESR policy correctly?
status-firefox-esr24:
--- → affected
Flags: needinfo?(gerv)
Assignee | ||
Comment 9•10 years ago
|
||
I have no objections. But I don't think we'd ship just for this. Do we plan another point release? Gerv
Flags: needinfo?(gerv)
Comment 10•10 years ago
|
||
We release in parallel of the release version. So, in about 2 weeks. See: https://www.mozilla.org/en-US/firefox/organizations/faq/
Comment 11•10 years ago
|
||
(In reply to Gervase Markham [:gerv] from comment #9) > I have no objections. But I don't think we'd ship just for this. Do we plan > another point release? > > Gerv Yes, apparently: http://www.mozilla.org/en-US/firefox/organizations/faq/ Can you handle the approval request and/or check that the patch applies cleanly? :-)
Assignee | ||
Comment 12•10 years ago
|
||
Gijs: bug 1033340. Gerv
Comment 13•10 years ago
|
||
Verified fixed on Firefox 31 Beta 8 (Build ID: 20140707160635), Windows 7 64-bit, using several newly added/removed domains. Monitored the Network:Cookies panel and the Browser Console's output for potential issues. Further details can be found in this etherpad [1]. Gerv, if there's anything else we should look at here, please let me know. [1] https://etherpad.mozilla.org/PSL-Fx31
Keywords: verifyme
Assignee | ||
Comment 14•10 years ago
|
||
No, that sounds great. Thanks! Gerv
Comment 15•10 years ago
|
||
(In reply to Gervase Markham [:gerv] from comment #14) > No, that sounds great. Thanks! > > Gerv Thank you for confirming. I'm going to mark this issue verified fixed and follow up with my results from Aurora 32 and Nightly 33 as soon as possible.
Status: RESOLVED → VERIFIED
Comment 16•10 years ago
|
||
Verified fixed on Aurora 32.0a2 2014-07-08 (Build ID: 20140708004001) and Nightly 33.0a1 2014-07-08 (Build ID: 20140708030203) as well. Updated the etherpad from Comment 13 with detailed results.
Comment 17•10 years ago
|
||
If this is to get on ESR24 we'll need a nomination asap (and make sure it applies to that branch).
Flags: needinfo?(gerv)
Assignee | ||
Comment 18•10 years ago
|
||
Lukas: it went on to ESR 24 in bug 1033340. Gerv
Flags: needinfo?(gerv)
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•