Sorry for leaving this ACKed-but-unmerged since July. It got heaped-over by DCO patches...
Anyway. I have no current test rig where I could recreate and test this, but if Arne as resident management interface expert is happy with this, this is fine for me. (The comment / message updates are easy to review and make sense). I have test compiled it :-) - and lightly tested, knowing that the interesting code path is not excercised. Just to be sure. Cherrypicking to 2.5 caused a funny conflict, as the variable is called "in_renog_time" there, and got renamed to "in_renegotiation_time" somewhere later... but solved easily enough. Your patch has been applied to the master and release/2.5 branch. commit ddbe6a6fc26586d09f5a9105f13124c479b4d993 (master) commit 5b178f591c882a6600414104a77a9240f7a29331 (release/2.5) Author: Selva Nair Date: Sun Jul 3 22:58:40 2022 -0400 Fix auth-token usage with management-def-auth Signed-off-by: Selva Nair <selva.n...@gmail.com> Acked-by: Arne Schwabe <a...@rfc2549.org> Message-Id: <20220704025840.2558-1-selva.n...@gmail.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg24627.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