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

Unified Diff: net/url_request/certificate_report_sender.h

Issue 1212973002: Add net::CertificateReportSender for handling cert report sending (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add missing NET_EXPORT Created 5 years, 5 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 | « net/net.gypi ('k') | net/url_request/certificate_report_sender.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/certificate_report_sender.h
diff --git a/net/url_request/certificate_report_sender.h b/net/url_request/certificate_report_sender.h
new file mode 100644
index 0000000000000000000000000000000000000000..9073836c0cf6694d74b81d738bc96277893c9fed
--- /dev/null
+++ b/net/url_request/certificate_report_sender.h
@@ -0,0 +1,76 @@
+// 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 NET_URL_REQUEST_CERTIFICATE_REPORT_SENDER_H_
+#define NET_URL_REQUEST_CERTIFICATE_REPORT_SENDER_H_
+
+#include <set>
+#include <string>
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "net/http/transport_security_state.h"
+#include "net/url_request/url_request.h"
+
+class GURL;
+
+namespace net {
+
+class URLRequestContext;
+
+// CertificateReportSender asynchronously sends serialized certificate
+// reports to a URI. It takes serialized reports as a sequence of bytes
+// so as to be agnostic to the format of the report being sent (JSON,
+// protobuf, etc.) and the particular data that it contains. Multiple
+// reports can be in-flight at once. This class owns inflight requests
+// and cleans them up when necessary.
+class NET_EXPORT CertificateReportSender
+ : public URLRequest::Delegate,
+ public TransportSecurityState::ReportSender {
+ public:
+ // Represents whether or not to send cookies along with reports.
+ enum CookiesPreference { SEND_COOKIES, DO_NOT_SEND_COOKIES };
+
+ // Constructs a CertificateReportSender that sends reports with the
+ // given |request_context| and includes or excludes cookies based on
+ // |cookies_preference|. Ownership of |request_context| is not
+ // transferred, so it must outlive the CertificateReportSender.
+ CertificateReportSender(URLRequestContext* request_context,
+ CookiesPreference cookies_preference);
+
+ ~CertificateReportSender() override;
+
+ // TransportSecurityState::ReportSender
+ void Send(const GURL& report_uri, const std::string& report) override;
+
+ // net::URLRequest::Delegate
+ void OnResponseStarted(URLRequest* request) override;
+ void OnReadCompleted(URLRequest* request, int bytes_read) override;
+
+ private:
+ // Creates a URLRequest with which to send a certificate report to the
+ // server.
+ //
+ // TODO(estark): inline this into Send() once
+ // ChromeFraudulentCertificateReporter goes away.
+ virtual scoped_ptr<URLRequest> CreateURLRequest(
+ net::URLRequestContext* context,
+ const GURL& report_uri);
+
+ // Performs post-report cleanup.
+ void RequestComplete(net::URLRequest* request);
+
+ net::URLRequestContext* const request_context_;
+
+ CookiesPreference cookies_preference_;
+
+ // Owns the contained requests.
+ std::set<URLRequest*> inflight_requests_;
+
+ DISALLOW_COPY_AND_ASSIGN(CertificateReportSender);
+};
+
+} // namespace net
+
+#endif // NET_URL_REQUEST_CERTIFICATE_REPORT_H_
« no previous file with comments | « net/net.gypi ('k') | net/url_request/certificate_report_sender.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698