Minor fix in text of warning and debug message.

Signed-off-by: Camila <hello@camila.codes>

Make sure we don't assert when calling fileStatus

It happens that sometimes we leak a directory path ending with a slash,
but that violates fileStatus' precondition so let's catch it early and
skip such path.

Of course the right fix would be a larger swipe in the sync engine and
around it to not use naked strings anymore but rely on the typesystem.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2271 from ivaradi/cstdint-fix

Include cstdint only when compiling C++ code

Merge branch 'master' into cstdint-fix

[tx-robot] updated from transifex

Include cstdint only when compiling C++ code

Signed-off-by: István Váradi <ivaradi@varadiistvan.hu>

[tx-robot] updated from transifex

Merge pull request #2268 from nextcloud/cherry_pick_testchunkingng_related_improvements

Cherry pick TestChunkingNG related improvements

Size cleanups in TestChunkingNG

Some of the comments didn't match the size or were missing. This also
means reducing one of the 150 MB payloads left behind reducing the
execution time by a few more seconds. This is now around 30s execution
time which is more acceptable.

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

Upload: Store the size in the UploadInfo, and compare it when resolving potential conflict

This is about the conflicts that happens when the file has been uploaded
correctly to the server, but the etag was not recieved because the connection
was closed before we got the reply.

We used to compare only the mtime when comparing the uploaded file and the
existing file.  However, to be perfectly correct, we also should check the
size.

This was found because TestChunkingNG::connectionDroppedBeforeEtagRecieved is
flaky. Example of faillure found in https://drone.owncloud.com/owncloud/client/481/5
while testing PR #6626

(very trimmed log:)

06-29 07:58:02:015 [ info sync.csync.csync ]: ## Starting local discovery ##
06-29 07:58:02:016 [ info sync.csync.updater ]: Database entry found, compare: 1530259082 <-> 1530259051, etag:  <-> 1644a8c8750, inode: 1935629 <-> 1935629, size: 301 <-> 300, perms: 0 <-> ff, type: 0 <-> 0, checksum:  <-> SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1, ignore: 0
06-29 07:58:02:016 [ info sync.csync.updater ]: file: A/a0, instruction: INSTRUCTION_EVAL <<=
06-29 07:58:02:972 [ warning sync.networkjob ]: QNetworkReply::NetworkError(OperationCanceledError) "Connection timed out" QVariant(Invalid)
.. next sync...
06-29 07:58:02:980 [ info sync.engine ]: #### Discovery start ####################################################
06-29 07:58:02:981 [ info sync.csync.csync ]: ## Starting local discovery ##
06-29 07:58:02:983 [ info sync.csync.updater ]: Database entry found, compare: 1530259082 <-> 1530259051, etag:  <-> 1644a8c8750, inode: 1935629 <-> 1935629, size: 302 <-> 300, perms: 0 <-> ff, type: 0 <-> 0, checksum:  <-> SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1, ignore: 0
06-29 07:58:02:983 [ info sync.csync.updater ]: file: A/a0, instruction: INSTRUCTION_EVAL <<=
06-29 07:58:02:985 [ info sync.csync.csync ]: ## Starting remote discovery ##
06-29 07:58:02:985 [ info sync.networkjob ]: OCC::LsColJob created for "http://localhost/owncloud" + "" "OCC::DiscoverySingleDirectoryJob"
06-29 07:58:02:987 [ info sync.csync.updater ]: Database entry found, compare: 1530259082 <-> 1530259051, etag: 1644a8c8b26 <-> 1644a8c8750, inode: 0 <-> 1935629, size: 301 <-> 300, perms: ff <-> ff, type: 0 <-> 0, checksum: SHA1:5adcdac9608ae0811247f07f4cf1ab0a2ef99154 <-> SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1, ignore: 0
06-29 07:58:02:987 [ info sync.csync.updater ]: file: A/a0, instruction: INSTRUCTION_EVAL <<=
06-29 07:58:02:989 [ info sync.csync.csync ]: Update detection for remote replica took 0.004 seconds walking 13 files
06-29 07:58:02:990 [ info sync.engine ]: #### Discovery end ####################################################  9 ms
06-29 07:58:02:990 [ info sync.database ]: Updating file record for path: "A/a0" inode: 1935629 modtime: 1530259082 type: 0 etag: "1644a8c8b26" fileId: "16383ea4" remotePerm: "WDNVCKR" fileSize: 301 checksum: "SHA1:cc9adedebe27a6259efb8d6ed09f4f2eff559ad1"
06-29 07:58:02:990 [ info sync.csync.reconciler ]: INSTRUCTION_UPDATE_METADATA    client file: A/a0
06-29 07:58:02:990 [ info sync.csync.csync ]: Reconciliation for local replica took  0 seconds visiting  13  files.
06-29 07:58:02:990 [ info sync.csync.reconciler ]: INSTRUCTION_UPDATE_METADATA    server dir:  A
06-29 07:58:02:990 [ info sync.csync.csync ]: Reconciliation for remote replica took  0 seconds visiting  13  files.
06-29 07:58:02:990 [ info sync.engine ]: #### Reconcile end ####################################################  9 ms
06-29 07:58:02:990 [ info sync.database ]: Updating local metadata for: "A/a0" 1530259082 302 1935629
FAIL!  : TestChunkingNG::connectionDroppedBeforeEtagRecieved(small file) '!fakeFolder.syncOnce()' returned FALSE. ()

