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

Unified 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 side-by-side diff with in-line comments
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 »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/certificate_error_reporter.h
diff --git a/chrome/browser/net/certificate_error_reporter.h b/chrome/browser/net/certificate_error_reporter.h
new file mode 100644
index 0000000000000000000000000000000000000000..5463d5a74bd69d6286bbb8cbe1115fc78e8bcd41
--- /dev/null
+++ b/chrome/browser/net/certificate_error_reporter.h
@@ -0,0 +1,87 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_NET_CERTIFICATE_ERROR_REPORTER_H_
+#define CHROME_BROWSER_NET_CERTIFICATE_ERROR_REPORTER_H_
+
+#include <set>
+#include <string>
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "net/url_request/url_request.h"
+#include "url/gurl.h"
+
+namespace net {
+class URLRequestContext;
+class SSLInfo;
+}
+
+namespace chrome_browser_net {
+
+class CertLoggerRequest;
+
+// Provides functionality for sending reports about invalid SSL
+// certificate chains to a report collection server.
+class CertificateErrorReporter : public net::URLRequest::Delegate {
+ public:
+ // These represent the types of reports that can be sent.
+ enum ReportType {
+ // A report of a certificate chain that failed a certificate pinning
+ // check.
+ REPORT_TYPE_PINNING_VIOLATION,
+ // A report for an invalid certificate chain that is being sent for
+ // a user who has opted-in to the extended reporting program.
+ REPORT_TYPE_EXTENDED_REPORTING
+ };
+
+ // Create a certificate error reporter that will send certificate
+ // error reports to |upload_url|, using |request_context| as the
+ // context for the reports.
+ CertificateErrorReporter(net::URLRequestContext* request_context,
+ const GURL& upload_url);
+
+ ~CertificateErrorReporter() override;
+
+ // Construct, serialize, and send a certificate reporter to the report
+ // collection server containing the |ssl_info| associated with a
+ // connection to |hostname|.
+ virtual void SendReport(ReportType type,
+ const std::string& hostname,
+ const net::SSLInfo& ssl_info);
+
+ // net::URLRequest::Delegate
+ void OnResponseStarted(net::URLRequest* request) override;
+ void OnReadCompleted(net::URLRequest* request, int bytes_read) override;
+
+ private:
+ // Create a URLRequest with which to send a certificate report to the
+ // server.
+ virtual scoped_ptr<net::URLRequest> CreateURLRequest(
+ net::URLRequestContext* context);
+
+ // Serialize and send a CertLoggerRequest protobuf to the report
+ // collection server.
+ void SendCertLoggerRequest(const CertLoggerRequest& request);
+
+ // Populate the CertLoggerRequest for a report.
+ static void BuildReport(const std::string& hostname,
+ const net::SSLInfo& ssl_info,
+ CertLoggerRequest* out_request);
+
+ // Performs post-report cleanup.
+ void RequestComplete(net::URLRequest* request);
+
+ net::URLRequestContext* const request_context_;
+ const GURL upload_url_;
+
+ // Owns the contained requests.
+ std::set<net::URLRequest*> inflight_requests_;
+
+ DISALLOW_COPY_AND_ASSIGN(CertificateErrorReporter);
+};
+
+} // namespace chrome_browser_net
+
+#endif // CHROME_BROWSER_NET_CERTIFICATE_ERROR_REPORTER_H_
« 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