Skip to content

Commit 04f7b34

Browse files
committed
Added SNI to openssl call for testing secure renegotiation
1 parent aa5235e commit 04f7b34

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

testssl.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16896,7 +16896,7 @@ run_renego() {
1689616896
local hint=""
1689716897
local jsonID=""
1689816898
local ssl_reneg_attempts=$SSL_RENEG_ATTEMPTS
16899-
# No SNI needed here as there won't be two different SSL stacks for one IP
16899+
# SNI is needed here as openssl return an error if missing
1690016900

1690116901
"$HAS_TLS13" && [[ -z "$proto" ]] && proto="-no_tls1_3"
1690216902

@@ -16913,7 +16913,7 @@ run_renego() {
1691316913
fileout "$jsonID" "OK" "TLS 1.3 only server" "$cve" "$cwe"
1691416914
else
1691516915
# first fingerprint for the Line "Secure Renegotiation IS NOT" or "Secure Renegotiation IS "
16916-
$OPENSSL s_client $(s_client_options "$proto $STARTTLS $BUGS -connect $NODEIP:$PORT $PROXY") 2>&1 </dev/null >$TMPFILE 2>$ERRFILE
16916+
$OPENSSL s_client $(s_client_options "$proto $STARTTLS $BUGS -connect $NODEIP:$PORT $PROXY $SNI") 2>&1 </dev/null >$TMPFILE 2>$ERRFILE
1691716917
if sclient_connect_successful $? $TMPFILE; then
1691816918
grep -iaq "Secure Renegotiation IS NOT" $TMPFILE
1691916919
sec_renego=$? # 0= Secure Renegotiation IS NOT supported

0 commit comments

Comments
 (0)