eric/duckdb-ui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Enable server certs verification by default

Browse Source
This commit is contained in:
Yves
2025-03-12 15:14:58 +01:00
parent 963e0e4d4c
commit 8df6ff3728
1 changed files with 1 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/http_server.cpp
View File

@@ -247,13 +247,7 @@ void HttpServer::HandleGet(const httplib::Request &req,
httplib::Client client(remote_url); httplib::Client client(remote_url);
client.set_keep_alive(true); client.set_keep_alive(true);
// Provide a way to turn on or off server certificate verification, at least if (IsEnvEnabled("ui_disable_server_certificate_verification")) {
// for now, because it requires httplib to correctly get the root certficates
// on each platform, which doesn't appear to always work. Currently, default
// to no verification, until we understand when it breaks things.
if (IsEnvEnabled("ui_enable_server_certificate_verification")) {
client.enable_server_certificate_verification(true);
} else {
client.enable_server_certificate_verification(false); client.enable_server_certificate_verification(false);
} }