Remove obsolete function

Author: MarcialRosales

deps/rabbitmq_auth_backend_oauth2/src/uaa_jwks.erl

@@ -1,31 +1,6 @@
 -module(uaa_jwks).
--export([get/2, ssl_options/1]).
+-export([get/2]).
 
 -spec get(string() | binary(), term()) -> {ok, term()} | {error, term()}.
 get(JwksUrl, KeyConfig) ->
     httpc:request(get, {JwksUrl, []}, [{ssl, ssl_options(KeyConfig)}, {timeout, 60000}], []).
-
--spec ssl_options(term()) -> list().
-ssl_options(KeyConfig) ->
-    PeerVerification = proplists:get_value(peer_verification, KeyConfig, verify_none),
-    Depth = proplists:get_value(depth, KeyConfig, 10),
-    FailIfNoPeerCert = proplists:get_value(fail_if_no_peer_cert, KeyConfig, false),
-    CrlCheck = proplists:get_value(crl_check, KeyConfig, false),
-    SslOpts0 = [{verify, PeerVerification},
-                {depth, Depth},
-                {fail_if_no_peer_cert, FailIfNoPeerCert},
-                {crl_check, CrlCheck},
-                {crl_cache, {ssl_crl_cache, {internal, [{http, 10000}]}}} | cacertfile(KeyConfig)],
-
-    case proplists:get_value(hostname_verification, KeyConfig, none) of
-        wildcard ->
-            [{customize_hostname_check, [{match_fun, public_key:pkix_verify_hostname_match_fun(https)}]} | SslOpts0];
-        none ->
-            SslOpts0
-    end.
-
-cacertfile(KeyConfig) ->
-    case proplists:get_value(cacertfile, KeyConfig) of
-        undefined -> [];
-        CaCertFile -> [{cacertfile, CaCertFile}]
-    end.

deps/rabbitmq_auth_backend_oauth2/test/unit_SUITE.erl

@@ -37,8 +37,6 @@ all() ->
         test_post_process_payload_rich_auth_request_using_regular_expression_with_cluster,
         test_unsuccessful_access_with_a_token_that_uses_missing_scope_alias_in_scope_field,
         test_unsuccessful_access_with_a_token_that_uses_missing_scope_alias_in_extra_scope_source_field,
-        test_default_ssl_options,
-        test_default_ssl_options_with_cacertfile,
         test_username_from,
         {group, with_rabbitmq_node}
     ].
@@ -125,10 +123,6 @@ init_per_testcase(test_post_process_payload_rich_auth_request_using_regular_expr
   application:set_env(rabbitmq_auth_backend_oauth2, resource_server_id, <<"rabbitmq-test">>),
   Config;
 
-init_per_testcase(test_default_ssl_options_with_cacertfile, Config) ->
-  application:set_env(rabbitmq_auth_backend_oauth2, key_config, [{ cacertfile, filename:join(["testca", "cacert.pem"]) }] ),
-  Config;
-
 init_per_testcase(_, Config) ->
   Config.
 
@@ -137,10 +131,6 @@ end_per_testcase(test_post_process_token_payload_complex_claims, Config) ->
   application:set_env(rabbitmq_auth_backend_oauth2, resource_server_id, undefined),
   Config;
 
-end_per_testcase(test_default_ssl_options_with_cacertfile, Config) ->
-  application:set_env(rabbitmq_auth_backend_oauth2, key_config, undefined),
-  Config;
-
 end_per_testcase(_, Config) ->
   Config.
 
@@ -1331,25 +1321,6 @@ test_validate_payload_when_verify_aud_false(_) ->
                         <<"scope">> => [<<"bar">>, <<"other.third">>]}},
                  rabbit_auth_backend_oauth2:validate_payload(?RESOURCE_SERVER_ID, WithAudWithUnknownResourceId, ?DEFAULT_SCOPE_PREFIX)).
 
-test_default_ssl_options(_) ->
-  ?assertEqual([
-              {verify, verify_none},
-              {depth, 10},
-              {fail_if_no_peer_cert, false},
-              {crl_check, false},
-              {crl_cache, {ssl_crl_cache, {internal, [{http, 10000}]}}}
-              ], uaa_jwks:ssl_options(rabbit_oauth2_config:get_key_config())).
-
-test_default_ssl_options_with_cacertfile(_) ->
-  ?assertEqual([
-              {verify, verify_none},
-              {depth, 10},
-              {fail_if_no_peer_cert, false},
-              {crl_check, false},
-              {crl_cache, {ssl_crl_cache, {internal, [{http, 10000}]}}},
-              {cacertfile, filename:join(["testca", "cacert.pem"])}
-              ], uaa_jwks:ssl_options(rabbit_oauth2_config:get_key_config())).
-
 %%
 %% Helpers
 %%