TestChunkingNG: Make it faster

Use smaller files so the test run faster.
Particulary usefull for TestChunkingNG::connectionDroppedBeforeEtagRecieved
Which had become so much slower after 2638332dc63cd79410b6fea5a24449696d63d3f2
increased the timeout for bigger files

Upload: Add more chunk resuming unit tests #6405

Merge pull request #2269 from nextcloud/enable_modernize_loop_convert

Enable the modernize-loop-convert check on clang-tidy

Enable the modernize-loop-convert check on clang-tidy

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

Remove unused method

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

Enable the modernize-deprecated-headers check on clang-tidy

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

Enable the modernize-use-equals-default check on clang-tidy

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

[tx-robot] updated from transifex

Enable the modernize-user-nullptr check on clang-tidy

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

Merge pull request #2264 from nextcloud/enable_modernize_use_using

Enable the modernize-use-using check on clang-tidy

Get rid of useless using constructs

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

Make QTokenizer use simpler by having optional second parameter

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

Enable the modernize-use-using check on clang-tidy

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

Merge pull request #2262 from nextcloud/enable_trivial_clang_tidy_modernize_checks

Enable "trivial" modernize clang-tidy checks

Enable "trivial" modernize clang-tidy checks

This is a bunch of modernize checks which we don't hit at all. This is
an easy win and also make sure those old constructs won't creep up
behind our back.

Note that some of them won't trigger until we bump our C++ baseline
standard to C++17. But I'd say that's OK, that'll force our hand to do
those C++17 related cleanups when the time comes.

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

Merge pull request #2037 from nextcloud/enable_modernize_raw_string_literal

Enable modernize raw string literal

We don't really need two \ here

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

Add more raw string literals missed previously

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

Merge pull request #2247 from nextcloud/improve_drone_config

Improve drone config

Make sure wget looks for the certificates at the right place

Might happen because we got too many copies of OpenSSL around

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

Switch to Ninja for the clang build

Clang is generally slower to compile than GCC so use a faster Make for
that build variant. This also allows us to verify our CMake files don't
do anything Make specific.

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

Parallelize the clang-tidy runs

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

Split cmake and compile into separate pipeline steps

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

Switch to actual commands inside of the steps

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

Split the CI pipelines into proper steps

Will make it easier to navigate the logs

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

Now make sure we're not picking up generated headers

This could happen (and started to happen more with clang-tidy 10) that
clang-tidy picked up errors in headers from the build directory. Now
that we moved the build directory out of source, we can simply filter
headers based on the pwd.

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

Move the build in a volume to be reused between steps

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

Remove unneeded environment variables

Those are set by the base CI image anyway, no need to do the work again.

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

Switch to the newer base CI image

We can now spare building QtKeychain over and over shortening a bit the
build. We also update to latest gcc and clang.

Note that I didn't switch the AppImage pipeline to the newer image to
keep the binary compatibility promise required by linuxdeployqt.
Probably would make sense to do a specific image for AppImage with some
other of the cleanups I did there (in particular to avoid building
qtkeychain every time).

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

Parallelize the CI build

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

[tx-robot] updated from transifex

Merge pull request #2258 from nextcloud/have_a_way_to_display_e2ee_mnemonic

Keep the E2EE info message around to allow displaying mnemonic

Keep the E2EE info message around to allow displaying mnemonic

