Upload: Do not error out if the server do not send the X-OC-MTime: accepted header
authorOlivier Goffart <ogoffart@woboq.com>
Tue, 9 Oct 2018 07:20:15 +0000 (09:20 +0200)
committerKevin Ottens <kevin.ottens@nextcloud.com>
Tue, 15 Dec 2020 09:58:11 +0000 (10:58 +0100)
If the server does not set the mtime, it is not a big problem for the
synchronisation.

The test was used before so we could do a PROPPATCH for server that did not
support this header. But now that all server supports that we don't need to
to the check. (We do not do the PROPPATCH since we got rid of the neon
dependency)

Apparently, it may happen that some backend don't support setting mtime
and this can lead to this error.

https://github.com/owncloud/client/issues/6797

src/libsync/propagateuploadv1.cpp

index e28a7be625ef0466297fc44fab5dc50f93ca4dfc..c305d8f3279a1a1d5e58af7cbcfd257ae928072d 100644 (file)
@@ -313,8 +313,6 @@ void PropagateUploadFileV1::slotPutFinished()
         // Normally Owncloud 6 always puts X-OC-MTime
         qCWarning(lcPropagateUpload) << "Server does not support X-OC-MTime" << job->reply()->rawHeader("X-OC-MTime");
         // Well, the mtime was not set
-        done(SyncFileItem::SoftError, "Server does not support X-OC-MTime");
-        return;
     }
 
     finalize();