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

Side by Side Diff: net/base/transport_security_state.cc

Issue 9602009: net: don't set HSTS for chart.apis.google.com (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | net/base/transport_security_state_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "net/base/transport_security_state.h" 5 #include "net/base/transport_security_state.h"
6 6
7 #if defined(USE_OPENSSL) 7 #if defined(USE_OPENSSL)
8 #include <openssl/ecdsa.h> 8 #include <openssl/ecdsa.h>
9 #include <openssl/ssl.h> 9 #include <openssl/ssl.h>
10 #else // !defined(USE_OPENSSL) 10 #else // !defined(USE_OPENSSL)
(...skipping 1313 matching lines...) Expand 10 before | Expand all | Expand 10 after
1324 {26, true, "\003ssl\020google-analytics\003com", true, 1324 {26, true, "\003ssl\020google-analytics\003com", true,
1325 kGooglePins, DOMAIN_GOOGLE_ANALYTICS_COM }, 1325 kGooglePins, DOMAIN_GOOGLE_ANALYTICS_COM },
1326 {18, true, "\005drive\006google\003com", true, kGooglePins, 1326 {18, true, "\005drive\006google\003com", true, kGooglePins,
1327 DOMAIN_GOOGLE_COM }, 1327 DOMAIN_GOOGLE_COM },
1328 {16, true, "\012googleplex\003com", true, kGooglePins, 1328 {16, true, "\012googleplex\003com", true, kGooglePins,
1329 DOMAIN_GOOGLEPLEX_COM }, 1329 DOMAIN_GOOGLEPLEX_COM },
1330 {19, true, "\006groups\006google\003com", true, kGooglePins, 1330 {19, true, "\006groups\006google\003com", true, kGooglePins,
1331 DOMAIN_GOOGLE_COM }, 1331 DOMAIN_GOOGLE_COM },
1332 {17, true, "\004apis\006google\003com", true, kGooglePins, 1332 {17, true, "\004apis\006google\003com", true, kGooglePins,
1333 DOMAIN_GOOGLE_COM }, 1333 DOMAIN_GOOGLE_COM },
1334 // chart.apis.google.com is *not* HSTS because the certificate doesn't match
1335 // and there are lots of links out there that still use the name. The correct
1336 // hostname for this is chart.googleapis.com.
1337 {23, true, "\005chart\004apis\006google\003com", false, kGooglePins,
1338 DOMAIN_GOOGLE_COM},
1339
1334 // Other Google-related domains that must use an acceptable certificate 1340 // Other Google-related domains that must use an acceptable certificate
1335 // iff using SSL. 1341 // iff using SSL.
1336 {11, true, "\005ytimg\003com", false, kGooglePins, 1342 {11, true, "\005ytimg\003com", false, kGooglePins,
1337 DOMAIN_YTIMG_COM }, 1343 DOMAIN_YTIMG_COM },
1338 {23, true, "\021googleusercontent\003com", false, kGooglePins, 1344 {23, true, "\021googleusercontent\003com", false, kGooglePins,
1339 DOMAIN_GOOGLEUSERCONTENT_COM }, 1345 DOMAIN_GOOGLEUSERCONTENT_COM },
1340 {13, true, "\007youtube\003com", false, kGooglePins, 1346 {13, true, "\007youtube\003com", false, kGooglePins,
1341 DOMAIN_YOUTUBE_COM }, 1347 DOMAIN_YOUTUBE_COM },
1342 {16, true, "\012googleapis\003com", false, kGooglePins, 1348 {16, true, "\012googleapis\003com", false, kGooglePins,
1343 DOMAIN_GOOGLEAPIS_COM }, 1349 DOMAIN_GOOGLEAPIS_COM },
(...skipping 290 matching lines...) Expand 10 before | Expand all | Expand 10 after
1634 1640
1635 return true; 1641 return true;
1636 } 1642 }
1637 1643
1638 bool TransportSecurityState::DomainState::ShouldRedirectHTTPToHTTPS() 1644 bool TransportSecurityState::DomainState::ShouldRedirectHTTPToHTTPS()
1639 const { 1645 const {
1640 return mode == MODE_STRICT; 1646 return mode == MODE_STRICT;
1641 } 1647 }
1642 1648
1643 } // namespace 1649 } // namespace
OLDNEW
« no previous file with comments | « no previous file | net/base/transport_security_state_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698