RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-05-31 13:38:11 +00:00
Code Issues Activity Actions

RequestServer: Ignore preconnect requests for available connections

Browse source 
There's no need to schedule a useless job when the connection is already
there and established.
This commit is contained in:
Ali Mohammad Pur 2021-10-01 15:07:09 +03:30 • committed by Andreas Kling
parent 4974727dbb
commit a3fe981daa
1 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
Userland/Services/RequestServer/ClientConnection.cpp
View file

@ -138,8 +138,7 @@ void ClientConnection::ensure_connection(URL const& url, ::RequestServer::CacheL
if (!is_tls && socket->is_connected())
is_connected = true;
if (is_connected)
return ConnectionCache::request_did_finish(m_url, socket);
VERIFY(!is_connected);
bool did_connect;
if (is_tls) {
@ -167,7 +166,9 @@ void ClientConnection::ensure_connection(URL const& url, ::RequestServer::CacheL
dbgln("EnsureConnection: Pre-connect to {}", url);
auto do_preconnect = [&](auto& cache) {
ConnectionCache::get_or_create_connection(cache, url, job);
auto it = cache.find({ url.host(), url.port_or_default() });
if (it == cache.end() || it->value->is_empty())
ConnectionCache::get_or_create_connection(cache, url, job);
};
if (url.scheme() == "http"sv)