Skip to content

Commit 62b5859

Browse files
authored
Merge pull request #2465 from Odinmylord/3.2
Add SNI to ticketbleed check
2 parents c30e541 + eb661da commit 62b5859

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

testssl.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16675,7 +16675,7 @@ sub_session_ticket_tls() {
1667516675
#ATTENTION: we DO NOT use SNI here as we assume ticketbleed is a vulnerability of the TLS stack. If we'd do SNI here, we'd also need
1667616676
# it in the ClientHello of run_ticketbleed() otherwise the ticket will be different and the whole thing won't work!
1667716677
#
16678-
sessticket_tls="$($OPENSSL s_client $(s_client_options "$BUGS $tls_proto $PROXY -connect $NODEIP:$PORT") </dev/null 2>$ERRFILE | awk '/TLS session ticket:/,/^$/' | awk '!/TLS session ticket/')"
16678+
sessticket_tls="$($OPENSSL s_client $(s_client_options "$BUGS $tls_proto $PROXY $SNI -connect $NODEIP:$PORT") </dev/null 2>$ERRFILE | awk '/TLS session ticket:/,/^$/' | awk '!/TLS session ticket/')"
1667916679
sessticket_tls="$(sed -e 's/^.* - /x/g' -e 's/ .*$//g' <<< "$sessticket_tls" | tr '\n' ',')"
1668016680
sed -e 's/ /,x/g' -e 's/-/,x/g' <<< "$sessticket_tls"
1668116681

0 commit comments

Comments
 (0)