From f2a4ee1090e209f09be5c568ae1256d995e6acc6 Mon Sep 17 00:00:00 2001 From: David Cooper Date: Wed, 8 Dec 2021 11:50:35 -0500 Subject: [PATCH] Document $keyopts Shellcheck complains on line 2234 that keyopts is referenced but not assigned and there is no explanation in the code (or in the documentation) why "$keyopts" is there. This commit adds a comment so that "$keyopts" isn't deleted as part of a code cleanup. --- testssl.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/testssl.sh b/testssl.sh index 9ff74d6..9b9c0f8 100755 --- a/testssl.sh +++ b/testssl.sh @@ -2231,6 +2231,7 @@ s_client_options() { [[ "$1" =~ secp192r1 ]] && options="${options//secp192r1/prime192v1}" [[ "$1" =~ secp256r1 ]] && options="${options//secp256r1/prime256v1}" fi + # $keyopts may be set as an environment variable to enable client authentication (see PR #1383) tm_out "$options $keyopts" }