Closed Bug 802568 Opened 12 years ago Closed 12 years ago

PSL uplift for Firefox 17

Categories

(Core Graveyard :: Networking: Domain Lists, defect)

defect
Not set
normal

Tracking

(firefox17+ fixed, firefox18+ fixed)

RESOLVED FIXED
Tracking Status
firefox17 + fixed
firefox18 + fixed

People

(Reporter: gerv, Assigned: gerv)

Details

Attachments

(2 files)

Firefox 17 is an ESR, and due for release on November 20th. We should uplift all the PSL changes which are on mozilla-central that it doesn't already have, which is quite a few. For consistency, we'll also need patches for the aurora channel.

All of these changes have baked on trunk, and they are config/policy changes rather than code changes. So they should be safe to take (and we have taken them many times in the past, even on the release channel).

We should work through the current batch of open requests, get those done, wait a few days, and then upload patches to this bug with all the necessary changes.

Gerv
Marking tracking-firefox-17?. 

The reason we'd want to take this for 17 is that it rolls in a number of policy changes which have been made since 17 branched from mozilla-central. We take changes to the IDN whitelist and the PSL on stable release branches anyway, but there has been quite a bit of activity recently and so it seems wiser and simpler to get them uplifted in time for the initial release of the ESR.

The next step is for me to produce a rollup patch, which I will do shortly. 

Gerv
This patch updates the PSL and IDN whitelist on Aurora to be the same as that of mozilla-central.

Gerv
Assignee: nobody → gerv
Status: NEW → ASSIGNED
This patch updates the PSL and IDN whitelist on Beta to be the same as that of mozilla-central.

Gerv
Comment on attachment 680059 [details] [diff] [review]
Patch v.1 for mozilla-aurora

[Approval Request Comment]
Bug caused by (feature/regressing bug #): N/A

User impact if declined: IDN not displaying correctly in .com, .net., .name, .gt, and domain display incorrect/cookie security parameters incorrect for various domains (see patch)

Testing completed (on m-c, etc.): A couple of policy changes are recent (finalized in time for ESR) but others have been baking for weeks.

Risk to taking this patch (and alternatives if risky): Near-zero.

String or UUID changes made by this patch: None.

Gerv
Attachment #680059 - Flags: approval-mozilla-aurora?
Comment on attachment 680060 [details] [diff] [review]
Patch v.1 for mozilla-beta

[Approval Request Comment]
Bug caused by (feature/regressing bug #): N/A

User impact if declined: IDN not displaying correctly in .com, .net., .name, .gt, and domain display incorrect/cookie security parameters incorrect for various domains (see patch)

Testing completed (on m-c, etc.): A couple of policy changes are recent (finalized in time for ESR) but others have been baking for weeks.

Risk to taking this patch (and alternatives if risky): Near-zero.

String or UUID changes made by this patch: None.

Gerv
Attachment #680060 - Flags: approval-mozilla-beta?
Comment on attachment 680059 [details] [diff] [review]
Patch v.1 for mozilla-aurora

Approving for Aurora, we'll get bake time there.  Gerv is there any reason this waited until just before the last Beta to get tracked?  It was filed 3 weeks ago and it would have been nice to know about it sooner.
Attachment #680059 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #680060 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
For Beta - we are going to build with our last 17 beta on Monday so please land this asap.
(In reply to Lukas Blakk [:lsblakk] from comment #6)
> Approving for Aurora, we'll get bake time there.  Gerv is there any reason
> this waited until just before the last Beta to get tracked?  It was filed 3
> weeks ago and it would have been nice to know about it sooner.

I'm afraid the reason is that I suck :-| Sorry.

Aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/e5aa14969f18
Beta:   https://hg.mozilla.org/releases/mozilla-beta/rev/c32e71b351b2

Gerv
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.