Jenkinsfile: Run verbose tests

Jenkinsfile: Run full testing

owncloudcmd: Explain --debug parameter

Remove debug messages by default from sync.engine

Other categories were already set properly.

Compare the hash of files with identical mtime/size #5589

* For conflicts where mtime and size are identical:

  a) If there's no remote checksum, skip (unchanged)
  b) If there's a remote checksum that's a useful hash, create a
     PropagateDownload job and compute the local hash. If the hashes
     are identical, don't download the file and just update metadata.

* Avoid exposing the existence of checksumTypeId beyond the database
  layer. This makes handling checksums easier in general because they
  can usually be treated as a single blob.

  This change was prompted by the difficulty of producing file_stat_t
  entries uniformly from PROPFINDs and the database.

SyncEngineTest: Add network override

This is useful for monitoring what kind of network requests are
sent to the fake server. Such as "did this sync cause an upload?"
and "was there a propfind for this path?". It can also inject
custom replies.

SyncEngineTest: Send 'checksums' in FakePropfind

SyncEngineTest: Fix date locale related bug

The client is very picky about date strings it accepts. If dates are
formatted with a non-C locale (such as localized weekday names), it
fails to parse it and tests fail in subtle ways.

shell/Windows: Update binaries to build 45

shell_integration/windows: Create _newQueries as a non-signaled auto-reset event.

Since ResetEvent() is never called, the call to WaitForMultipleObjects()
can always return immediately once the event has been signaled.

Initialize stop variable.

Avoid stopping the worker thread immediately if the stop variable happens to be true.

AccountSettings: Triggering log in re-ask about previously rejected certificates #5819

Issue #5819

OAuth: cleanup debug messages

- Add category to the all messages (they did not have it was merged right after
the patch to add category everywhere, but this code did not have it.)

- Make sure there is no warnings in the normal flow. (The wizard does a request
without authentication to determine the auth type)

Don't link dirrectly against openssl

All our crypto code is handled by qt nodaways.
No need to carry this dependency.

Especially since it causes warnings on system where there are
twp openssl version installed:
/usr/bin/ld: warning: libcrypto.so.1.0.0, needed by /usr/lib/libQt5Network.so.5.9.0, may conflict with libcrypto.so.1.1

Update ChangeLog

OAuth: Fix a typo in the success message

Wizard: Don't let the second to last page go back with OAuth2

We don't want to re-open the browser in that case.

OAuth: Error handling in the wizard

Issues: #5813 and #5811

SyncEngine: SelectiveSync: Remove local files of undelected folder despite other modified files

Issue #5783

When the directry that should be removed by selective sync contains changes,
we ignore the whole sub tree instead of only ignoreing new files.
We cannot ignore the whole directory, we need to ignore only the directory
that do not have files to remove

Merge branch '2.3'

ownsql: fix build on jenkins

sqlite3_system_errno was only added in sqlite 3.12

(cherry picked from commit e2e56f45af059299d942f237ecf9cefbba0c43be)

Add extra logging for SQLITE_CANTOPEN errors #5633

(cherry picked from commit 05c1dcef9b35a18d358ab307922eceff312e3304)

[tx-robot] updated from transifex

[tx-robot] updated from transifex

OAuth: Fix the secret to the one from commit 05fd985f9f in the oauth2 repository

https://github.com/owncloud/oauth2/commit/05fd985f9fb58d7d2305ce2db6fe24fe6fc08c79

FolderStatusModel: fix assert in Qt when the list of subfolder is empty

Fix an assert that happens in beginInsertRows when opening a folder
and that folder is empty.
This can only be reproduced with a debug build of Qt.

ownsql: fix build on jenkins

sqlite3_system_errno was only added in sqlite 3.12

Add extra logging for SQLITE_CANTOPEN errors #5633

Fix maintenance mode detection #5755

The server sends "maintenance": "true" which isn't a json boolean...

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Authentication with OAuth2

