Thanks. This is a welcome addition for CI tests where we test functionality that might fail "if built with the wrong SSL library" (like, NTLM proxy) - and I could see a few more of this ("compression variants", etc)
The test looks larger than the actual t_client.sh(.in) + doc change as it brings an actual "can NTLM proxy work?" test program - as we discussed the handling of mock_msg.c/mock_msg.h could be improved, but we all haven't been able to come up with a really nice way. Tested ntlm_support with an mbedTLS build, works as designed... openvpn$ tests/ntlm_support FATAL ERROR:MD4 not supported openvpn$ echo $? 1 (Building this on top of a tree that has already been built without this patch leads to confused Makefiles and failures to build "ntlm_support" - but this goes away on a fresh checkout / autoreconf, so will not normally hit users) Your patch has been applied to the master branch. commit 0c7cf0694ee6f878168330e9a084c255c51a9e8b Author: Frank Lichtenheld Date: Fri Mar 8 11:28:18 2024 +0100 t_client.sh: Allow to skip tests Signed-off-by: Frank Lichtenheld <fr...@lichtenheld.com> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <20240308102818.9249-1-g...@greenie.muc.de> URL: https://www.mail-archive.com/search?l=mid&q=20240308102818.9249-1-g...@greenie.muc.de Signed-off-by: Gert Doering <g...@greenie.muc.de> -- kind regards, Gert Doering _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel