I have stared a bit at the code, and it all looks sane (though kp@ and
Antonio are the experts here, anyway :-) ).  I have not tested things
beyond "passes client/server tests, with and without DCO" - my kernel
does not have the new code bits yet (and for a real tests, one would
have to modify it to send OVPN_NOTIF_ROTATE_KEY after like 1000 packets
or so...).

There were a few indentations that uncrustify did not like, so fixed
those one the fly (only whitespace).

Your patch has been applied to the master and release/2.6 branch
(this is somewhere between "bugfix" and "long-term compatibility",
and less of a "new feature").

commit ec71489bfc7c1d798f5f6de8e9fc187b9127072c (master)
commit c468af2cd90c9f682519eff38a21fac8a3feb148 (release/2.6)
Author: Kristof Provost
Date:   Fri Apr 14 11:42:27 2023 +0200

     DCO: support key rotation notifications

     Signed-off-by: Kristof Provost <kprov...@netgate.com>
     Acked-by: Antonio Quartulli <anto...@openvpn.net>
     Message-Id: <20230414094227.9153-1-kprov...@netgate.com>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26590.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