Do not change servers when retrying sync RPC

This commit is contained in:
Duke
2023-03-31 05:12:36 -07:00
parent bfdda1f1af
commit 828a912d85

View File

@@ -57,6 +57,7 @@ ConnectionLoader::ConnectionLoader(MainWindow* main, Controller* rpc)
ConnectionLoader::~ConnectionLoader() ConnectionLoader::~ConnectionLoader()
{ {
DEBUG("destroying ConnectionLoader");
delete isSyncing; delete isSyncing;
delete connD; delete connD;
delete d; delete d;
@@ -244,12 +245,12 @@ void ConnectionLoader::doAutoConnect()
}, [=](auto) mutable { }, [=](auto) mutable {
DEBUG("sync rpc error! server=" << config->server); DEBUG("sync rpc error! server=" << config->server);
// continually retry sync RPC until it succeeds // continually retry sync RPC until it succeeds
// change server each time it fails // don't change server each time it fails
bool failed = true; bool failed = true;
do { do {
config->server = Settings::getRandomServer(); // config->server = Settings::getRandomServer();
auto connection = makeConnection(config); // auto connection = makeConnection(config);
DEBUG("changed server to " << config->server); // DEBUG("changed server to " << config->server);
connection->doRPC("sync", "", [=](auto) mutable { connection->doRPC("sync", "", [=](auto) mutable {
DEBUG("sync success with server=" << config->server); DEBUG("sync success with server=" << config->server);
failed = false; failed = false;