fix no proxy hosts for remote and list resolvers

This commit is contained in:
Arthur Passos 2024-06-06 09:48:48 -03:00
parent 946a5913b1
commit 001ac9f847
3 changed files with 4 additions and 4 deletions

View File

@ -55,7 +55,7 @@ struct ProxyConfiguration
uint16_t port = 0;
bool tunneling = false;
Protocol original_request_protocol = Protocol::HTTP;
std::string no_proxy_hosts = "";
std::string no_proxy_hosts = std::string{};
bool isEmpty() const { return host.empty(); }
};

View File

@ -1,6 +1,7 @@
#include <Common/ProxyConfigurationResolverProvider.h>
#include <Common/EnvironmentProxyConfigurationResolver.h>
#include <Common/proxyConfigurationToPocoProxyConfig.h>
#include <Common/Exception.h>
#include <Common/ProxyListConfigurationResolver.h>
#include <Common/RemoteProxyConfigurationResolver.h>
@ -54,7 +55,7 @@ namespace
return std::make_shared<RemoteProxyConfigurationResolver>(
server_configuration,
request_protocol,
getNoProxyHosts(configuration),
buildPocoNonProxyHosts(getNoProxyHosts(configuration)),
std::make_shared<RemoteProxyHostFetcherImpl>(),
isTunnelingDisabledForHTTPSRequestsOverHTTPProxy(configuration));
}
@ -97,7 +98,7 @@ namespace
: std::make_shared<ProxyListConfigurationResolver>(
uris,
request_protocol,
getNoProxyHosts(configuration),
buildPocoNonProxyHosts(getNoProxyHosts(configuration)),
isTunnelingDisabledForHTTPSRequestsOverHTTPProxy(configuration));
}

View File

@ -1,7 +1,6 @@
#include <Common/ProxyListConfigurationResolver.h>
#include <Common/StringUtils.h>
#include <Common/logger_useful.h>
#include <Poco/URI.h>
namespace DB