From 0f16ee66e2ab2e592f3d153435fcc34ccca2646b Mon Sep 17 00:00:00 2001 From: Kim Alvefur Date: Sat, 6 Feb 2021 22:12:38 +0100 Subject: core.certmanager: Add comments explaining the 'verifyext' TLS settings Thanks to debacle for reminding me, in the context of mod_auth_ccert I wonder if we still need lsec_ignore_purpose, Let's Encrypt seems to include both client and server purposes in certs. --- core/certmanager.lua | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/core/certmanager.lua b/core/certmanager.lua index 023218fa..4bc98935 100644 --- a/core/certmanager.lua +++ b/core/certmanager.lua @@ -118,7 +118,10 @@ local core_defaults = { single_dh_use = luasec_has.options.single_dh_use; single_ecdh_use = luasec_has.options.single_ecdh_use; }; - verifyext = { "lsec_continue", "lsec_ignore_purpose" }; + verifyext = { + "lsec_continue", -- Continue past certificate verification errors + "lsec_ignore_purpose", -- Validate client certificates as if they were server certificates + }; curve = luasec_has.algorithms.ec and not luasec_has.capabilities.curves_list and "secp384r1"; curveslist = { "X25519", -- cgit v1.2.3