Skip to content

Commit 700cbac

Browse files
committed
Merge remote-tracking branch 'public/pr/2448' into development
* public/pr/2448: Reword changelog entry Update change log Reenable GnuTLS next based tests
2 parents 86016a0 + cddbd01 commit 700cbac

File tree

2 files changed

+10
-21
lines changed

2 files changed

+10
-21
lines changed

ChangeLog

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ Changes
5353
underlying OS actually guarantees.
5454
* Fix configuration queries in ssl-opt.h. #2030
5555
* Ensure that ssl-opt.h can be run in OS X. #2029
56+
* Re-enable certain interoperability tests in ssl-opt.sh which had previously
57+
been disabled for lack of a sufficiently recent version of GnuTLS on the CI.
5658

5759
= mbed TLS 2.16.0 branch released 2018-12-21
5860

tests/ssl-opt.sh

Lines changed: 8 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -7061,13 +7061,7 @@ run_test "DTLS fragmenting: 3d, gnutls server, DTLS 1.0" \
70617061
-c "fragmenting handshake message" \
70627062
-C "error"
70637063

7064-
## The two tests below are disabled due to a bug in GnuTLS client that causes
7065-
## handshake failures when the NewSessionTicket message is lost, see
7066-
## https://gitlab.com/gnutls/gnutls/issues/543
7067-
## We can re-enable them when a fixed version fo GnuTLS is available
7068-
## and installed in our CI system.
7069-
skip_next_test
7070-
requires_gnutls
7064+
requires_gnutls_next
70717065
requires_config_enabled MBEDTLS_SSL_PROTO_DTLS
70727066
requires_config_enabled MBEDTLS_RSA_C
70737067
requires_config_enabled MBEDTLS_ECDSA_C
@@ -7079,12 +7073,11 @@ run_test "DTLS fragmenting: 3d, gnutls client, DTLS 1.2" \
70797073
crt_file=data_files/server7_int-ca.crt \
70807074
key_file=data_files/server7.key \
70817075
hs_timeout=250-60000 mtu=512 force_version=dtls1_2" \
7082-
"$G_CLI -u --insecure 127.0.0.1" \
7076+
"$G_NEXT_CLI -u --insecure 127.0.0.1" \
70837077
0 \
70847078
-s "fragmenting handshake message"
70857079

7086-
skip_next_test
7087-
requires_gnutls
7080+
requires_gnutls_next
70887081
requires_config_enabled MBEDTLS_SSL_PROTO_DTLS
70897082
requires_config_enabled MBEDTLS_RSA_C
70907083
requires_config_enabled MBEDTLS_ECDSA_C
@@ -7096,7 +7089,7 @@ run_test "DTLS fragmenting: 3d, gnutls client, DTLS 1.0" \
70967089
crt_file=data_files/server7_int-ca.crt \
70977090
key_file=data_files/server7.key \
70987091
hs_timeout=250-60000 mtu=512 force_version=dtls1" \
7099-
"$G_CLI -u --insecure 127.0.0.1" \
7092+
"$G_NEXT_CLI -u --insecure 127.0.0.1" \
71007093
0 \
71017094
-s "fragmenting handshake message"
71027095

@@ -7666,29 +7659,23 @@ run_test "DTLS proxy: 3d, gnutls server" \
76667659
-s "Extra-header:" \
76677660
-c "Extra-header:"
76687661

7669-
# The next two test are disabled because they tend to trigger a bug in the
7670-
# version of GnuTLS that's currently installed on our CI. The bug occurs when
7671-
# different fragments of the same handshake message are received out-of-order
7672-
# by GnuTLS and results in a timeout. It's been fixed in GnuTLS 3.5.2.
7673-
skip_next_test
7674-
requires_gnutls
7662+
requires_gnutls_next
76757663
client_needs_more_time 8
76767664
not_with_valgrind # risk of non-mbedtls peer timing out
76777665
run_test "DTLS proxy: 3d, gnutls server, fragmentation" \
76787666
-p "$P_PXY drop=5 delay=5 duplicate=5" \
7679-
"$G_SRV -u --mtu 512" \
7667+
"$G_NEXT_SRV -u --mtu 512" \
76807668
"$P_CLI dgram_packing=0 dtls=1 hs_timeout=500-60000" \
76817669
0 \
76827670
-s "Extra-header:" \
76837671
-c "Extra-header:"
76847672

7685-
skip_next_test
7686-
requires_gnutls
7673+
requires_gnutls_next
76877674
client_needs_more_time 8
76887675
not_with_valgrind # risk of non-mbedtls peer timing out
76897676
run_test "DTLS proxy: 3d, gnutls server, fragmentation, nbio" \
76907677
-p "$P_PXY drop=5 delay=5 duplicate=5" \
7691-
"$G_SRV -u --mtu 512" \
7678+
"$G_NEXT_SRV -u --mtu 512" \
76927679
"$P_CLI dgram_packing=0 dtls=1 hs_timeout=500-60000 nbio=2" \
76937680
0 \
76947681
-s "Extra-header:" \

0 commit comments

Comments
 (0)