Commit 5b1b6fbf authored by Eric Roman's avatar Eric Roman Committed by Commit Bot

Remove the explicit bypass rule for 127.0.0.0/8 used by DRP.

It is no longer necessary as ProxyBypassRules now implicitly bypasses loopback addresses.

Change-Id: Ia6c480870458e63a70e6ba753137f9169d58ba9e
Reviewed-on: https://chromium-review.googlesource.com/c/1327143Reviewed-by: default avatarTarun Bansal <tbansal@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606610}
parent 04baa857
...@@ -657,14 +657,14 @@ void DataReductionProxyConfig::ContinueNetworkChanged( ...@@ -657,14 +657,14 @@ void DataReductionProxyConfig::ContinueNetworkChanged(
void DataReductionProxyConfig::AddDefaultProxyBypassRules() { void DataReductionProxyConfig::AddDefaultProxyBypassRules() {
DCHECK(configurator_); DCHECK(configurator_);
// Under the hood we use an instance of ProxyBypassRules to evaluate these
// rules. ProxyBypassRules implicitly bypasses localhost, loopback, and
// link-local addresses, so it is not necessary to explicitly add them here.
// See ProxyBypassRules::MatchesImplicitRules() for details.
configurator_->SetBypassRules( configurator_->SetBypassRules(
// localhost // localhost
"<local>," "<local>,"
// RFC6890 loopback addresses.
// TODO(tbansal): Remove this once crbug/446705 is fixed.
"127.0.0.0/8,"
// RFC6890 current network (only valid as source address). // RFC6890 current network (only valid as source address).
"0.0.0.0/8," "0.0.0.0/8,"
......
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