Skip to content

Commit

Permalink
Fix test runs on builds without tls1_3
Browse files Browse the repository at this point in the history
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #23319)
  • Loading branch information
fwh-dc authored and mattcaswell committed Feb 9, 2024
1 parent 5f7694c commit a909113
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions test/recipes/70-test_sslrecords.t
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,12 @@ my $proxy = TLSProxy::Proxy->new(
(!$ENV{HARNESS_ACTIVE} || $ENV{HARNESS_VERBOSE})
);

# Avoid failures with tls1_3 disabled builds
# TLSProxy defaults to use tls1_3 and tls1_2 is required by the tests so
# set it here and check that a simple proxy works before running the tests
$proxy->serverflags("-tls1_2");
$proxy->clientflags("-no_tls1_3");

$proxy->start() or plan skip_all => "Unable to start up Proxy for tests";
plan tests => 42;

Expand Down Expand Up @@ -79,6 +85,7 @@ sub run_tests
SKIP: {
skip "Record tests not intended for dtls", 1 if $run_test_as_dtls == 1;
#Test 1: Injecting out of context empty records should fail
$proxy->clear();
$content_type = TLSProxy::Record::RT_APPLICATION_DATA;
$inject_recs_num = 1;
$fatal_alert = 0;
Expand Down

0 comments on commit a909113

Please sign in to comment.