Make client delay less before sending PUSH_REQUEST

Speed up PUSH_REQUEST handling on the client side by reducing
the amount of dependent 1s-coarse-timers to "just one".

After "TLS is up!", one timer needs to fire to wakeup
"check_connection_established_dowork()", and that one used to
setup another 1s timer before sending PUSH_REQUEST - which is
just dead time where we sit idly...  So, fire immediately.

Acked-by: Arne Schwabe <arne@rfc2549.org>
Message-Id: <20150701154039.GG382@greenie.muc.de>
URL: http://article.gmane.org/gmane.network.openvpn.devel/9851

Signed-off-by: Gert Doering <gert@greenie.muc.de>

diff --git a/src/openvpn/forward.c b/src/openvpn/forward.c
index 92e443ac8fc2b64500f484f775176755e8ae8b79..7a5d383853694db8db90e4d9faf0897c665ae0fb 100644 (file)
--- a/src/openvpn/forward.c
+++ b/src/openvpn/forward.c
@@ -212,8 +212,8 @@ check_connection_established_dowork (struct context *c)
                                        0);
                }
 #endif
-             /* send push request in 1 sec */
-             event_timeout_init (&c->c2.push_request_interval, 1, now);
+             /* fire up push request right away (already 1s delayed) */
+             event_timeout_init (&c->c2.push_request_interval, 0, now);
              reset_coarse_timers (c);
            }
          else