Sébastien Blin authored 2 years ago and Adrien Béraud committed 2 years ago

When a second permanent put request was received, we didn't call
async_wait after changing the expiration, causing push to not be
sent the second day and presence to expire on some devices after
~3 days.

http: modify timeout behaviour

Sébastien Blin authored 2 years ago and Adrien Béraud committed 2 years ago

The current master is broken with clang, this fix the issue.

Sébastien Blin authored 2 years ago and Adrien Béraud committed 2 years ago

When switching between two network interfaces, local address will
change but not the public ip. This generally cause the client to
not correctly updates its candidates.
In this patch, we inject in the proxy client the local candidates
and use it to check if the network changed.

In the previous version, the timeout for the connection could be
set only after async_connect while the request is sending.
This means if async_connect is slow (it can happen as if there is
an ipv6 endpoint, it can try to resolve it during dozens of seconds
before falling back on ipv4).

In this patch, it moves the timeout callback in the request. This
callback will be set just before async_connect, to be sure that we
are using the callback while connecting.