We lost the ability to display the E2EE mnemonic during the GUI
redesign and the info message wasn't displayed again on restart. So now
we display it every time, it still can be dismissed and the button text
is different in such a case to make the intent clearer.

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

[tx-robot] updated from transifex

Merge pull request #2255 from nextcloud/restore_e2ee_old_key_storage_format_compatibility

Restore E2EE old key storage format compatibility

Add support for BASE64 encoded '|' when decrypting

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

Move the private key salt handling in its own function

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

Ignore the salt part of the key during decryption

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

Restore the symmetry between *StringSymmetric functions

If we receive data without base64 encoding for encryption, it makes
sense to get it without base64 encoding out of decryption.

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

Add unit tests for the encryption helpers

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Master is now what will become 3.1

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

[tx-robot] updated from transifex

Merge pull request #2248 from nextcloud/bump_version_to_3.0

There won't be a 2.7, this is called 3.0 instead

There won't be a 2.7, this is called 3.0 instead

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2245 from nextcloud/fix-issue-2243

Fix #2243

Remove extra check for talk app on server in AccountState.

- Remove repeated hard coded "spreed" string.

Signed-off-by: Camila <hello@camila.codes>

Fix #2243: use server url instead of hard coded string.

Signed-off-by: Camila <hello@camila.codes>

Make sure we always have a CLSID when Explorer integration is used

We update properly the CLSID of folders to integrate them with the
Explorer pane when they are added or removed from the GUI. That said,
there might be cases of users adding folders to sync straight from the
GUI. In such a case the CLSID might be missing on config load so now we
assign one on the fly when that happens.

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

[tx-robot] updated from transifex

Windows: Enable CFG and SafeSEH linker security flags

See:
https://docs.microsoft.com/en-us/windows/win32/secbp/control-flow-guard
https://docs.microsoft.com/en-us/cpp/build/reference/safeseh-image-has-safe-exception-handlers

Signed-off-by: Michael Schuster <michael@schuster.ms>

Always show Settings in the context menu (amends #2164)

The context menu should allow access to the Settings even when no accounts are configured.

Users may specify proxy / startup / update and other settings at any time.

This slipped through in #2164.

Signed-off-by: Michael Schuster <michael@schuster.ms>

[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 pull request #2227 from nextcloud/support_png_for_branded_clients_systray

Support png for branded clients systray

Use PNGs for branded clients

For branded clients try to load a png of the right size, for unbranded
clients use the SVGs. If something fails at load time skip it.

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

Move the svg rendering part in a lambda

This simplifies the loop body a bit, opening the door to what's next.

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

Extract the svg renderer from the loop

No need to recreate it each time for nothing.

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

Branded clients have more size variety for their icons

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

Switch to for range loop

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

Let Theme tell us if we're branded or not

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

Add OpacityMask to fix mouse hover on round corner.

- Fix for #2173.
- Removed the workaround for the left round corner which was also
fixed with the OpacityMask.

Signed-off-by: Camila <hello@camila.codes>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Let context menu open wizard if there are no accounts

The context menu offers to open the main dialog and the settings
even if no accounts are configured.  In this case, the main dialog
is useless and the settings are probably confusing.  Hence, this
commit replaces these actions in the context menu by an action
to open the wizard (which also opens on left click, so this is
the most natural thing to do).

Signed-off-by: Stephan Beyer <s-beyer@gmx.net>

[tx-robot] updated from transifex

Merge pull request #2224 from nextcloud/qt-5.12.9-update

Update CI for Qt 5.12.9

Update CI for Qt 5.12.9

Also remove the jq package installation from the AppImage build script because
the package is included in the new Docker image (see nextcloud/docker-ci#231).

Signed-off-by: Michael Schuster <michael@schuster.ms>

[tx-robot] updated from transifex

Merge pull request #2209 from nextcloud/fix-pause-sync-states

Fix pause sync states

sigh, too

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Get a const auto for the FolderMap to iterate over

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Replaced Qt foreach by C++11 ranged for loop

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Properly set syncPause state on Systray creation when account init is done

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Fixed copy/paste bug, resulting in a wrong signal ::pauseSync used instead of ::resumeSync

Signed-off-by: Dominique Fuchs <32204802+DominiqueFuchs@users.noreply.github.com>

Fix Windows installer graphics files for 2.7

For some reason the exported bitmaps in #2211 were not properly saved in the right
format for display by the NSIS installer system.

Signed-off-by: Michael Schuster <michael@schuster.ms>