Acked-by: Gert Doering <g...@greenie.muc.de> This indeed is good news - as in, the p2p --tls-server test case that used to reliably break when the client disconnected because --ping wasn't working is now working. Tested on Linux/DCO, FreeBSD/DCO and "Linux no DCO", with the full set of client/server tests - it fixes some bits, and breaks nothing else, so good :-)
Stare-at-code and thinking about the flow of control also confirms that this is a good fix - do_up() is executed always (after PUSH_REPLY for --client, and "on init" for --tls-client / --tls-server) so setting up DCO ping/mss is needed there. (And thanks for adding the logging entry to dco_freebsd.c - I was staring at my logs and wondering why I didn't see anything, even in p2mp mode...) (... and, as an afterthought, I re-checked to be sure we've not lost multipoint keepalive on the way - we haven't, I can see proper "ping interval..." messages for peer-id 0, 1, 2 in my udp-p2mp instance) Your patch has been applied to the master and release/2.6 branch. commit 7c66a6dab54d8efcde57c8fb562f95d95f9b18d4 (master) commit c6ba846384b6a7ab610e5d7de689868fa0a48395 (release/2.6) Author: Arne Schwabe Date: Mon Dec 19 18:40:27 2022 +0100 Ensure that dco keepalive and mssfix options are also set in pure p2p mode Signed-off-by: Arne Schwabe <a...@rfc2549.org> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <20221219174027.2567505-1-a...@rfc2549.org> URL: https://www.mail-archive.com/search?l=mid&q=20221219174027.2567505-1-a...@rfc2549.org 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