Port the exclude test to the QTest Framework

This is just a port to QtTest, I did not change the layout of the test.
I did search and replace to replace the assert with QCOMPARE/QVERIFY
I still call setup and setup_init like before (only explicitly, now)

Also ported the preformence tests to QBENCHMAK because windows don't have
gettimeofday.

Relates #6358

Test: Move the test for Utility::normalizeEtag to testutility.cpp

I just moved the text and did the minimum to port it to QtTest
Did not change hte layout of it.

Relates #6358

Remove check_csync_util

It's testing nothing.
Only that one can convert a csync instruction to a string.
But this is only used in debug anyway

Relates to #6358

Database: Change path for new dbs to .sync_* #5904

This is to avoid issues on OSX, where the ._ prefix has special meaning.

Originally (before 2.3.2) ._ was necessary to guarantee exclusion. But
since then the .sync_ prefix is excluded as well.

This does not affect existing database files.

Vfs: Have a static list of potential plugins for now

Fixes in-source builds and other cases where more non-plugin directories
are created in src/libsync/vfs.

Vfs: Ensure older versions gracefully ignore winvfs folders

Previously there'd likely be a mess if a 2.6 winvfs folder was attempted
to be used with a 2.5 client. Now the older clients will ignore these
folders.

Vfs plugins: Available plugins must load

A plugin that can't be loaded due to dependency issues should not be
considered as available.

SyncEngine: Ensure that the paths passed to the discovery ends with slashes

This was making the tx.pl test fail

Propagator: Don't abort sync on error 503

Only do it when it is actually a maintenance mode

Issues #5088, #5859, https://github.com/owncloud/enterprise/issues/2637

Allow to open log window via command line to already running client

Issue: #4098

Fix installation of vfs suffix plugin

VFS: Plugin needs to be installed, otherwise it can't be loaded

Fix compilation warning

