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

Side by Side Diff: net/quic/quic_crypto_client_stream.cc

Issue 413363005: QUIC - track CHLO's reject reason for secure QUIC vs insecure QUIC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Final_0723
Patch Set: updated histogram_suffix name Created 6 years, 4 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 | « net/quic/crypto/quic_crypto_client_config.cc ('k') | tools/metrics/histograms/histograms.xml » ('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/quic/quic_crypto_client_stream.h" 5 #include "net/quic/quic_crypto_client_stream.h"
6 6
7 #include "net/quic/crypto/crypto_protocol.h" 7 #include "net/quic/crypto/crypto_protocol.h"
8 #include "net/quic/crypto/crypto_utils.h" 8 #include "net/quic/crypto/crypto_utils.h"
9 #include "net/quic/crypto/null_encrypter.h" 9 #include "net/quic/crypto/null_encrypter.h"
10 #include "net/quic/quic_client_session_base.h" 10 #include "net/quic/quic_client_session_base.h"
(...skipping 270 matching lines...) Expand 10 before | Expand all | Expand 10 after
281 // perform a handshake, or we sent a full hello that the server 281 // perform a handshake, or we sent a full hello that the server
282 // rejected. Here we hope to have a REJ that contains the information 282 // rejected. Here we hope to have a REJ that contains the information
283 // that we need. 283 // that we need.
284 if (in->tag() != kREJ) { 284 if (in->tag() != kREJ) {
285 CloseConnectionWithDetails(QUIC_INVALID_CRYPTO_MESSAGE_TYPE, 285 CloseConnectionWithDetails(QUIC_INVALID_CRYPTO_MESSAGE_TYPE,
286 "Expected REJ"); 286 "Expected REJ");
287 return; 287 return;
288 } 288 }
289 error = crypto_config_->ProcessRejection( 289 error = crypto_config_->ProcessRejection(
290 *in, session()->connection()->clock()->WallNow(), cached, 290 *in, session()->connection()->clock()->WallNow(), cached,
291 &crypto_negotiated_params_, &error_details); 291 server_id_.is_https(), &crypto_negotiated_params_, &error_details);
292 if (error != QUIC_NO_ERROR) { 292 if (error != QUIC_NO_ERROR) {
293 CloseConnectionWithDetails(error, error_details); 293 CloseConnectionWithDetails(error, error_details);
294 return; 294 return;
295 } 295 }
296 if (!cached->proof_valid()) { 296 if (!cached->proof_valid()) {
297 if (!server_id_.is_https()) { 297 if (!server_id_.is_https()) {
298 // We don't check the certificates for insecure QUIC connections. 298 // We don't check the certificates for insecure QUIC connections.
299 SetCachedProofValid(cached); 299 SetCachedProofValid(cached);
300 } else if (!cached->signature().empty()) { 300 } else if (!cached->signature().empty()) {
301 next_state_ = STATE_VERIFY_PROOF; 301 next_state_ = STATE_VERIFY_PROOF;
(...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after
500 } 500 }
501 } 501 }
502 return false; 502 return false;
503 } 503 }
504 504
505 QuicClientSessionBase* QuicCryptoClientStream::client_session() { 505 QuicClientSessionBase* QuicCryptoClientStream::client_session() {
506 return reinterpret_cast<QuicClientSessionBase*>(session()); 506 return reinterpret_cast<QuicClientSessionBase*>(session());
507 } 507 }
508 508
509 } // namespace net 509 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/crypto/quic_crypto_client_config.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698