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

Side by Side Diff: net/quic/crypto/proof_verifier_chromium.h

Issue 2016143002: Expose when PKP is bypassed in SSLInfo. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Make CertVerifyResult Great Again. Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_QUIC_CRYPTO_PROOF_VERIFIER_CHROMIUM_H_ 5 #ifndef NET_QUIC_CRYPTO_PROOF_VERIFIER_CHROMIUM_H_
6 #define NET_QUIC_CRYPTO_PROOF_VERIFIER_CHROMIUM_H_ 6 #define NET_QUIC_CRYPTO_PROOF_VERIFIER_CHROMIUM_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 13 matching lines...) Expand all
24 class CTPolicyEnforcer; 24 class CTPolicyEnforcer;
25 class CertVerifier; 25 class CertVerifier;
26 class CTVerifier; 26 class CTVerifier;
27 class TransportSecurityState; 27 class TransportSecurityState;
28 28
29 // ProofVerifyDetailsChromium is the implementation-specific information that a 29 // ProofVerifyDetailsChromium is the implementation-specific information that a
30 // ProofVerifierChromium returns about a certificate verification. 30 // ProofVerifierChromium returns about a certificate verification.
31 class NET_EXPORT_PRIVATE ProofVerifyDetailsChromium 31 class NET_EXPORT_PRIVATE ProofVerifyDetailsChromium
32 : public ProofVerifyDetails { 32 : public ProofVerifyDetails {
33 public: 33 public:
34 ProofVerifyDetailsChromium();
35 ProofVerifyDetailsChromium(const ProofVerifyDetailsChromium&);
36 ~ProofVerifyDetailsChromium() override;
37
34 // ProofVerifyDetails implementation 38 // ProofVerifyDetails implementation
35 ProofVerifyDetails* Clone() const override; 39 ProofVerifyDetails* Clone() const override;
36 40
37 CertVerifyResult cert_verify_result; 41 CertVerifyResult cert_verify_result;
38 ct::CTVerifyResult ct_verify_result; 42 ct::CTVerifyResult ct_verify_result;
39 43
40 // pinning_failure_log contains a message produced by 44 // pinning_failure_log contains a message produced by
41 // TransportSecurityState::PKPState::CheckPublicKeyPins in the event of a 45 // TransportSecurityState::PKPState::CheckPublicKeyPins in the event of a
42 // pinning failure. It is a (somewhat) human-readable string. 46 // pinning failure. It is a (somewhat) human-readable string.
43 std::string pinning_failure_log; 47 std::string pinning_failure_log;
48
49 // True if PKP was bypassed due to a local trust anchor.
50 bool pkp_bypassed;
Ryan Sleevi 2016/06/09 23:32:22 If you added lines 34-36 to default initialize to
davidben 2016/06/10 17:24:23 +1
44 }; 51 };
45 52
46 // ProofVerifyContextChromium is the implementation-specific information that a 53 // ProofVerifyContextChromium is the implementation-specific information that a
47 // ProofVerifierChromium needs in order to log correctly. 54 // ProofVerifierChromium needs in order to log correctly.
48 struct ProofVerifyContextChromium : public ProofVerifyContext { 55 struct ProofVerifyContextChromium : public ProofVerifyContext {
49 public: 56 public:
50 ProofVerifyContextChromium(int cert_verify_flags, const BoundNetLog& net_log) 57 ProofVerifyContextChromium(int cert_verify_flags, const BoundNetLog& net_log)
51 : cert_verify_flags(cert_verify_flags), net_log(net_log) {} 58 : cert_verify_flags(cert_verify_flags), net_log(net_log) {}
52 59
53 int cert_verify_flags; 60 int cert_verify_flags;
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 101
95 TransportSecurityState* const transport_security_state_; 102 TransportSecurityState* const transport_security_state_;
96 CTVerifier* const cert_transparency_verifier_; 103 CTVerifier* const cert_transparency_verifier_;
97 104
98 DISALLOW_COPY_AND_ASSIGN(ProofVerifierChromium); 105 DISALLOW_COPY_AND_ASSIGN(ProofVerifierChromium);
99 }; 106 };
100 107
101 } // namespace net 108 } // namespace net
102 109
103 #endif // NET_QUIC_CRYPTO_PROOF_VERIFIER_CHROMIUM_H_ 110 #endif // NET_QUIC_CRYPTO_PROOF_VERIFIER_CHROMIUM_H_
OLDNEW
« no previous file with comments | « net/http/transport_security_state_unittest.cc ('k') | net/quic/crypto/proof_verifier_chromium.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698