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

Reply via email to