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

Issue 2511583002: Defang the CT Timebomb (Closed)

Created:
4 years, 1 month ago by Ryan Sleevi
Modified:
4 years, 1 month ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2883
Project:
chromium
Visibility:
Public.

Description

Defang the CT Timebomb A timebomb existed that if the set of known CT logs goes stale, no certificate can comply with the Certificate Transparency policy, beause the logs may be out of date and not trustworthy. The set of known logs goes stale 10 weeks after the build date. While this was acceptable to cause EV certificates to downgrade to DV certificates, this also means that certificates issued by CAs that MUST be CT compliant also fail to work - they fail closed, rather than fail open. In particular, certificates issued by Symantec fail to work if it's more than 10 weeks after the build date - in effect, the default behaviour is to distrust Symantec. While the proper behaviour is debated - to either fail open (like HSTS and HPKP do), treating CT as additive, or to fail closed, treating CT as a restrictive policy that must be made - change the code to allow an out of date build to skip the CT checks, failing open. BUG=664177 Review-Url: https://codereview.chromium.org/2495583002 Cr-Commit-Position: refs/heads/master@{#431707} (cherry picked from commit ec8e431e9a0f80ace76368ce7edce006f3d409f2) Committed: https://chromium.googlesource.com/chromium/src/+/98538c32c96bed2d6d68d89cb4d06c49b8f5e0b5

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M net/quic/chromium/crypto/proof_verifier_chromium.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M net/socket/ssl_client_socket_impl.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M net/spdy/spdy_session.cc View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Ryan Sleevi
4 years, 1 month ago (2016-11-16 22:44:32 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
98538c32c96bed2d6d68d89cb4d06c49b8f5e0b5.

Powered by Google App Engine
This is Rietveld 408576698