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

Side by Side Diff: chrome/browser/net/certificate_error_reporter.h

Issue 979893003: Refactor ChromeFraudulentCertReporter for code reuse by SSL reporting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use URLRequestMockDataJob to avoid IO in unit test Created 5 years, 9 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
« no previous file with comments | « chrome/browser/net/cert_logger.proto ('k') | chrome/browser/net/certificate_error_reporter.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_NET_CERTIFICATE_ERROR_REPORTER_H_
6 #define CHROME_BROWSER_NET_CERTIFICATE_ERROR_REPORTER_H_
7
8 #include <set>
9 #include <string>
10
11 #include "base/macros.h"
12 #include "base/memory/scoped_ptr.h"
13 #include "net/url_request/url_request.h"
14 #include "url/gurl.h"
15
16 namespace net {
17 class URLRequestContext;
18 class SSLInfo;
19 }
20
21 namespace chrome_browser_net {
22
23 class CertLoggerRequest;
24
25 // Provides functionality for sending reports about invalid SSL
26 // certificate chains to a report collection server.
27 class CertificateErrorReporter : public net::URLRequest::Delegate {
28 public:
29 // These represent the types of reports that can be sent.
30 enum ReportType {
31 // A report of a certificate chain that failed a certificate pinning
32 // check.
33 REPORT_TYPE_PINNING_VIOLATION,
34 // A report for an invalid certificate chain that is being sent for
35 // a user who has opted-in to the extended reporting program.
36 REPORT_TYPE_EXTENDED_REPORTING
37 };
38
39 // Create a certificate error reporter that will send certificate
40 // error reports to |upload_url|, using |request_context| as the
41 // context for the reports.
42 CertificateErrorReporter(net::URLRequestContext* request_context,
43 const GURL& upload_url);
44
45 ~CertificateErrorReporter() override;
46
47 // Construct, serialize, and send a certificate reporter to the report
48 // collection server containing the |ssl_info| associated with a
49 // connection to |hostname|.
50 virtual void SendReport(ReportType type,
51 const std::string& hostname,
52 const net::SSLInfo& ssl_info);
53
54 // net::URLRequest::Delegate
55 void OnResponseStarted(net::URLRequest* request) override;
56 void OnReadCompleted(net::URLRequest* request, int bytes_read) override;
57
58 private:
59 // Create a URLRequest with which to send a certificate report to the
60 // server.
61 virtual scoped_ptr<net::URLRequest> CreateURLRequest(
62 net::URLRequestContext* context);
63
64 // Serialize and send a CertLoggerRequest protobuf to the report
65 // collection server.
66 void SendCertLoggerRequest(const CertLoggerRequest& request);
67
68 // Populate the CertLoggerRequest for a report.
69 static void BuildReport(const std::string& hostname,
70 const net::SSLInfo& ssl_info,
71 CertLoggerRequest* out_request);
72
73 // Performs post-report cleanup.
74 void RequestComplete(net::URLRequest* request);
75
76 net::URLRequestContext* const request_context_;
77 const GURL upload_url_;
78
79 // Owns the contained requests.
80 std::set<net::URLRequest*> inflight_requests_;
81
82 DISALLOW_COPY_AND_ASSIGN(CertificateErrorReporter);
83 };
84
85 } // namespace chrome_browser_net
86
87 #endif // CHROME_BROWSER_NET_CERTIFICATE_ERROR_REPORTER_H_
OLDNEW
« no previous file with comments | « chrome/browser/net/cert_logger.proto ('k') | chrome/browser/net/certificate_error_reporter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698