Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(575)

Issue 2118723002: Require CT for the Symantec-operated, Thawte-branded roots (active and retired) (Closed)

Created:
4 years, 5 months ago by Ryan Sleevi
Modified:
4 years, 5 months ago
Reviewers:
eroman
CC:
chromium-reviews, cbentzel+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Require CT for the Symantec-operated, Thawte-branded roots (active and retired) BUG=620178 Committed: https://crrev.com/bd59b11124c47ae3e62bed05fc46cc924f7bc4b7 Cr-Commit-Position: refs/heads/master@{#403399}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+707 lines, -0 lines) Patch
A net/data/ssl/symantec/roots/3f9f27d583204b9e09c8a3d2066c4b57d3a2479c3693650880505698105dbce9.pem View 1 chunk +57 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/4b03f45807ad70f21bfc2cae71c9fde4604c064cf5ffb686bae5dbaad7fdd34c.pem View 1 chunk +82 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/5b38bd129e83d5a0cad23921089490d50d4aae370428f8ddfffffa4c1564e184.pem View 1 chunk +57 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/87c678bfb8b25f38f7e97b336956bbcf144bbacaa53647e61a2325bc1055316b.pem View 1 chunk +56 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/8d722f81a9c113c0791df136a2966db26c950a971db46b4199f4ea54b78bfb9f.pem View 1 chunk +82 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/a4310d50af18a6447190372a86afaf8b951ffb431d837f1e5688b45971ed1557.pem View 1 chunk +53 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/cb6b05d9e8e57cd882b10b4db70de4bb1de42ba48a7bd0318b635bf6e7781a9d.pem View 1 chunk +54 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/cbb02707160f4f77291a27561448691ca5901808e5f36e758449a862aa5272cb.pem View 1 chunk +122 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/f5074a8f5b9a5b8142f34abe152f60364d770eae75ee3eeceb45b6b996509788.pem View 1 chunk +57 lines, -0 lines 0 comments Download
A net/data/ssl/symantec/roots/f59db3f45d57fcec94ccd516e6c8ccb20dd4363feb2c44d8656e95f50fdd8df8.pem View 1 chunk +57 lines, -0 lines 0 comments Download
M net/http/transport_security_state_ct_policies.inc View 5 chunks +30 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (3 generated)
Ryan Sleevi
Eric: Stamp? :)
4 years, 5 months ago (2016-07-01 01:36:27 UTC) #2
eroman
lgtm
4 years, 5 months ago (2016-07-01 01:45:00 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2118723002/1
4 years, 5 months ago (2016-07-01 01:56:16 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-01 03:25:04 UTC) #6
commit-bot: I haz the power
CQ bit was unchecked.
4 years, 5 months ago (2016-07-01 03:25:05 UTC) #7
commit-bot: I haz the power
4 years, 5 months ago (2016-07-01 03:28:58 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/bd59b11124c47ae3e62bed05fc46cc924f7bc4b7
Cr-Commit-Position: refs/heads/master@{#403399}

Powered by Google App Engine
This is Rietveld 408576698