Commit a0595201 authored by Erik Jensen's avatar Erik Jensen Committed by Commit Bot

Temporarily increase signaling timeouts.

This is for signaling-timeout troubleshooting. Once rolled out, we will
see if telemetry reports a significant number of connections that
succeed but would have previously timed out, which will help narrow down
the cause.

Bug: 838711
Change-Id: I28a62ebfd46b9e896f6cba6a2c6ba4694205bd30
Reviewed-on: https://chromium-review.googlesource.com/1033626
Commit-Queue: Erik Jensen <rkjnsn@chromium.org>
Reviewed-by: default avatarJamie Walch <jamiewalch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555200}
parent 409f281b
......@@ -37,15 +37,20 @@ namespace protocol {
namespace {
// Timeouts have been temporarily increased for testing.
// TODO(rkjnsn): Revert default and session timeouts once done with testing.
// How long we should wait for a response from the other end. This value is used
// for all requests except |transport-info|.
const int kDefaultMessageTimeout = 10;
// const int kDefaultMessageTimeout = 10;
const int kDefaultMessageTimeout = 35; // For testing
// During a reconnection, it usually takes longer for the peer to respond due to
// pending messages in the channel from the previous session. From experiment,
// it can take up to 20s for the session to reconnect. To make it safe, setting
// the timeout to 30s.
const int kSessionInitiateAndAcceptTimeout = kDefaultMessageTimeout * 3;
// const int kSessionInitiateAndAcceptTimeout = kDefaultMessageTimeout * 3;
const int kSessionInitiateAndAcceptTimeout = 45; // For testing
// Timeout for the transport-info messages.
const int kTransportInfoTimeout = 10 * 60;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment