diff options
| author | Carl Hetherington <cth@carlh.net> | 2020-11-26 02:05:25 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2020-11-26 02:24:32 +0100 |
| commit | dfb23c675ca145b5570ccb5790ea804ad736a247 (patch) | |
| tree | ac336998be15bd3843abf301a0af2113a6b65ddc | |
| parent | 82fbf2b9e43fb234f843dc3352fecbd08eeed3f7 (diff) | |
If we don't query a server (because we already know about it)
the "last seen time" will never be updated, so the server will
be discarded.
It seems that we should always ping servers (so that set_seen gets
called on receipt of the response), no matter whether
"auto-discovered" or configured, so that the "discard" code doesn't
kick in.
Otherwise we remove and re-add our configured servers every
10 seconds, which is inefficient and which possibly triggers
other bugs.
| -rw-r--r-- | src/lib/encode_server_finder.cc | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/lib/encode_server_finder.cc b/src/lib/encode_server_finder.cc index 3401a5cf5..c823d868f 100644 --- a/src/lib/encode_server_finder.cc +++ b/src/lib/encode_server_finder.cc @@ -123,10 +123,6 @@ try /* Query our `definite' servers (if there are any) */ BOOST_FOREACH (string const& i, Config::instance()->servers()) { - if (server_found(i)) { - /* Don't bother asking a server that we already know about */ - continue; - } try { boost::asio::ip::udp::resolver resolver (io_service); boost::asio::ip::udp::resolver::query query (i, raw_convert<string> (HELLO_PORT)); |