From 3084f241af9c43f1045f768225503789e769811b Mon Sep 17 00:00:00 2001 From: David Cooper Date: Thu, 8 Feb 2018 11:57:54 -0500 Subject: [PATCH] Fix #983 This commit fixes #983 by ensuring that the line printed just before calling "return" prints a newline character. It also fixes the problem that no output is sent to the JSON/CSV file in some cases in which no fallback is possible since the server does not support two different protocols below TLSv1.3. --- testssl.sh | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/testssl.sh b/testssl.sh index 987522d..c743a3f 100755 --- a/testssl.sh +++ b/testssl.sh @@ -12535,7 +12535,7 @@ run_tls_fallback_scsv() { fileout "$jsonID" "HIGH" "only SSLv3 supported" return 7 ;; - *) pr_svrty_good "No fallback possible, TLS 1.3 is the only protocol (OK)" + *) prln_svrty_good "No fallback possible, TLS 1.3 is the only protocol (OK)" fileout "$jsonID" "OK" "only TLS 1.3 supported" return 7 esac @@ -12557,11 +12557,12 @@ run_tls_fallback_scsv() { if [[ -z "$low_proto" ]]; then case "$high_proto" in "tls1_2") - pr_svrty_good "No fallback possible, no protocol below $high_proto_str offered (OK)" + prln_svrty_good "No fallback possible, no protocol below $high_proto_str offered (OK)" ;; - *) out "No fallback possible, no protocol below $high_proto_str offered (OK)" + *) outln "No fallback possible, no protocol below $high_proto_str offered (OK)" ;; esac + fileout "$jsonID" "OK" "no protocol below $high_proto_str offered" return 7 fi case "$low_proto" in