Chromium Code Reviews| Index: components/data_reduction_proxy/common/data_reduction_proxy_headers.cc |
| diff --git a/components/data_reduction_proxy/common/data_reduction_proxy_headers.cc b/components/data_reduction_proxy/common/data_reduction_proxy_headers.cc |
| index c7775c81216f46579e772a3b56357031133cef20..ebc922b7afd8b54ccac2208e1336a7ac030b900e 100644 |
| --- a/components/data_reduction_proxy/common/data_reduction_proxy_headers.cc |
| +++ b/components/data_reduction_proxy/common/data_reduction_proxy_headers.cc |
| @@ -121,6 +121,10 @@ GetDataReductionProxyBypassEventType( |
| // Fall back if a 500, 502 or 503 is returned. |
| return ProxyService::INTERNAL_SERVER_ERROR_BYPASS; |
| } |
| + if (headers->response_code() == net::HTTP_PROXY_AUTHENTICATION_REQUIRED && |
|
bengr
2014/06/05 20:37:22
Add a TODO that we should also report that the res
Not at Google. Contact bengr
2014/06/05 20:43:43
Done.
|
| + !headers->HasHeader("Proxy-Authenticate")) { |
| + return ProxyService::MALFORMED_407_BYPASS; |
| + } |
| if (!IsDataReductionProxyResponse(headers) && |
| (headers->response_code() != net::HTTP_NOT_MODIFIED)) { |
| // A Via header might not be present in a 304. Since the goal of a 304 |