Merge pull request #6160 from nextcloud/bugfix/vfs-error-dialog-on-uk…

…nown-error Do not show failure popul for VFS when hydration finished with code 200 or 204.
nextcloud · Oct 24, 2023 · 511af3a · 511af3a
2 parents ad4b7b1 + 5f9e04d
commit 511af3a
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/libsync/vfs/cfapi/hydrationjob.cpp b/src/libsync/vfs/cfapi/hydrationjob.cpp
@@ -351,9 +351,15 @@ void OCC::HydrationJob::onGetFinished()
 {
     _errorCode = _job->reply()->error();
     _statusCode = _job->reply()->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt();
-    _errorString = _job->reply()->errorString();
-
-    qCInfo(lcHydration) << "GETFileJob finished" << _requestId << _folderPath << _errorCode << _statusCode << _errorString;
+    if (_errorCode != 0 || (_statusCode != 200 && _statusCode != 204)) {
+        _errorString = _job->reply()->errorString();
+    }
+
+    if (!_errorString.isEmpty()) {
+        qCInfo(lcHydration) << "GETFileJob finished" << _requestId << _folderPath << _errorCode << _statusCode << _errorString;
+    } else {
+        qCInfo(lcHydration) << "GETFileJob finished" << _requestId << _folderPath;
+    }
     // GETFileJob deletes itself after this signal was handled
     _job = nullptr;
     if (_isCancelled) {