From 28d9bc483dd5fe31ac106e13d58cf90dc68a826d Mon Sep 17 00:00:00 2001 From: Dirk Date: Sat, 19 Jun 2021 15:27:03 +0200 Subject: [PATCH] Fix redundant message (BREACH) when client certificate required same as #1916. Fixes #1915 --- testssl.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/testssl.sh b/testssl.sh index 82a6c36..7ec39ea 100755 --- a/testssl.sh +++ b/testssl.sh @@ -16399,8 +16399,9 @@ run_breach() { [[ $VULN_COUNT -le $VULN_THRESHLD ]] && outln && pr_headlineln " Testing for BREACH (HTTP compression) vulnerability " && outln pr_bold " BREACH"; out " ($cve) " if [[ "$CLIENT_AUTH" == required ]]; then - outln "cannot be tested (server side requires x509 authentication)" - fileout "$jsonID" "INFO" "was not tested, server side requires x509 authentication" "$cve" "$cwe" + prln_warning "client x509-based authentication prevents this from being tested" + fileout "$jsonID" "WARN" "client x509-based authentication prevents this from being tested" "$cve" "$cwe" + return 7 fi [[ -z "$url" ]] && url="/"