mirror of
https://github.com/klzgrad/naiveproxy.git
synced 2024-11-24 22:36:09 +03:00
socket: Force tunneling for all sockets
In the socket system, only WebSocket sockets are allowed to tunnel through HTTP/1 proxies. "Raw" sockets in the normal socket pool don't have it, and their CONNECT headers are not sent, instead the raw payload is sent as-is to the HTTP/1 proxy, breaking the proxying. The socket system works like this: - HTTP sockets via HTTP/1 proxies: normal pool, no tunneling. - HTTPS sockets via HTTP/1 proxies: normal pool, no tunneling, but does its own proxy encapsulation. - WS sockets via HTTP/1 proxies: WS pool, tunneling. In Naive, we need the normal pool because the WS pool has some extra restrictions but we also need tunneling to produce a client socket with proxy tunneling built in. Therefore force tunneling for all sockets and have them always send CONNECT headers. This will otherwise break regular HTTP client sockets via HTTP/1 proxies, but as we don't use this combination, it is ok.
This commit is contained in:
parent
5c3e8e6d1d
commit
62af6f55bb
@ -82,8 +82,7 @@ ClientSocketPool* ClientSocketPoolManagerImpl::GetSocketPool(
|
|||||||
new_pool = std::make_unique<TransportClientSocketPool>(
|
new_pool = std::make_unique<TransportClientSocketPool>(
|
||||||
sockets_per_proxy_server, sockets_per_group,
|
sockets_per_proxy_server, sockets_per_group,
|
||||||
unused_idle_socket_timeout(pool_type_), proxy_server,
|
unused_idle_socket_timeout(pool_type_), proxy_server,
|
||||||
pool_type_ == HttpNetworkSession::WEBSOCKET_SOCKET_POOL,
|
/*force_tunnel=*/true, &common_connect_job_params_);
|
||||||
&common_connect_job_params_);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
std::pair<SocketPoolMap::iterator, bool> ret =
|
std::pair<SocketPoolMap::iterator, bool> ret =
|
||||||
|
Loading…
Reference in New Issue
Block a user