aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArne Schwabe2023-05-22 11:12:31 +0200
committerGert Doering2023-07-10 19:08:28 +0200
commit66f51e80b981f08ebc3c38f3fac7d0c88caeb85d (patch)
tree5722a5ddc0607daa3d5c9c5333154db64ac1c81e
parentc5d31ddead4ce12903fb36cbf8b154afbd2aa7f1 (diff)
downloadopenvpn-66f51e80b981f08ebc3c38f3fac7d0c88caeb85d.zip
openvpn-66f51e80b981f08ebc3c38f3fac7d0c88caeb85d.tar.gz
Print a more user-friendly error when tls-crypt-v2 client auth fails
While it might be clear to people being (too?) well versed in typical crypto applications that an authentication failure probably mean wrong decryption key, this is not really obvious for the typical user/server admin. Change-Id: If0f0e7d53f915d39ab69aaaac43dc73bb9c26ae9 Signed-off-by: Arne Schwabe <arne@rfc2549.org> Acked-by: Gert Doering <gert@greenie.muc.de> Message-Id: <20230522091231.2837468-1-arne@rfc2549.org> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26718.html Signed-off-by: Gert Doering <gert@greenie.muc.de> (cherry picked from commit 7a477c16a7c2a7016c7b15ea98fe3c40e8ef675b)
-rw-r--r--src/openvpn/tls_crypt.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/openvpn/tls_crypt.c b/src/openvpn/tls_crypt.c
index 88b2d6d..7354236 100644
--- a/src/openvpn/tls_crypt.c
+++ b/src/openvpn/tls_crypt.c
@@ -524,6 +524,8 @@ tls_crypt_v2_unwrap_client_key(struct key2 *client_key, struct buffer *metadata,
dmsg(D_CRYPTO_DEBUG, "tag_check: %s",
format_hex(tag_check, sizeof(tag_check), 0, &gc));
CRYPT_ERROR("client key authentication error");
+ msg(D_TLS_DEBUG_LOW, "This might be a client-key that was generated for "
+ "a different tls-crypt-v2 server key)");
}
if (buf_len(&plaintext) < sizeof(client_key->keys))