From 8060919dbcda0a872cb3c8da1ff22ee0bd86f7c8 Mon Sep 17 00:00:00 2001
From: Michael Klishin <michaelklishin@icloud.com>
Date: Mon, 29 Sep 2025 15:50:26 -0400
Subject: [PATCH] amqp_ssl: simplify

Now that Erlang 26 is the minimum required version.

(cherry picked from commit d4ce44319ef44f7c776437a883193388ed56779c)
---
 deps/amqp_client/src/amqp_client.erl |  4 ----
 deps/amqp_client/src/amqp_ssl.erl    | 16 ++++------------
 2 files changed, 4 insertions(+), 16 deletions(-)

diff --git a/deps/amqp_client/src/amqp_client.erl b/deps/amqp_client/src/amqp_client.erl
index 4dce3fa71e..6f6306caba 100644
--- a/deps/amqp_client/src/amqp_client.erl
+++ b/deps/amqp_client/src/amqp_client.erl
@@ -18,10 +18,6 @@
 %%---------------------------------------------------------------------------
 
 start() ->
-    %% rabbit_common needs compiler and syntax_tools, see
-    %%
-    %%  * https://github.com/rabbitmq/rabbitmq-erlang-client/issues/72
-    %%  * https://github.com/rabbitmq/rabbitmq-common/pull/149
     {ok, _} = application:ensure_all_started(rabbit_common),
     {ok, _} = application:ensure_all_started(amqp_client),
     ok.
diff --git a/deps/amqp_client/src/amqp_ssl.erl b/deps/amqp_client/src/amqp_ssl.erl
index fc04bdfbfc..fa7c098a58 100644
--- a/deps/amqp_client/src/amqp_ssl.erl
+++ b/deps/amqp_client/src/amqp_ssl.erl
@@ -19,9 +19,10 @@ maybe_enhance_ssl_options(Params) ->
 
 % https://github.com/erlang/otp/blob/master/lib/inets/src/http_client/httpc_handler.erl
 maybe_add_sni(Host, Options) ->
-    maybe_add_sni_0(lists:keyfind(server_name_indication, 1, Options), Host, Options).
+    ServerNameIndicationKeyFound = lists:keyfind(server_name_indication, 1, Options),
+    maybe_add_sni_0(ServerNameIndicationKeyFound, Host, Options).
 
-maybe_add_sni_0(false, Host, Options) ->
+maybe_add_sni_0(false = _ServerNameIndicationKeyFound, Host, Options) ->
     % NB: this is the case where the user did not specify
     % server_name_indication at all. If Host is a DNS host name,
     % we will specify server_name_indication via code
@@ -37,23 +38,14 @@ maybe_add_sni_1(false, _Host, Options) ->
 maybe_add_sni_1(true, Host, Options) ->
     [{server_name_indication, Host} | Options].
 
-%% This check is no longer necessary starting with OTP-26.
-%% @todo Remove the check once we support only OTP-26 and above.
 maybe_add_verify(Options) ->
-    %% This function is only defined starting in OTP-26.
-    case erlang:function_exported(user_drv, whereis_group, 0) of
-        true -> Options;
-        false -> maybe_add_verify1(Options)
-    end.
-
-maybe_add_verify1(Options) ->
     case lists:keymember(verify, 1, Options) of
         true ->
             % NB: user has explicitly set 'verify'
             Options;
         _ ->
             ?LOG_WARNING("Connection (~tp): certificate chain verification is not enabled for this TLS connection. "
-                    "Please see https://rabbitmq.com/ssl.html for more information.", [self()]),
+            "Please see https://rabbitmq.com/ssl.html for more information.", [self()]),
             Options
     end.