mirror of
https://github.com/monero-project/monero.git
synced 2024-12-25 16:29:23 -05:00
Merge pull request #8899
0961c2a
Set SSL SNI even when server verification is disabled (Lee *!* Clagett)
This commit is contained in:
commit
9537329962
@ -497,6 +497,13 @@ void ssl_options_t::configure(
|
|||||||
const std::string& host) const
|
const std::string& host) const
|
||||||
{
|
{
|
||||||
socket.next_layer().set_option(boost::asio::ip::tcp::no_delay(true));
|
socket.next_layer().set_option(boost::asio::ip::tcp::no_delay(true));
|
||||||
|
{
|
||||||
|
// in case server is doing "virtual" domains, set hostname
|
||||||
|
SSL* const ssl_ctx = socket.native_handle();
|
||||||
|
if (type == boost::asio::ssl::stream_base::client && !host.empty() && ssl_ctx)
|
||||||
|
SSL_set_tlsext_host_name(ssl_ctx, host.c_str());
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
/* Using system-wide CA store for client verification is funky - there is
|
/* Using system-wide CA store for client verification is funky - there is
|
||||||
no expected hostname for server to verify against. If server doesn't have
|
no expected hostname for server to verify against. If server doesn't have
|
||||||
@ -514,10 +521,6 @@ void ssl_options_t::configure(
|
|||||||
{
|
{
|
||||||
socket.set_verify_mode(boost::asio::ssl::verify_peer | boost::asio::ssl::verify_fail_if_no_peer_cert);
|
socket.set_verify_mode(boost::asio::ssl::verify_peer | boost::asio::ssl::verify_fail_if_no_peer_cert);
|
||||||
|
|
||||||
// in case server is doing "virtual" domains, set hostname
|
|
||||||
SSL* const ssl_ctx = socket.native_handle();
|
|
||||||
if (type == boost::asio::ssl::stream_base::client && !host.empty() && ssl_ctx)
|
|
||||||
SSL_set_tlsext_host_name(ssl_ctx, host.c_str());
|
|
||||||
|
|
||||||
socket.set_verify_callback([&](const bool preverified, boost::asio::ssl::verify_context &ctx)
|
socket.set_verify_callback([&](const bool preverified, boost::asio::ssl::verify_context &ctx)
|
||||||
{
|
{
|
||||||
|
Loading…
Reference in New Issue
Block a user