New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 798114 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac
Pri: 2
Type: Task



Sign in to add a comment

Update Public Suffix for M-65 before Stable

Project Member Reported by mart...@martijnc.be, Dec 29 2017

Issue description

(Reposting explanation from Umbrella Bug, Issue 610495)

Chrome ships a copy of the Public Suffix List ( https://publicsuffix.org ) for a variety of security purposes. Because it's key to web origin security, it needs to be reliably available before any networking, and is thus baked into the binary (rather than a component updater).

The list changes over time, and so this list must be updated every release to include the latest changes.

This is a data file-only change that includes the expressed security policies of domain holders, from the domain holders.

For every release, a bug will be created to track that releases' updates to the PSL.
 
Labels: ReleaseBlock-Stable
Thanks, you beat me to it. I hadn't posted umbrella bugs because the PSL hasn't been updated nor has the root zone database for several releases. However, one of the maintainers finally had time to review the PRs, so we should update during the 65 Beta cycle.
NextAction: 2018-02-15
Summary: Update Public Suffix for M-65 before Stable (was: Update Public Suffix for M-65 before Branch)
I'll land the CL 2 weeks before M65 stable (after updating, if more PRs are merged before then).
Friendly ping to get an update on this issue as it is marked as release block stable for M65.

Thanks..!

Comment 4 by gov...@chromium.org, Feb 13 2018

M65 Stable promotion is coming VERY soon. Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and request a merge  into the release branch ASAP. Thank you.
Project Member

Comment 5 by bugdroid1@chromium.org, Feb 13 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/db2608b46afa5baa7ec26f16e97700e3b614e18f

commit db2608b46afa5baa7ec26f16e97700e3b614e18f
Author: martijn@martijnc.be <martijn@martijnc.be>
Date: Tue Feb 13 20:16:11 2018

Refresh Public Suffix List to the latest version.

This updates from revision
fe6b36209b2cee5740d8909e3f1c9624756a628a (2017-08-21)

to revision
0f3b07d9aab6d6c9fe74990af98316468d40f488 (2018-01-25)

Bug:  798114 
Change-Id: Ic2b68b8f5541d43867ce18bfc7c055df92d64833
Reviewed-on: https://chromium-review.googlesource.com/846806
Commit-Queue: Martijn Croonen <martijnc@chromium.org>
Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536439}
[modify] https://crrev.com/db2608b46afa5baa7ec26f16e97700e3b614e18f/net/base/registry_controlled_domains/effective_tld_names.dat
[modify] https://crrev.com/db2608b46afa5baa7ec26f16e97700e3b614e18f/net/base/registry_controlled_domains/effective_tld_names.gperf
[modify] https://crrev.com/db2608b46afa5baa7ec26f16e97700e3b614e18f/net/tools/tld_cleanup/README

Labels: Merge-Request-65 Target-65
Setting Merge-Request for the data-only change. See Issue 610495 for justification
Labels: -Pri-3 Pri-2
Project Member

Comment 8 by sheriffbot@chromium.org, Feb 14 2018

Labels: -Merge-Request-65 Merge-Review-65 Hotlist-Merge-Review
This bug requires manual review: Less than 16 days to go before AppStore submit on M65
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 9 by gov...@chromium.org, Feb 14 2018

Labels: -Merge-Review-65 Merge-Approved-65
Approving merge to M65 branch 3325 based on comment #6. Please merge ASAP. Thank you.
Project Member

Comment 10 by bugdroid1@chromium.org, Feb 14 2018

Labels: -merge-approved-65 merge-merged-3325
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e117e0dd710f60197560996d6fc5dcb6e3288fc0

commit e117e0dd710f60197560996d6fc5dcb6e3288fc0
Author: Ryan Sleevi <rsleevi@chromium.org>
Date: Wed Feb 14 16:08:53 2018

Refresh Public Suffix List to the latest version.

This updates from revision
fe6b36209b2cee5740d8909e3f1c9624756a628a (2017-08-21)

to revision
0f3b07d9aab6d6c9fe74990af98316468d40f488 (2018-01-25)

TBR=martijn@martijnc.be

(cherry picked from commit db2608b46afa5baa7ec26f16e97700e3b614e18f)

Bug:  798114 
Change-Id: Ic2b68b8f5541d43867ce18bfc7c055df92d64833
Reviewed-on: https://chromium-review.googlesource.com/846806
Commit-Queue: Martijn Croonen <martijnc@chromium.org>
Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#536439}
Reviewed-on: https://chromium-review.googlesource.com/919068
Cr-Commit-Position: refs/branch-heads/3325@{#462}
Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369}
[modify] https://crrev.com/e117e0dd710f60197560996d6fc5dcb6e3288fc0/net/base/registry_controlled_domains/effective_tld_names.dat
[modify] https://crrev.com/e117e0dd710f60197560996d6fc5dcb6e3288fc0/net/base/registry_controlled_domains/effective_tld_names.gperf
[modify] https://crrev.com/e117e0dd710f60197560996d6fc5dcb6e3288fc0/net/tools/tld_cleanup/README

Comment 11 by sleevi@google.com, Feb 14 2018

Status: Verified (was: Started)
The NextAction date has arrived: 2018-02-15
NextAction: ----
Blocking: -610495

Sign in to add a comment