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

Unified Diff: components/cronet/ios/cronet_environment.cc

Issue 2493863002: [Cronet] Ignore Certificate Transparency to avoid CT log expiration. (Closed)
Patch Set: Use IgnoresCTVerifier / PolicyEnforcer on iOS. Created 4 years, 1 month 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 | « components/cronet/ios/BUILD.gn ('k') | net/url_request/url_request_context_builder.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/ios/cronet_environment.cc
diff --git a/components/cronet/ios/cronet_environment.cc b/components/cronet/ios/cronet_environment.cc
index 0fef4af2e2266ab840e1b76a457f213ba763f5a6..18fd1a0ece00819c3afca74b2dd693ab9d9e5e8b 100644
--- a/components/cronet/ios/cronet_environment.cc
+++ b/components/cronet/ios/cronet_environment.cc
@@ -23,6 +23,7 @@
#include "base/single_thread_task_runner.h"
#include "base/synchronization/waitable_event.h"
#include "base/threading/worker_pool.h"
+#include "components/cronet/ct_ignores.h"
#include "components/cronet/histogram_manager.h"
#include "components/cronet/ios/version.h"
#include "components/prefs/json_pref_store.h"
@@ -31,11 +32,6 @@
#include "net/base/net_errors.h"
#include "net/base/network_change_notifier.h"
#include "net/cert/cert_verifier.h"
-#include "net/cert/ct_known_logs.h"
-#include "net/cert/ct_log_verifier.h"
-#include "net/cert/ct_policy_enforcer.h"
-#include "net/cert/ct_verifier.h"
-#include "net/cert/multi_log_ct_verifier.h"
#include "net/dns/host_resolver.h"
#include "net/dns/mapped_host_resolver.h"
#include "net/http/http_auth_handler_factory.h"
@@ -303,12 +299,8 @@ void CronetEnvironment::InitializeOnNetworkThread() {
if (!cert_verifier_)
cert_verifier_ = net::CertVerifier::CreateDefault();
main_context_->set_cert_verifier(cert_verifier_.get());
-
- std::unique_ptr<net::MultiLogCTVerifier> ct_verifier =
- base::MakeUnique<net::MultiLogCTVerifier>();
- ct_verifier->AddLogs(net::ct::CreateLogVerifiersForKnownLogs());
- main_context_->set_cert_transparency_verifier(ct_verifier.release());
- main_context_->set_ct_policy_enforcer(new net::CTPolicyEnforcer());
+ main_context_->set_cert_transparency_verifier(new IgnoresCTVerifier());
mef 2016/11/10 20:55:43 The iOS implementation is being reworked to use Ur
+ main_context_->set_ct_policy_enforcer(new IgnoresCTPolicyEnforcer());
main_context_->set_http_auth_handler_factory(
net::HttpAuthHandlerRegistryFactory::CreateDefault(
« no previous file with comments | « components/cronet/ios/BUILD.gn ('k') | net/url_request/url_request_context_builder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698