Merge pull request #26525 from nextcloud/backport/26433/stable21

[stable21] Limit size of properties to 2048 characters

Limit size of properties to 2048 characters

It is unreasonable to expect that one of these fields would be longer
than 2048 characters. Whilst some have definitely lower limits (such as
for phone numbers or domain names), a upper bound as sanity check makes
sense.

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Merge pull request #26514 from nextcloud/backport/26439/stable21

[stable21] Increase subnet matcher

Increase subnet matcher

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>

Merge pull request #26508 from nextcloud/backport/26230/stable21

[stable21] l10n: Add word user in FederatedShareProvider.php

l10n: Add word user in FederatedShareProvider.php

Standardizing text strings with other Nextcloud applications.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #26453 from nextcloud/version/21.0.1/final

21.0.1 final

Merge pull request #26459 from nextcloud/backport/26458/stable21

[stable21] Show icon-phone when setting is set to private instead of local

Merge pull request #26447 from nextcloud/3rdparty/stable21/phpseclib-2.0.31

[3rdparty][stable21] phpseclib-2.0.31

Show icon-phone when setting is set to private instead of local

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

[3rdparty][stable21] phpseclib-2.0.31

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

Merge pull request #26451 from nextcloud/revert-26326-backport/26070/stable21

Revert "[stable21] add a prefix index to filecache.path"

21.0.1 final

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

Revert "[stable21] add a prefix index to filecache.path"

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #26400 from nextcloud/backport/26061/stable21

[stable21] Log and continue when failing to update encryption keys during for individual files

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #26413 from nextcloud/dependabot/npm_and_yarn/stable21/y18n-4.0.1

[Security] Bump y18n from 4.0.0 to 4.0.1

