-
Notifications
You must be signed in to change notification settings - Fork 17
Closed
Description
This library does not appear to check whether certificates are valid, as seen here:
- The default is
false:bool verify_ssl_cert = false; - The default is always used: and
ai-sdk-cpp/src/providers/base_provider_client.cpp
Lines 17 to 19 in 6e8ec01
// Initialize HTTP handler with parsed config auto http_config = http::HttpRequestHandler::parse_base_url(config.base_url); http_handler_ = std::make_unique<http::HttpRequestHandler>(http_config); HttpConfig HttpRequestHandler::parse_base_url(const std::string& base_url) { - That value is actually applied:
cli.enable_server_certificate_verification(config_.verify_ssl_cert);
This is a significant security vulnerability!
Mitigation
I believe you just need to set that boolean to true.
Metadata
Metadata
Assignees
Labels
No labels