src/libsync/propagatorjobs.cpp:63:10: warning: lambda capture 'this' is not used [-Wunused-lambda-capture]
        [this, &deleted](const QString &path, bool isDir) {
         ^~~~~

vfs: Make Vfs ctor explicit

vfs: Separate vfs availability from new-files-virtual

This helps support 2.5 settings where there are virtual files in the
tree but new files aren't created virtual.

It's also a prelude for #6815

There's currently no way of
- upgrading vfs plugins (a silent suffix->winvfs upgrade is attempted
  once only, when moving to master)
- disabling vfs capabilities outright

vfs: Don't always load plugins, check metadata

vfs: ensure backwards-compatible settings value is always set

vfs: Ensure SyncOptions::_vfs is never null

- Create a VfsOff derived class
- Make it a shared pointer shared with Folder::_vfs

vfs: Switch order of deletion and journal removal in dehydration

To be more crash-resilient.

vfs: Update outdated comment

vfs: Ensure local discovery is done on dehydration request

Folder settings: Keep "usePlaceholders" to be backwards compatible

vfs: Be more careful about Vfs instance ownership

vfs: Remove VfsSuffixPrivate

vfs: Document ignored return value.

Plugin: Drop PluginLoader

Plugin: style fixes

Checksums: Make file ownership more explicit

vfs: Improve modeFromString() signature

Generalize Result<> class, add Optional<>

To make it nicer to use outside of HTTP results.

vfs: Add API docs

Remove unused include directories

vfs: Fix dealing with missing vfs instance

Maybe there should just be a VfsOff instance to make this less error
prone.

Fix export of vfs functions

Cleanup CMake (output)

vfs: Add check for presence of win plugin

vfs: Make switching vfs on/off work again

vfs: Fix plugin decision in wizards, sanitize loading

vfs: Reenable local metadata updating of vfs files

vfs: Make some behaviors suffix-vfs specific

vfs: Allow remote renames to propagate as such

vfs: Allow folders without vfs

Fix big-folder detection when vfs is disabled

vfs: Implement dehydration tagging

vfs: Fix suffix detection and handling

winvfs: Better error reporting

- fix download and validation error reporting
- add tests

Fixes after rebase to master

- use vfs suffix in ProcessDirectoryJob
- fix include vfs.h
- fix local vio passing vfs
- fix checksum computation
- vfs mode use
- mingw lambda compile issue

winvfs: initial work

Done by ckamm and dschmidt

FolderMan: Remove assumption of unique running sync

Checksums: Work on QIODevice*s

Needed for cfapi where we want to feed data through a custom device
which retrieves data from the windows api.

IgnoreListEditor: increase a bit the size

Should Help for issue #6641

Account: Fix crash when deleting an account

We should not have request parented to the account, otherwise we might get
a loop on the deletion order.

Issue #6893

LockWatcher: Add chrono header

cjhash: Fix build fix

See 17a4055e8d658f24b1fc47c44fd2daf66c7d5dbf

cjhash: use Q_FALLTHROUGH

This fixes the warning
   warning: unknown option after ‘#pragma GCC diagnostic’ kind [-Wpragmas]

Issue #6872

Excludes: switch all the path handling to QString

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

CMakeLists.txt: increment sqlite version

3.9 is required for custom index to work

csync_vio_locale: use QString for the path in opendir

So we don't have to convert to utf8 and back again

Remove csync_misc: The code no longer use fnmatch

Exclude: do everything with QString wiuthout converting to char*

ProcessDirectoryJob::process: optimize so there is only one map

PropagateUpload:  Avoid many allocations by using QByteArrayLiteral

Optimisation: Add a cache SyncJournalDb::mapChecksumType

No need to do two sql query for something that's always the same and
there are very few checksum types

Sync: optimize by removing setFileRecordMetadata

Inh most case we already have a record from before, so avoid doing a useless
lookup in the database.
In owncloudpropagator.cpp, directories do not have a checksum so no need
to call a function that preserves it

Test System: Optimisations so the benchmark is significant

OwnSql: Speedup by avoiding allocating an error string on success

owncloudcmd: fix compiliation with TOKEN_AUTH_ONLY

libsync: Fix build error with TOKEN_AUTH_ONLY

When enabling TOKEN_AUTH_ONLY, the code path using QPainter is disabled.
So we also don't need the includes.
This header is not available for Remarkable.

Disable stack protector features when cross compiling

When cross-compiling this for remarkable using the poky toolchain
this results in linker errors with stack protector: libssp_nonshared.a
not found

Add CMake option to disable GUI build

Get DatabaseErrorTest to pass

The E2EE code path would get the engine to go wrong in case of db error.
It's just better to have a failing upload or failing mkdir later in those
cases.

Emitting signals from a ctor is a bad idea anyway

Signed-off-by: Kevin Ottens <kevin.ottens@nextcloud.com>

Test that the sync behave well if there are errors while reading the database

Discovery: fix double emission of finished in case of error

Discovery: Fix downloading files when database is used for local discovery

This also fix the currently failling LockedFilesTest

Remove csync.cpp: It's only dead code

Add Ctrl-L as log window shortcut

F12 is taken on OSX and there's no other way of showing it.

Test: Add test for locked file tracking and propagation

Folder: Treat file unlock similar to external change #6822

For consistent handling of incoming notifications.

Propagator: Add assert against duplicate done() calls

RemoteDiscoveryTest: Fix after the merge from 2.5

The new discovery's message is slightly different

Test: Check for folder in error message #6826

gitignore: XCode userdata

Discovery: Fix renaming on windows

buf.type is ItemFileSkip because csync_vio_local_stat does not set this field

New Discovery: Fix trailing slash causing failure on windows

Folder: update the folder version

Since the new index would crash old version of the client, we need to
upgrade folder version so they do not load in the client 2.5.0

Settings: Make FoldersWithPlaceholders group sticky

If virtual files are disabled on a folder it might still have db entries
or local virtual files that would confuse older client versions.

owncloudcmd: Read the server version from the capabilities

Issue: #6846

owncloudcmd: fetch the dav user

This is required to get the new endpoint working when the server
uses ldap or that the dav user is not the same as the login.

Issue #6830

Sharing: add the `shareWithAdditionalInfo` string in autocompletion results

Issue #6749

Migration from 2.4: fallback to move file by file if directory move failled (#6807)

Migration from 2.4: fallback to move file by file if directory move failed

This can happen if the directory already exist because, say, it was
created by the ownCloud outlook plugin which save its file in the same directory

Move: Fix too many starting slashes in the destination header

QDir::cleanPath does not remove starting slashes on windows.
So use account::davUrl which is already cleaned

Issue: #6824

Wizard: Reset the QSslConfiguration before checking the server

Because a previous call with another demain might have set some
config for another server.

Issue #6832

Fix compiler warning

OAuth2: Refresh the token without aborting the sync

OAuth2 access token typically only has a token valid for 1 hour.
Before this patch, when the token was timing out during the sync, the
sync was aborted, and the ConnectionValidator was then requesting a new
token, so the sync can be started over.
If the discovery takes longer than the oauth2 validity, this means that
the sync can never proceed, as it would be always restarted from scratch.

With this patch, we try to transparently renew the OAuth2 token and restart
the jobs that failed because the access token was invalid.

Note that some changes were required in the GETFile job because it handled
the error itself and so it was erroring the jobs before its too late.

Issue #6814

HttpCredentials: initialize all member inline

Disable autostartCheckBox if autostart is configured system wide

Discovery: Virtual file handling adjustments

- adjust virtual file path handing
- helpers for vfs suffix adding/removal
- helpers for isDirectory/isVirtual on SyncJournalRecords
- be clear about what PathTuple _local/_server mean

Database: Add an index on the parent path

So we can quickly query the items in a parent directory

This uses a custom slite3 function, which means that when downgrading the client,
or using another tool to add entries in the database, any insertion in the metadata
table will produce an error: "unknown function: parent_hash()"
(This will crash the client 2.5)

Add missing export

Discovery: Fix log output for instructions