Skip to content

Commit

Permalink
test_server_mtu_larger_than_max_fragment_length() should be run for D…
Browse files Browse the repository at this point in the history
…TLS 1.3

Previously it was forced to run on DTLS 1.2>. But the underlying issue was fixed on master and it works now that the feature branch has been rebased on top of a more recent master.
  • Loading branch information
fwh-dc committed Jan 28, 2025
1 parent 1ee54c6 commit 4c5e8fb
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions test/dtls_mtu_test.c
Original file line number Diff line number Diff line change
Expand Up @@ -217,13 +217,6 @@ static int test_server_mtu_larger_than_max_fragment_length(void)
NULL, NULL)))
goto end;

/**
* TODO(DTLSv1.3): Test fails with
* SSL routines:tls_parse_ctos_maxfragmentlen:ssl3 ext invalid max fragment length:
* ssl/statem/extensions_srvr.c:202:
*/
OPENSSL_assert(SSL_set_max_proto_version(clnt_ssl, DTLS1_2_VERSION) == 1);

SSL_set_options(srvr_ssl, SSL_OP_NO_QUERY_MTU);
if (!TEST_true(DTLS_set_link_mtu(srvr_ssl, 1500)))
goto end;
Expand Down

0 comments on commit 4c5e8fb

Please sign in to comment.