Bypass the DataReductionProxy for all POST requests

net/url_request/url_request_ftp_job.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/url_request/url_request_ftp_job.h"
 
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 83 matching lines @@
   DCHECK(!pac_request_);
   DCHECK(!ftp_transaction_);
   DCHECK(!http_transaction_);
 
   int rv = OK;
   if (request_->load_flags() & LOAD_BYPASS_PROXY) {
     proxy_info_.UseDirect();
   } else {
     DCHECK_EQ(request_->context()->proxy_service(), proxy_service_);
     rv = proxy_service_->ResolveProxy(
-        request_->url(),
-        request_->load_flags(),
-        &proxy_info_,
-        base::Bind(&URLRequestFtpJob::OnResolveProxyComplete,
-                   base::Unretained(this)),
-        &pac_request_,
-        NULL,
-        request_->net_log());
+        request_->url(), request_->method(), request_->load_flags(),

[mmenke, 2016/03/02 22:53:45]

Methods are meaningless for FTP requests.  Should just send a GET, since that's
the only method we use support for proxied FTP requests.

[RyanSturm, 2016/03/03 18:54:31]

Done.

+        &proxy_info_, base::Bind(&URLRequestFtpJob::OnResolveProxyComplete,
+                                 base::Unretained(this)),
+        &pac_request_, NULL, request_->net_log());
 
     if (rv == ERR_IO_PENDING)
       return;
   }
   OnResolveProxyComplete(rv);
 }
 
 void URLRequestFtpJob::Kill() {
   if (pac_request_) {
     proxy_service_->CancelPacRequest(pac_request_);
@@ skipping 254 matching lines @@
   if (cached_auth) {
     // Retry using cached auth data.
     SetAuth(cached_auth->credentials);
   } else {
     // Prompt for a username/password.
     NotifyHeadersComplete();
   }
 }
 
 }  // namespace net