Do not require server replies to contain an mtime
authorChristian Kamm <mail@ckamm.de>
Wed, 25 Jul 2018 09:13:14 +0000 (11:13 +0200)
committerKevin Ottens <kevin.ottens@nextcloud.com>
Tue, 15 Dec 2020 09:58:02 +0000 (10:58 +0100)
The check was added for #6317 in commit
13eb64584f5f96647ced39dcd3252860ebec5a37.

We did see missing mtimes in replies in tests with live servers though.
Possibly those were old incomplete responses cached in the stat cache?

src/libsync/discoveryphase.cpp

index d765634c629bd17be5c902e983bbf68f3a7f21b0..93efdd0c827a2c1dccd97e8899f85b5b2a6de9bb 100644 (file)
@@ -280,12 +280,10 @@ void DiscoverySingleDirectoryJob::directoryListingIteratedSlot(QString file, con
         int slash = file.lastIndexOf('/');
         result.name = file.mid(slash + 1);
         result.size = -1;
-        result.modtime = -1;
         propertyMapToFileStat(map, result);
         if (result.isDirectory)
             result.size = 0;
         if (result.size == -1
-            || result.modtime == -1
             || result.remotePerm.isNull()
             || result.etag.isEmpty()
             || result.fileId.isEmpty()) {