This is the same patch that Antonio tested when it was still only
discussed on the security@ list (except for the "two spaces" comment
fix). Also, fixed a "do" -> "does" in plugin-options.rst.
As discussed before, I still think that "aborting the server process"
is excessive and we should just return OPENVPN_PLUGIN_FUNC_ERROR in
this case (with a proper log message), but we'll have to agree to
disagree on this one. I have a patch to bring proper functionality
instead... :-)
I have not tested this particular scenario (Antonio has), just made
sure nothing else got broken.
I have not updated Changes.rst in the context of this patch - will
do that when preparing 2.5.6 release.
Your patch has been applied to the master and release/2.5 branch.
commit 282ddbac54f8d4923844f69983b38dd2b813a00a (master)
commit af3e382649d96ae77cc5e42be8270f355e5cfec5 (release/2.5)
Author: David Sommerseth
Date: Sun Mar 13 20:31:53 2022 +0100
plug-ins: Disallow multiple deferred authentication plug-ins
Signed-off-by: David Sommerseth <dav...@openvpn.net>
Acked-by: Antonio Quartulli <anto...@openvpn.net>
Message-Id: <20220313193154.9350-3-open...@sf.lists.topphemmelig.net>
URL:
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg23931.html
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
