Skip to content

Commit

Permalink
fix(tests): re-enable and update SSL verification tests in katipo_SUI…
Browse files Browse the repository at this point in the history
…TE.erl

Fixes #132

I'm not sure why they were failing...
  • Loading branch information
puzza007 committed Jan 25, 2025
1 parent 6af4b76 commit df8cafb
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions test/katipo_SUITE.erl
Original file line number Diff line number Diff line change
Expand Up @@ -142,10 +142,8 @@ groups() ->
max_pipeline_length]},
{https, [parallel],
[verify_host_verify_peer_ok,
%% TODO :Fix this test. See https://github.com/puzza007/katipo/runs/5281801454?check_suite_focus=true
%% verify_host_verify_peer_error,
%% TODO: Fix this test. See https://github.com/puzza007/katipo/runs/5281750037?check_suite_focus=true
%% cacert_self_signed,
verify_host_verify_peer_error,
cacert_self_signed,
badssl]},
{https_mutual, [],
[badssl_client_cert]},
Expand Down Expand Up @@ -699,7 +697,7 @@ cacert_self_signed(Config) ->
CACert = ?config(cacert_file, Config),
{ok, #{status := 200}} =
katipo:get(?POOL, <<"https://localhost:8443">>,
#{verbose => true, ssl_verifyhost => true, ssl_verifypeer => true, cacert => CACert}).
#{ssl_verifyhost => true, ssl_verifypeer => true, cacert => CACert}).

badssl(_) ->
{error, _} =
Expand Down

0 comments on commit df8cafb

Please sign in to comment.