As for the previous patch, this had quite a journey... +2 from Lev on v12, then rebases up to v21, stylistic discussions with me (and "please add comments so we can remember in two months why this extra check needs to be done" ;-) ), to be integrated into v22, +2'ed again, and v23 another rebase... so recording my +2 as "Acked-by:".
Lev has tested this and confirms the new stuff (808+809+810) works. I have stared long and hard at the code, and run the client/server side testbeds on it, and nothing breaks. I have not tested the PUSH_UPDATE machinery yet (the t_client tests do excercise the change to split delete_routes(), but that's trivial enough) Your patch has been applied to the master branch. commit 73f3247e89bd66fa0b5142e3e1773951f6c3cba0 Author: Marco Baffo Date: Tue Jul 29 12:40:50 2025 +0200 PUSH_UPDATE: Added remove_option() and do_update(). Signed-off-by: Marco Baffo <ma...@mandelbit.com> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <20250729104056.27634-1-g...@greenie.muc.de> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg32407.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