New issue
Advanced search Search tips

Issue 635561 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Add integration test to verify bypass when connection to remote site fails

Project Member Reported by robertogden@chromium.org, Aug 8 2016

Issue description

Add an integration to verify that when the remote site times out on an HTTP response, the proxy bypasses and client uses a direct connection instead. 
 
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/deb4af80f00e9455ce2cfcb61ce1fc9400462deb

commit deb4af80f00e9455ce2cfcb61ce1fc9400462deb
Author: robertogden <robertogden@chromium.org>
Date: Tue Aug 09 23:39:28 2016

Added integration test to Chrome Proxy to verify direct connection on remote site timeout.

Added an integration test to Chrome Proxy, testing that when a remote site times out connecting, the proxy will bypass and the client will use a direct connection to the remote site. This should not take longer than 30 seconds.

This CL adds a handler to the chromeproxy-test appspot server to facilitate this test. This handler waits 60 seconds on any request coming from the proxy (causing a timeout) but responds immediately to direct connections.

BUG= 635561 

Review-Url: https://codereview.chromium.org/2206363002
Cr-Commit-Position: refs/heads/master@{#410869}

[modify] https://crrev.com/deb4af80f00e9455ce2cfcb61ce1fc9400462deb/tools/chrome_proxy/integration_tests/chrome_proxy_benchmark.py
[modify] https://crrev.com/deb4af80f00e9455ce2cfcb61ce1fc9400462deb/tools/chrome_proxy/integration_tests/chrome_proxy_measurements.py
[modify] https://crrev.com/deb4af80f00e9455ce2cfcb61ce1fc9400462deb/tools/chrome_proxy/integration_tests/chrome_proxy_metrics.py
[add] https://crrev.com/deb4af80f00e9455ce2cfcb61ce1fc9400462deb/tools/chrome_proxy/integration_tests/chrome_proxy_pagesets/http_timeout.py
[modify] https://crrev.com/deb4af80f00e9455ce2cfcb61ce1fc9400462deb/tools/chrome_proxy/testserver/app.yaml
[modify] https://crrev.com/deb4af80f00e9455ce2cfcb61ce1fc9400462deb/tools/chrome_proxy/testserver/server.go

Status: Verified (was: Started)
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 19 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/clank/internal/apps/+/ad8b6cd6a10b038f5581bda63a3dfa56fae3b2fd

commit ad8b6cd6a10b038f5581bda63a3dfa56fae3b2fd
Author: Robert Ogden <robertogden@chromium.org>
Date: Wed Aug 17 22:03:27 2016

Sign in to add a comment