Merge pull request #23077 from nextcloud/version/19.0.4/RC1

19.0.4 RC1

Merge pull request #22784 from nextcloud/backport/22234/stable19

[stable19] Use user mount with matching shared storage only

Merge pull request #22941 from nextcloud/backport/22395/stable19

[stable19] Fix settings chunk loading

Merge pull request #23086 from nextcloud/backport/22983/stable19

[stable19] Sync all users to the system addresssbook

Merge pull request #23087 from nextcloud/backport/23013/stable19

[stable19] Show federation and email results also with exact user match unless c…

Merge pull request #23090 from nextcloud/backport/23074/stable19

[stable19] Do not match sharees on an empty email address

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #23093 from nextcloud/backport/23083/stable19

[stable19] Generate exception to log on php errors

Generate exception to log on php errors

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Do not match sharees on an empty email address

When asking for sharees we compare not only UID and displayname but also
the email address. And if that matches we return the sharee as an exact
match. This logic had a flaw as in that it also matched the empty string
to users with no email address.

This is most noticeable when you disable sharee enumeration and open the
ownership transfer dialog. It suggested other users of the instance
before. This has stopped now.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Show federation and email results also with exact user match unless containing @

Before when you have a user "smith" and a federated user "smith@example.com"
you could see the federation result with "smit" but not with "smith" anymore.
With most LDAP configurations and local backend setups this is disturbing and
causes issues.
The idea of not showing the email and federation on a matching user was with:
Local user registered with "smith@example.com" user id and having that same
email / cloud id in your contacts addressbook. So we now only hide those
"side results" when the search does contain an @

Signed-off-by: Joas Schilling <coding@schilljs.com>

Sync all users to the system addresssbook

Signed-off-by: Joas Schilling <coding@schilljs.com>

[tx-robot] updated from transifex

Merge pull request #22804 from nextcloud/backport/stable19/22116-22648-22761

[stable19] Fix share transfer of single files and on the transfered node

Merge pull request #22904 from nextcloud/backport/22636/stable19

[stable19] Make sure that getConfig is still called for browsers that do not support CSPv3

Merge pull request #22988 from nextcloud/backport/22533/stable19

[stable19] Flow: do not hide "matches" and "does not match" checkers

Merge pull request #23002 from nextcloud/backport/22940/stable19

[stable19] Never copy the share link when the password is forced

Never copy the share link when the password is forced

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

19.0.4 RC1

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Merge pull request #23049 from nextcloud/backport/23034/stable19

[stable19] Fix numeric folders throwing on markDirty

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fix numeric folders throwing on markDirty

TypeError: strpos() expects parameter 1 to be string, int given

The problem is that in cacheNode() we strip of any slashes, so
a folder "0/" will be trimmed to "0" and be used as an array key.
Since PHP automatically casts numeric array keys to integers,
you afterwards get $nodePath as int(0). Since it's now a number,
the strpos() function does not accept it anymore. Simply casting
$nodePath to a string again in the foreach solves the issue

Signed-off-by: Joas Schilling <coding@schilljs.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

do not hide "matches" and "does not match" checkers

- they show up previously with / typed – end user needs to know about it
- inconsistent with some other fields, that show it directly

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Add integration tests to check that only the given path is transferred

Until recently (it was fixed in ac2999a26a) when a path was transferred
other shares with the target user were removed, so a test was added to
ensure that it does not happen again.

Besides that a test to ensure that other files with the target user are
not transferred was added too (it did not fail before, but seemed
convenient to have that covered too :-) ).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Add integration tests for transferring files of a user with a risky name

