diff options
author | Christian Kamm <mail@ckamm.de> | 2019-04-10 12:07:15 +0300 |
---|---|---|
committer | Christian Kamm <mail@ckamm.de> | 2019-04-11 14:48:56 +0300 |
commit | 440c06c54a9012a115a7b39f1096e90edebb4b5a (patch) | |
tree | 98b780d4a0ac375a6d07a01de4e1b8778646dbf3 /src | |
parent | 702a494cbbd5fbdab39ee25a5d6c0c703b467d6d (diff) |
Discovery: 403 and 503 on root cause error
Previously these result codes during remote discovery of the sync root
would not cause an error and the discovery would get stuck.
Also extends RemoteDiscovery tests to check for errors on the root item.
Diffstat (limited to 'src')
-rw-r--r-- | src/libsync/discovery.cpp | 32 |
1 files changed, 19 insertions, 13 deletions
diff --git a/src/libsync/discovery.cpp b/src/libsync/discovery.cpp index 93d8a24d7..7dce97b4c 100644 --- a/src/libsync/discovery.cpp +++ b/src/libsync/discovery.cpp @@ -1334,31 +1334,37 @@ DiscoverySingleDirectoryJob *ProcessDirectoryJob::startAsyncServerQuery() if (_localQueryDone) this->process(); } else { - if (results.error().code == 403) { - // 403 Forbidden can be sent by the server if the file firewall is active. - // A file or directory should be ignored and sync must continue. See #3490 - qCWarning(lcDisco, "Directory access Forbidden (File Firewall?)"); + auto fatalError = [&] { + emit _discoveryData->fatalError(tr("Server replied with an error while reading directory '%1' : %2") + .arg(_currentFolder._server, results.error().message)); + }; + auto ignoreOrFatal = [&] { if (_dirItem) { _dirItem->_instruction = CSYNC_INSTRUCTION_IGNORE; _dirItem->_errorString = results.error().message; emit this->finished(); - return; + } else { + // Fatal for the root job since it has no SyncFileItem + fatalError(); } + }; + + if (results.error().code == 403) { + // 403 Forbidden can be sent by the server if the file firewall is active. + // A file or directory should be ignored and sync must continue. See #3490 + qCWarning(lcDisco, "Directory access Forbidden (File Firewall?)"); + ignoreOrFatal(); } else if (results.error().code == 503) { // The server usually replies with the custom "503 Storage not available" // if some path is temporarily unavailable. But in some cases a standard 503 // is returned too. Thus we can't distinguish the two and will treat any // 503 as request to ignore the folder. See #3113 #2884. qCWarning(lcDisco(), "Storage was not available!"); - if (_dirItem) { - _dirItem->_instruction = CSYNC_INSTRUCTION_IGNORE; - _dirItem->_errorString = results.error().message; - emit this->finished(); - return; - } + ignoreOrFatal(); + } else { + qCWarning(lcDisco) << "Server error in directory" << _currentFolder._server << results.error().message; + fatalError(); } - emit _discoveryData->fatalError(tr("Server replied with an error while reading directory '%1' : %2") - .arg(_currentFolder._server, results.error().message)); } }); connect(serverJob, &DiscoverySingleDirectoryJob::firstDirectoryPermissions, this, |