Hi, On 20/05/2021 17:11, Arne Schwabe wrote: > the socket_info->connection_establish is set through > link_socket_set_outgoing_addr when we reach FULL_SYNC. This patch > introduces a new state in context_auth that replaces the > connection_established state for TLS connections. This make the state > machine easier to understand. > > Patch v2: fix p2p mode server without (without ncp) > > Signed-off-by: Arne Schwabe <a...@rfc2549.org>
Compile zoo is happy. Performed various tests, including: * classic server/client * p2p+tls with ncp * p2p+tls without ncp All passed and renegotiation also worked as expected. The patch looks sane (already reviewed v1 and the added change makes sense) and it's a reasonable step forward towards making the state machine simpler to understand. Acked-by: Antonio Quartulli <anto...@openvpn.net> -- Antonio Quartulli _______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel