mirror of
				https://github.com/drwetter/testssl.sh.git
				synced 2025-10-31 05:45:26 +01:00 
			
		
		
		
	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
This commit is contained in:
		| @@ -8792,7 +8792,6 @@ run_alpn() { | |||||||
|           # for some reason OpenSSL doesn't list the advertised protocols, so instead try common protocols |           # for some reason OpenSSL doesn't list the advertised protocols, so instead try common protocols | ||||||
|           if "$HAS_ALPN"; then |           if "$HAS_ALPN"; then | ||||||
|                $OPENSSL s_client $(s_client_options "-connect $NODEIP:$PORT $BUGS $SNI -alpn $proto") </dev/null 2>$ERRFILE >$TMPFILE |                $OPENSSL s_client $(s_client_options "-connect $NODEIP:$PORT $BUGS $SNI -alpn $proto") </dev/null 2>$ERRFILE >$TMPFILE | ||||||
|                [[ $? -ne 0 ]] && ret=1 |  | ||||||
|           else |           else | ||||||
|                alpn_extn="$(printf "%02x" ${#proto}),$(string_to_asciihex "$proto")" |                alpn_extn="$(printf "%02x" ${#proto}),$(string_to_asciihex "$proto")" | ||||||
|                len="$(printf "%04x" $((${#proto}+1)))" |                len="$(printf "%04x" $((${#proto}+1)))" | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Dirk
					Dirk