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

Issue 749207 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Merge HSTS preload changes to M61.

Project Member Reported by lgar...@chromium.org, Jul 26 2017

Issue description

Requesting a merge of https://chromium.googlesource.com/chromium/src.git/+/ea7eb66c294a9bab471604434d878c9bab4a2388 and https://chromium.googlesource.com/chromium/src.git/+/e8bf5028e22c03199c5252e3613f099f88110d99 to M61.

We don't yet have an auto-roller (Issue 736188), so I still have to land these by hand. The changes intended for M61 ended up right after branch this time because I was OOO.
 
Labels: Merge-Request-61
Project Member

Comment 2 by sheriffbot@chromium.org, Jul 30 2017

Labels: -Merge-Request-61 Hotlist-Merge-Approved Merge-Approved-61
Your change meets the bar and is auto-approved for M61. Please go ahead and merge the CL to branch 3163 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid @(ChromeOS), govind@(Desktop)

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

Comment 3 by gov...@chromium.org, Jul 30 2017

Pls merge you change to M61 branch 3163 before 3:00 PM PT on Monday so we can take it in for next week last M61 Dev release. Thank you.
I was out sick yesterday. :-(

Merging now.
From  https://crbug.com/527947#c64 

--------------------------------

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

commit 6516d5948edd8936f9716900dba03f9198ad01db
Author: Lucas Garron <lgarron@chromium.org>
Date: Tue Aug 01 18:24:43 2017

HSTS preload list removals for Chrome 61.

rus-trip.ru:
> [love.rus-trip.ru] - This subdomain of the affiliate program, it was not
> planned to https

brztec.com:
> now this domains will host only static pages from blogspot and blogspot
> doesn't support HTTPS when using personalized domains.

supinbot.ovh:
> I mistakenly enabled HSTS preloading a while back but I frequently have to
> test new HTTP services that do not support HTTPS.

enterprise-threat-monitor.com:
> www.enterprise-threat-monitor.com - too many issues at enterprise customers

pixelneat.com:
> we added testing HSTS header by mistake and browsers already cached. We did
> not have a problem until letsencrypt SSL has been reloaded, so encryption keys
> are not the same which have browsers.

fumo.se:
> I have mistakenly used "includesubdomain and preload" which was not intended.

pepchid.com:
> I decide to use the new domain yilanju.com and set pepchid.com skip to the new
> one, so I request you for the removal.

xliu.cf:
> The reason is that my registration for this domain name has expired, and the
> provider Freenom is now reselling the domain.

inspy.me:
> The CA that issued my certificate (WoSign) became untrusted. And even though I
> have shut my project down I think it is still a good idea to remove it from
> HSTS preload list.

graysonsmith.co.uk:
> The domain is being transferred to a new company and they do not yet have SSL
> set up on their servers.

Bug:  527947 
Change-Id: Ic69a2aa871995041ec13cbe95ba65405a361c874

TBR=lgarron@chromium.org, palmer@chromium.org

(cherry picked from commit ea7eb66c294a9bab471604434d878c9bab4a2388)

Change-Id: Ic69a2aa871995041ec13cbe95ba65405a361c874
Reviewed-on: https://chromium-review.googlesource.com/565801
Reviewed-by: Lucas Garron <lgarron@chromium.org>
Commit-Queue: Lucas Garron <lgarron@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#488679}
Reviewed-on: https://chromium-review.googlesource.com/596534
Cr-Commit-Position: refs/branch-heads/3163@{#214}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/6516d5948edd8936f9716900dba03f9198ad01db/net/http/transport_security_state_static.json

Labels: -Merge-Approved-61 M-61 merge-merged-3163
Per comment #5, this is already merged to M61 branch 3163. Hence, removing "Merge-Approved-61" label and applying "merge-merged-3163"label. Thank you.
> Per comment #5, this is already merged to M61 branch 3163. Hence, removing "Merge-Approved-61" label and applying "merge-merged-3163"label. Thank you.

I still need to merge the second commit. Coming up right now.
Project Member

Comment 8 by bugdroid1@chromium.org, Aug 1 2017

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

commit 9c821120dfc5153f2b083cc11c0271141df100c9
Author: Lucas Garron <lgarron@chromium.org>
Date: Tue Aug 01 18:33:30 2017

HSTS preload list additions for Chrome 61.

Bug:  749207 
Change-Id: If7591266b870dc3dc93b136c3f9baa70f49cfa33

TBR=lgarron@chromium.org, palmer@chromium.org

(cherry picked from commit e8bf5028e22c03199c5252e3613f099f88110d99)

Change-Id: If7591266b870dc3dc93b136c3f9baa70f49cfa33
Reviewed-on: https://chromium-review.googlesource.com/565799
Commit-Queue: Lucas Garron <lgarron@chromium.org>
Reviewed-by: Lucas Garron <lgarron@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#489369}
Reviewed-on: https://chromium-review.googlesource.com/596535
Cr-Commit-Position: refs/branch-heads/3163@{#215}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/9c821120dfc5153f2b083cc11c0271141df100c9/net/http/transport_security_state_static.json

Status: Fixed (was: Assigned)

Sign in to add a comment