From d9f025817124b3fae4d84cfbb59d69c1d718ec1c Mon Sep 17 00:00:00 2001
From: Dirk <dirk@testssl.sh>
Date: Tue, 25 Sep 2018 10:54:17 +0200
Subject: [PATCH] Fix of errorneous return value due to ALPN check

This commit fixes a problem when not every supplied $ALPN_PROTOs
was suported which is probably never the case ;-/

See #1133
---
 testssl.sh | 1 -
 1 file changed, 1 deletion(-)

diff --git a/testssl.sh b/testssl.sh
index 6f02d61..34c2d44 100755
--- a/testssl.sh
+++ b/testssl.sh
@@ -8792,7 +8792,6 @@ run_alpn() {
           # for some reason OpenSSL doesn't list the advertised protocols, so instead try common protocols
           if "$HAS_ALPN"; then
                $OPENSSL s_client $(s_client_options "-connect $NODEIP:$PORT $BUGS $SNI -alpn $proto") </dev/null 2>$ERRFILE >$TMPFILE
-               [[ $? -ne 0 ]] && ret=1
           else
                alpn_extn="$(printf "%02x" ${#proto}),$(string_to_asciihex "$proto")"
                len="$(printf "%04x" $((${#proto}+1)))"