Explanation is very reasonable - good catch, "just adding a field to the mroute" was far less trivial than the original patch made us assume.
Since it has an +2 from Giaan I have not further tested it, just stared a bit at the change and it makes sense. The server test framework does not currently test floating, which is unfortunately and I really need to address this (but it's somewhat tricky to get right with the t_client framework). mail-archive.com is not cooperating (it seems to categorize all the patches I relay from gerrit to the list as "spam" - the mails from gerrit itself are there but the patch I sent is not...?) - so I recorded the gerrit URL and the initial patch submission URL instead. Your patch has been applied to the master branch. commit db9632f7f976a5e0b810d76173a38f76879d99c0 Author: Ralf Lici Date: Tue Jun 17 10:41:28 2025 +0200 Preserve socket protocol during float processing Signed-off-by: Ralf Lici <r...@mandelbit.com> Acked-by: Gianmarco De Gregori <gianma...@mandelbit.com> Message-Id: <20250617084136.10215-1-g...@greenie.muc.de> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg31902.html URL: https://gerrit.openvpn.net/c/openvpn/+/1063 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