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

Reply via email to