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

Issue 731836 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac , Fuchsia
Pri: 1
Type: Bug



Sign in to add a comment

Remove PuChuangSiDa Certificate Transparency Log

Project Member Reported by rsleevi@chromium.org, Jun 9 2017

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Jun 12 2017

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

commit 85bf190d18689ad2466771fb99b0d353aa25e5ae
Author: rsleevi <rsleevi@chromium.org>
Date: Mon Jun 12 15:51:58 2017

Remove PuChuangSiDa as a known Certificate Transparency Log

As noted on
https://groups.google.com/a/chromium.org/forum/#!topic/ct-policy/M-rhwDQ1h9E
, the log has failed to abide by the uptime requirement.

BUG= 731836 
TBR=eroman@chromium.org

Review-Url: https://codereview.chromium.org/2932093002
Cr-Commit-Position: refs/heads/master@{#478639}

[modify] https://crrev.com/85bf190d18689ad2466771fb99b0d353aa25e5ae/net/data/ssl/certificate_transparency/log_list.json

Status: Verified (was: Started)
Labels: Merge-Request-60
Already have approval to merge two log_list.json changes that happened after this (crbug.com/692782,  crbug.com/703700 ), but one of them conflicts with this change. Can I get approval to merge this change as well, to simplify the process?
Cc: awhalley@chromium.org
Pending LGTM from rsleevi@/awhalley@ of course.
Project Member

Comment 5 by sheriffbot@chromium.org, Jul 3 2017

Labels: -Merge-Request-60 Hotlist-Merge-Review Merge-Review-60
This bug requires manual review: M60 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
This is an opportunistic merge of a data-only change with no user impact.
Good for M60
Please add appropriate OSs.  Thanks.
Labels: OS-Android OS-Chrome OS-Fuchsia OS-Linux OS-Mac OS-Windows
Project Member

Comment 10 by bugdroid1@chromium.org, Jul 6 2017

Labels: merge-merged-3112
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/317ac81885617d1f3d0d3f18175fbfec65c2ee9f

commit 317ac81885617d1f3d0d3f18175fbfec65c2ee9f
Author: Eran Messeri <eranm@google.com>
Date: Thu Jul 06 10:57:32 2017

Remove PuChuangSiDa as a known Certificate Transparency Log

As noted on
https://groups.google.com/a/chromium.org/forum/#!topic/ct-policy/M-rhwDQ1h9E
, the log has failed to abide by the uptime requirement.

BUG= 731836 
TBR=eroman@chromium.org

Review-Url: https://codereview.chromium.org/2932093002
Cr-Original-Commit-Position: refs/heads/master@{#478639}
Review-Url: https://codereview.chromium.org/2966373003 .
Cr-Commit-Position: refs/branch-heads/3112@{#527}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}

[modify] https://crrev.com/317ac81885617d1f3d0d3f18175fbfec65c2ee9f/net/data/ssl/certificate_transparency/log_list.json

Labels: -Merge-Review-60 Merge-Approved-60
Can we please make sure to not merge until Merge-Approved-60 label is applied? We need to ensure that Release Owner has provided approval. 

Marking bug as Merge-Approved-60. 
My bad, sorry, I was under the impression it was approved, as it's similar to a few other merge requests mentioned above.
Labels: -Merge-Approved-60 M-60
Sorry about that, I too saw "Good for M60" and missed the fact that the approval hadn't actually been granted yet.
My fault, sorry - should have been more explicit.

Sign in to add a comment