The files:transfer-ownership performs a sanitization of users with
"risky" display names (including characters like "\" or "/").

In order to allow (escaped) double quotes in the display name the
regular expression used in the "user XXX with displayname YYY exists"
step had to be adjusted.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Add integration test for transferring the path of a single file

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Fix settings chunk loading

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Merge pull request #22926 from nextcloud/backport/22915/stable19

[stable19] improve handling of out of space errors for smb

[tx-robot] updated from transifex

improve handling of out of space errors for smb

Signed-off-by: Robin Appelman <robin@icewind.nl>

Add integration tests for transferring ownership of reshares

Currently only transferring ownership of a reshare with a group to a
user in the group is possible.

When transferring ownership of a reshare with another user or with a
group to a user not in the group restoring the share fails (but the
command succeeds, it only fails for the specific files that are
reshares).

When transferring ownership of a path that is a reshare the command
fails (as when a specific path is provided the path tries to move the
file, it does not take into account reshares).

The added integration tests reflect the above behaviours.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Check share ownership after transferring file ownership

The integration tests did not verify that the shares were actually
transferred between the users (or that they were removed due to being
transferred to the sharee).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Check whether file exists or not after transferring ownership

The integration tests did not verify that the files were actually
transferred between the users, only that the files were downloadable.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Transfer shares if no path provided

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Transfer shares of the transferred root node

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #22899 from nextcloud/backport/21894/stable19

[stable19] fix: file quota was not applied in all cases

Make sure that getConfig is still called for browsers that do not support CSPv3

Signed-off-by: Julius Härtl <jus@bitgrid.net>

fix: file quota was not applied in all cases

[tx-robot] updated from transifex

Merge pull request #22893 from nextcloud/backport/22880/stable19

[stable19] Revoke secsignid

Merge pull request #22892 from nextcloud/backport/22744/stable19

[stable19] only get the permissions from the share source if it's not already cached

Revoke secsignid

See nextcloud/app-certificate-requests#343

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

only get the permissions from the share source if it's not already cached

this prevents having to setup the share source when doing metadata stuff on the share root

Signed-off-by: Robin Appelman <robin@icewind.nl>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #22840 from nextcloud/backport/22770/stable19

[stable19] Mitigate encoding issue with user principal uri

Mitigate encoding issue with user principal uri

Signed-off-by: Georg Ehrke <developer@georgehrke.com>

Merge pull request #22775 from nextcloud/fix/noid/stable19-ldap-group-update-events

[stable19] makes LDAP's group updater dispatch type events

Merge pull request #22778 from nextcloud/backport/22739/stable19

[stable19] Don't fail if copying a file of 0 byte size

Merge pull request #22776 from nextcloud/backport/22747/stable19

[stable19] fix s3 doesDirectoryExist check for empty directories

Merge pull request #22591 from nextcloud/backport/22577/stable19

[stable19] Better error message when blocked by access control

Use user mount with matching shared storage only

Signed-off-by: Julius Härtl <jus@bitgrid.net>

makes LDAP's group updater dispatch type events

* fixes group share updates on added users

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Annotate that writeStream can throw a GenericFileException

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Properly catch exception from writing to stream when copying a file

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Set delimiter so that the CommonPrefixes response can properly be determined

Signed-off-by: Julius Härtl <jus@bitgrid.net>

fix s3 doesDirectoryExist check for empty directories

if a directory is empty, only the 'marker' object `/` exists.

since not all s3 implementations return just the prefix when listing objects by prefix,
when listing objects by the folder prefix, nothing will be returned if the directory is empty.
by not including the trailing slash in the prefix, the folder marked will always be returned if it exists

Signed-off-by: Robin Appelman <robin@icewind.nl>

Merge pull request #22769 from nextcloud/backport/22766/stable19

[stable19] Clarify PHP warning in admin settings

Clarify PHP warning in admin settings

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

Merge pull request #22752 from nextcloud/backport/22587/stable19

[stable19] Check if var debugMode exists

Merge pull request #22753 from nextcloud/backport/22651/stable19

[stable19] Fix reading empty files from objectstorage

Merge pull request #22630 from nextcloud/backport/22583/stable19

Fix reading empty files from objectstorage

Since we try to do range requests this will fail hard.
However since empty files are not that interesting to read anyways we
just read from an emptry memory stream.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Check if var debugMode exists

Closes #21150 a second time.
2nd appearance of debugMode may not seen in first fix.

Merge pull request #22742 from nextcloud/backport/22589/stable19

[stable19] Allow additional personal settings via normal registration

Merge pull request #22745 from nextcloud/backport/22646/stable19

[stable19] Fix detecting text/x-php mimetype and secure mimetype mapping

Merge pull request #22652 from nextcloud/backport/22643/stable19

[stable19] Fix installing on Oracle

Merge pull request #22713 from nextcloud/backport/stable19/22557

[stable19] Do not fail if share for mountpoint is no longer available

Merge pull request #22720 from nextcloud/backport/22469/stable19

[stable19] Don't use SELECT DISTINCT when to_char() is used in a WHERE statement

Add text/x-php as an alternative for application/x-php

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Properly add both mimetypes to secure mimetype mapping

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Merge pull request #22725 from nextcloud/version/19.0.3/final

19.0.3 final

Allow additional personal settings via normal registration

Signed-off-by: Joas Schilling <coding@schilljs.com>

19.0.3 final

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

Don't use SELECT DISTINCT when to_char() is used in a WHERE statement

Nice bug in Oracle

Signed-off-by: Joas Schilling <coding@schilljs.com>

Do not fail if share for mountpoint is no longer available

Signed-off-by: Julius Härtl <jus@bitgrid.net>

Warn about adding NotNull columns with nullable default

Signed-off-by: Joas Schilling <coding@schilljs.com>

Fix installing on Oracle

Empty strings are stored as null in Oracle,
so a column with NotNull can not have an empty string as default

Signed-off-by: Joas Schilling <coding@schilljs.com>

add repair step to clean up DB off lastFeatureRefresh entries in user prefs

- also removes related app setting "updateAttributesInterval"

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

[tx-robot] updated from transifex

these code bits were part of old logic that was already refactored out

- only references were in unit tests

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #22574 from nextcloud/version/19.0.3/rc1

19.0.3 RC1

Better error message when blocked by access control

Signed-off-by: Joas Schilling <coding@schilljs.com>

Merge pull request #22515 from nextcloud/backport/22472/stable19

[stable19] Fix writing BLOBs to postgres with recent contacts interaction

Merge pull request #22579 from nextcloud/backport/22550/stable19

[stable19] Use the correct root to determinate the webroot for the resource

Merge pull request #22442 from nextcloud/backport/22421/stable19

[stable19] show better quota warning for group folders and external storage

Merge pull request #22581 from nextcloud/backport/22578/stable19

[stable19] Upgrade icewind/smb to 3.2.7

Upgrade icewind/smb to 3.2.7

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>