When the OAuth2 app (https://github.com/owncloud/oauth2) is enabled,
We will open a browser and perform the OAuth2 authentication

Issue: #4798 and https://github.com/owncloud/platform/issues/17

ChangeLog: More things that were done until now

Set proper application name, icon name for freedesktop notifications

Use application name defined in OEM build configuration

[tx-robot] updated from transifex

Application: debug implies logdebug

Remove obsolete FindQtKeychain for Qt4

Fix build on Debian 7 (CMake pre 2.8.12)

Download: Retrigger folder discovery on 404

See owncloud/enterprise#1966

If the server and the client's database go out of sync, there could be
persistent 404 errors. This change ensures that the problem corrects
itself eventually by triggering a remote discovery of the file's
parent folders.

It does not address the root cause that might have lead to the
divergence.

Don't call QElapsedTimer::restart on uninitialized timer

That's an undefined behavior.
Since we don't use the return value anyway, we should just use start()

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Sharing: Disable 'set password' button when password is empty #5788

Share dialog: Use default icon if available #5781

Share link: Fix disabled API/sharing #5778

* Don't hide the displayed error directly after showing it
* Don't try to fetch shares when sharing isn't possible

Share Dialog: Remove unused hline

Share link: Fix parsing integer ids #5777

Older ownCloud servers like 8.2.0 used to use integer ids.

This broke when switching to QJson because toString() on JSON
values is strict and returns "" for integers.

Sharing dialog: Ensure error labels are expanding #5774

Share dialog: Hide the icon when unused

Previously this used up some space when there was no icon, leading
to an oddly indented header.

Credentials: Simplify credential flow #5728

And as a side effect: don't ask for user password when we can't
connect to the server in the first place.

Account: forget credentials on delete #5752

Sharing: Show warning that link shares are public #5785 #5747

Backport of 3872e060ba33913e43714776a1afce787e772351

Settings: Ensure the 'about' label isn't cut off #5760

Merge branch '2.3'

mirall.desktop.in: Remove superfluous lines #5767

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge remote-tracking branch 'origin/2.3'

Add the account name and folder when starting the sync

This should help clearly delimiting them in the log for multi-account setups.
This information is already available elsewhere in the log in any case.

Issue #5672

Reformatting: Add .git-blame-ignore-revs

For git hyper-blame support

Apply clang-format

Reformatting: Reformat some lines that would become really long

Reformatting: Adjust trailing comments

These would otherwise be line-wrapped by clang-format,
and then consecutive reformattings remove the aligned
comment indentation

Example:

int a; // too long comment

->

int a; // too long
       // comment

->

int a; // too long
// comment

Add .clang-format style

From http://code.qt.io/cgit/qt/qtrepotools.git/tree/config/_clang-format

[tx-robot] updated from transifex

Selective sync: Skip excluded folders when reading db

When a new folder becomes selective-sync excluded, we already mark it
and all its parent folders with _invalid_ etags to force rediscovery.

That's not enough however. Later calls to csync_statedb_get_below_path
could still pull data about the excluded files into the remote tree.

That lead to incorrect behavior, such as uploads happening for folders
that had been explicitly excluded from sync.

To fix the problem, statedb_get_below_path is adjusted to not read the
data about excluded folders from the database.

Currently we can't wipe this data from the database outright because we
need it to determine whether the files in the excluded folder can be
wiped away or not.

See owncloud/enterprise#1965

[tx-robot] updated from transifex

Wizard: Skip result page with acceptance instead of closing #5764

[tx-robot] updated from transifex

Promote a few more logs to info/warning

macOS: Fix usage of qCDebug in .mm file

Add the possiblility to enable debug logging categories

Add a checkbox in the log window as well as a --logdebug command-line
option that should have the same effect.

Issue #5647

Also pass csync logs through QMessageLogger

This allows using logging rules and custom message patterns with csync.
This also pass the a relative verbosity to the message logger.

Issue #5647

Upgrade some qCDebug to qCInfo or qCWarning

Use qCInfo for anything that has general value for support and
development. Use qCWarning for any recoverable error and qCCritical
for anything that could result in data loss or would identify a serious
issue with the code.

Issue #5647

Use Qt logging categories for logging

This gives more insight about the logs and allow setting fine-tuned
logging rules. The categories are set to only output Info by default
so this allows us to provide more concise logging while keeping the
ability to extract more information for a specific category when
developping or debugging customer issues.

Issue #5647

Improve the logging message pattern

Add the log level and category name in the output. Only output the
thread ID and function name for qCDebug statements as they are not
necessary for general use and make the log harder to read.

Also make sure that the message pattern is set when NO_MSG_HANDLER is
used. Using an environment variable should have priority over it anyway.

Remove the unused folderwatcher_qt implementation

Add explanation of how to replace local sync folder to FAQ

This fixes owncloud/documentation/issues/2923.

(cherry picked from commit 6580af65c1cd02ed632522f7ed128dfda3200be7)

ChangeLog: 2.4 until now

Detect maintenance mode #4485

When we first detect a 503 (probably from a PROPFIND) and enter the
ServiceUnavailable state, we new trigger a status.php query that will
switch the state to MaintenanceMode if necessary.

[tx-robot] updated from transifex

Sharing: Fix UI glitch when sharing is disabled

Sharing: Optionally show warning that links are public

For https://github.com/owncloud/enterprise/issues/1914

[tx-robot] updated from transifex

Merge branch 2.3 into master

Update release_template.md

VERSION.cmake: This is 2.3.3 now

Remove stray debug output

[tx-robot] updated from transifex

Use QTemporaryDir in TestOwnSql

This will improve the cleanup and possibly help for #5366.

Jenkinsfile: Also use 4 parallel jobs for linux builds

Favor QtKeychain installed together with Qt

By default we will look in /usr/lib, which would prevent us from
using a different path for Qt as QtKeychain depends on Qt itself.

Update release_template.md

Update release_template.md