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

Side by Side Diff: net/cert/ct_verifier.h

Issue 92443002: Extract Certificate Transparency SCTs from stapled OCSP responses (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@extract_scts
Patch Set: review comments Created 7 years 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_CERT_CT_VERIFIER_H_ 5 #ifndef NET_CERT_CT_VERIFIER_H_
6 #define NET_CERT_CT_VERIFIER_H_ 6 #define NET_CERT_CT_VERIFIER_H_
7 7
8 #include "net/base/net_export.h" 8 #include "net/base/net_export.h"
9 9
10 namespace net { 10 namespace net {
11 11
12 namespace ct { 12 namespace ct {
13 struct CTVerifyResult; 13 struct CTVerifyResult;
14 } // namespace ct 14 } // namespace ct
15 15
16 class BoundNetLog; 16 class BoundNetLog;
17 class X509Certificate; 17 class X509Certificate;
18 18
19 // Interface for verifying Signed Certificate Timestamps over a certificate. 19 // Interface for verifying Signed Certificate Timestamps over a certificate.
20 class NET_EXPORT CTVerifier { 20 class NET_EXPORT CTVerifier {
21 public: 21 public:
22 virtual ~CTVerifier() {} 22 virtual ~CTVerifier() {}
23 23
24 // Verifies either embedded SCTs or SCTs obtained via the 24 // Verifies either embedded SCTs or SCTs obtained via the
25 // signed_certificate_timestamp TLS extension or OCSP on the given |cert| 25 // signed_certificate_timestamp TLS extension or OCSP on the given |cert|
wtc 2013/12/10 04:23:17 The comment should be updated because the second p
ekasper 2013/12/10 14:45:20 The comment is also wrong because it's not either/
26 // |result| will be filled with these SCTs, divided into categories based on 26 // |result| will be filled with these SCTs, divided into categories based on
27 // the verification result. 27 // the verification result.
28 virtual int Verify(X509Certificate* cert, 28 virtual int Verify(X509Certificate* cert,
29 const std::string& sct_list_from_ocsp, 29 const std::string& stapled_ocsp_response,
30 const std::string& sct_list_from_tls_extension, 30 const std::string& sct_list_from_tls_extension,
31 ct::CTVerifyResult* result, 31 ct::CTVerifyResult* result,
32 const BoundNetLog& net_log) = 0; 32 const BoundNetLog& net_log) = 0;
33 }; 33 };
34 34
35 } // namespace net 35 } // namespace net
36 36
37 #endif // NET_CERT_CT_VERIFIER_H_ 37 #endif // NET_CERT_CT_VERIFIER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698