Acked-by: Gert Doering <g...@greenie.muc.de> Stared-at-code (looks good), client side tested with the usual parcours (good), and also tested with my two known-weird/broken test cases for "compress not restored" (fixed!).
Your patch has been applied to the master branch. I tried to apply it to release/2.5, but the resulting binary did not behave - I can see that it restored the comp settings, but it does so *after* OCC handshake (so the OCC warning is still there) *and* it does not re-init compression - so the case "config has comp-lzo, server A pushes compress lz4, server B pushes nothing" (so it should do LZO) still fails. I think I need one of the other patches that changes the "when are options saved/restored" as prerequisite here. Will come back... commit 2a414c3c79bba089183a517a2d355b5aaae0b4a4 (master) Author: Arne Schwabe Date: Thu Apr 8 14:00:28 2021 +0200 Also restore/save compress related options in reconnects Signed-off-by: Arne Schwabe <a...@rfc2549.org> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <20210408120029.19438-2-a...@rfc2549.org> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg22078.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