Bumps [y18n](https://github.com/yargs/y18n) from 4.0.0 to 4.0.1. **This update includes a security fix.**
- [Release notes](https://github.com/yargs/y18n/releases)
- [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yargs/y18n/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Merge pull request #26406 from nextcloud/backport/26396/stable21

[tx-robot] updated from transifex

Merge pull request #26313 from nextcloud/version/21.0.1/RC1

21.0.1 RC1

Merge pull request #26402 from nextcloud/ldap-factory-no-ldap-21

[21] make ILDAPProviderFactory usable when there is no ldap setup

Merge pull request #26404 from nextcloud/backport/26403/stable21

[stable21] remove leftover debug @NoCSRFRequired introduced with #26198

Properly handle creating the template directory in a subfolder (fixes #25787)

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

Get the parent directory before creating a file from a template

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

remove leftover debug @NoCSRFRequired introduced with #26198

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

make ILDAPProviderFactory usable when there is no ldap setup

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

Merge pull request #26398 from nextcloud/backport/26394/stable21

[stable21] Allow apps to write/update account data

Fix phpcs issues in encryption update

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Log and continue when failing to update encryption keys during for individual files

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

Merge pull request #26391 from nextcloud/backport/26386/stable21

[stable21] Also check the default phone region when the number has no country code

Allow apps to write/update account data

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

[tx-robot] updated from transifex

Also check the default phone region when the number has no country code

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

Merge pull request #26377 from nextcloud/backport/25500/stable21

[stable21] fix(translation): replace static error message

Merge pull request #26376 from nextcloud/backport/25804/stable21

[stable21] Do not attempt to read 0 bytes when manually iterating over a non-seekable file

Merge pull request #26379 from nextcloud/backport/25924/stable21

[stable21] Only mark migrations as installed after execution

Merge pull request #26382 from nextcloud/backport/26372/stable21

[stable21] gracefully handle deleteFromSelf when share is already gone

[tx-robot] updated from transifex

gracefully handle deleteFromSelf when share is already gone

- handling race conditions

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

Only mark migrations as installed after execution

The problem is that if a developer creates a structural error in their migration file,
they will already be marked as executed and an not be rerun.

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

revert(comment)

Signed-off-by: Samuel <faust64@gmail.com>

revert(exception): keep throwing an exception for now

Signed-off-by: Samuel <faust64@gmail.com>

fix(translation): replace static error message

as suggested by @kesselb in nextcloud/server#25490

Signed-off-by: Samuel <faust64@gmail.com>

Properly end iterating over the file once the target has been reached

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

Merge pull request #26371 from nextcloud/backport/26370/stable21

[stable21] Remove notifications when retesting profile field input

Merge pull request #26363 from nextcloud/backport/26325/stable21

[stable21] Update cipher defaults

Remove notifications when retesting

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

Merge pull request #26366 from nextcloud/backport/26355/stable21

[stable21] Fix working for phone number integration

[tx-robot] updated from transifex

Merge pull request #26326 from nextcloud/backport/26070/stable21

[stable21] add a prefix index to filecache.path

Fix working for phone number integration

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

Merge pull request #26357 from nextcloud/backport/26342/stable21

[stable21] Fix broken Calendar Event Invite email icons in Gmail by using PNGs instead of SVGs

Update cipher defaults

Signed-off-by: J0WI <J0WI@users.noreply.github.com>

change calendar invite emails to use pngs instead of svgs

Signed-off-by: Nick Sweeting <git@sweeting.me>

Merge pull request #26352 from nextcloud/backport/26243/stable21

[stable21] Avatar privacy and new scope

Capability for federated scope

Added additional capability in the provisioning API to signal whether
the federation scope values can be used.

This is based on whether the lookup server upload is enabled or not.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Update autoloader for PlaceholderAvatar

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Int tests for provisioning API scopes

Added integration tests for the scope attributes in the provisioning
API.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Add known user check in avatar when v2-private scope

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Update psalm-baseline for Avatar API quirks

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Change account property capability

Include version number in capability

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com>

Added PlaceholderAvatar with own cached images

When avatar scope is private, the PlaceholderAvatar is used to deliver a
placeholder avatar based on the user's initials.

This was implemented as a separate class for now to avoid messing with
the existing UserAvatar implementation and its generated vs
non-generated logic.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Add capability for editable scopes in provisioning API

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Add property scope tests for AccountManager

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Enhance UsersControllerTest of provisioning API with scopes

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Expose avatarScope through provisioning API

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

OCS allow reading and writing account property scopes

Extends the provisioning API to allow a user to get and set their own
account property scopes.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Make extra user profile fields always editable

The fields for phone number, address, website and twitter are now
editable regardless whether federated sharing and the lookup server
are enabled or not.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Map old account scope properties to new names

Use new scope values in settings page.
Adjust all consumers to use the new constants.
Map old scope values to new ones in account property getter.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Adjust scopes menu based on conditions

Now not all fields have the "v2-private" option in place.
Fix dropdown issue when a scope was stored that is not listed after
disabling the lookup server.
Whenever the lookup server upload is disabled, the scope menu is now
displayed where it makes sense to allow switching between the two private
scopes.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

Add new v2-private account scope

Added new v2-private account manager scope that restricts the scope
further by excluding public link access.

Avatars with v2-private account scope are now showing the guest avatar
instead of the real avatar.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

add a prefix index to filecache.path

The reason that `filecache.path` hasn't had an index added is the mysql limitation of ~1kb for indexeded fields,
which is to small for the `path`, however mysql supports indexing only the first N bytes of a column instead of the entire column,
allowing us to add an index even if the column is to long.

Because the index doesn't cover the entire column it can't be used in all situations where a normal index would be used, but it does cover the `path like 'folder/path/%'` queries that are used in various places.

Sqlite and Postgresql don't support prefix indexes, but they also don't have the 1kb limit and DBAL handles the differences in index creation.

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

Merge pull request #26307 from nextcloud/backport/22201/stable21

[stable21] Delete old birthday calendar object when moving contact to another ad…

Merge pull request #26291 from nextcloud/backport/21484/stable21

[stable21] show better error messages when a file with a forbidden path is encountered

Merge pull request #26263 from nextcloud/backport/26046/stable21

[stable21] update icewind/smb to 3.4.0

21.0.1 RC1

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

Delete old birthday calendar object when moving contact to another address book

When an contact is moved to another address book, the contact is copied to
the second address book.
During copying, the birthday event is created - but it gets the same UID
as the contact's birthday event in the first address book.

To prevent the "Calendar object with uid already exists" error that followed,
we need to delete the old entry before the new one is created.

Resolves: https://github.com/nextcloud/server/issues/20492
Signed-off-by: Christian Weiske <cweiske@cweiske.de>

Merge pull request #26298 from nextcloud/backport/26287/stable21

[stable21] Fix l10n

[tx-robot] updated from transifex

Merge pull request #26301 from nextcloud/backport/26299/stable21

[stable21] Log when a storage is marked as unavailable

Log when a storage is marked as unavailable

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

Fix l10n

Issue: #26286

show better error messages when a file with a forbidden path is encountered

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

[tx-robot] updated from transifex

Merge pull request #26257 from nextcloud/backport/26198/stable21

[stable21] Handle limit offset and sorting in files search

Merge pull request #26279 from nextcloud/backport/26255/stable21

[stable21] l10n: Separate ellipsis

Merge pull request #26276 from nextcloud/backport/26265/stable21

[stable21] Fix casing of core test folder, bring back missing tests