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

Issue 45373002: net: add pinning for liberty.lavabit.com. (Closed)

Created:
7 years, 1 month ago by agl
Modified:
7 years, 1 month ago
Reviewers:
palmer
CC:
chromium-reviews, cbentzel+watch_chromium.org, Chris Evans
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Reupload due to 500 error. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -0 lines) Patch
M net/http/transport_security_state.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/http/transport_security_state_static.h View 3 chunks +14 lines, -0 lines 0 comments Download
M net/http/transport_security_state_static.certs View 1 chunk +38 lines, -0 lines 0 comments Download
M net/http/transport_security_state_static.json View 2 chunks +7 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
agl
7 years, 1 month ago (2013-10-25 20:22:51 UTC) #1
palmer
Cert looks right, fingerprint looks right. LGTM.
7 years, 1 month ago (2013-10-25 20:56:43 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/agl@chromium.org/45373002/60001
7 years, 1 month ago (2013-10-28 14:33:29 UTC) #3
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=32845
7 years, 1 month ago (2013-10-28 14:50:26 UTC) #4
agl
7 years, 1 month ago (2013-10-28 19:29:56 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r231375 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698