Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(242)

Unified Diff: jingle/notifier/base/proxy_resolving_client_socket.cc

Issue 9148011: Allow chrome to handle 407 auth challenges to CONNECT requests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | net/base/net_error_list.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: jingle/notifier/base/proxy_resolving_client_socket.cc
===================================================================
--- jingle/notifier/base/proxy_resolving_client_socket.cc (revision 118888)
+++ jingle/notifier/base/proxy_resolving_client_socket.cc (working copy)
@@ -20,6 +20,18 @@
namespace notifier {
+namespace {
+
+void HandleProxyTunnelAuth(const net::HttpResponseInfo& response_info,
+ net::HttpAuthController* auth_controller,
+ net::CompletionCallback callback) {
+ // Since we have no way to respond, simply invoke the callback and the
+ // request will fail.
+ callback.Run(net::OK);
+}
+
+} // namespace
+
ProxyResolvingClientSocket::ProxyResolvingClientSocket(
net::ClientSocketFactory* socket_factory,
const scoped_refptr<net::URLRequestContextGetter>& request_context_getter,
@@ -172,7 +184,9 @@
// Now that we have resolved the proxy, we need to connect.
status = net::InitSocketHandleForRawConnect(
dest_host_port_pair_, network_session_.get(), proxy_info_, ssl_config_,
- ssl_config_, bound_net_log_, transport_.get(), connect_callback_);
+ ssl_config_, bound_net_log_, transport_.get(),
+ base::Bind(&HandleProxyTunnelAuth), connect_callback_);
+
if (status != net::ERR_IO_PENDING) {
// Since this method is always called asynchronously. it is OK to call
// ProcessConnectDone synchronously.
« no previous file with comments | « no previous file | net/base/net_error_list.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698