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

Issue 156723002: Disable key verification by default. (Closed)

Created:
6 years, 10 months ago by Andrew T Wilson (Slow)
Modified:
6 years, 10 months ago
Reviewers:
Joao da Silva
CC:
chromium-reviews
Visibility:
Public.

Description

Disable key verification by default. The production DMServer does not yet generate the correct signatures, so we will stop verifying signatures by default. We will revert this CL once the signature verification vs DMServer in the final format is fully working. BUG=275291 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=249685

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -6 lines) Patch
M components/policy/core/common/cloud/cloud_policy_constants.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M components/policy/core/common/cloud/cloud_policy_validator_unittest.cc View 3 chunks +7 lines, -0 lines 0 comments Download
M components/policy/core/common/policy_switches.h View 1 chunk +1 line, -1 line 0 comments Download
M components/policy/core/common/policy_switches.cc View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Andrew T Wilson (Slow)
I'm changing our signature verification code (in a separate CL) in a way that is ...
6 years, 10 months ago (2014-02-06 16:29:25 UTC) #1
Joao da Silva
lgtm
6 years, 10 months ago (2014-02-06 17:22:12 UTC) #2
Andrew T Wilson (Slow)
The CQ bit was checked by atwilson@chromium.org
6 years, 10 months ago (2014-02-07 10:04:41 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/atwilson@chromium.org/156723002/1
6 years, 10 months ago (2014-02-07 10:04:52 UTC) #4
commit-bot: I haz the power
6 years, 10 months ago (2014-02-07 14:27:26 UTC) #5
Message was sent while issue was closed.
Change committed as 249685

Powered by Google App Engine
This is Rietveld 408576698