Use \OC::$CLI instead of PHP_SAPI

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

Fix password reset saying Admin changed my password when reset from
login page

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

Merge pull request #21201 from nextcloud/backport/21181/stable18

[stable18] Do not only catch Exceptions but any Throwable during rmt share delete

Merge pull request #21184 from nextcloud/version/18.0.5/final

18.0.5

[tx-robot] updated from transifex

Do not only catch Exceptions but any Throwable during rmt share delete

When a remote share is deleted, we notify the remove instance. This may
trigger various error conditions, which we want to catch in order to
avoid undeletable shares. The try-catch, however, did only capture
exceptions, so things like TypeErrors were not caught and caused the
process to fail hard.

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

[tx-robot] updated from transifex

18.0.5

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 #21159 from nextcloud/revert-21152-backport/21143/stable18

Revert "[stable18] Fix password changes in link and mail shares"

Revert "[stable18] Fix password changes in link and mail shares"

Merge pull request #21152 from nextcloud/backport/21143/stable18

[stable18] Fix password changes in link and mail shares

Add integration tests for video verification

Enabling the "send password by Talk" property of shares require that
Talk is installed and enabled, so the Drone step that runs them has to
first clone the Talk repository.

When the integration tests are run on a local development instance,
however, it is not guaranteed that Talk is installed. Due to this the
"@Talk" tag was added, which ensures that any feature or scenario marked
with it will first check if Talk is installed and, if not, skip the
scenario (instead of failing).

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

Add integration tests for creating and updating a mail share

In most cases, when a mail share is created or updated an e-mail is sent
to the sharee, which is done by connecting to the SMTP server set in the
configuration. If the server can not be contacted then the creation or
update of the mail share fails.

To make possible to test mail shares without using a real SMTP server a
fake one has been added. The original script, which is MIT licensed, was
based on inetd, so it was slightly modified to run on its own.

In order to use it from the integration tests the "Given dummy mail
server is listening" step has to be called in the scenarios in which the
mail server is needed.

For now that is the only available step; things like checking the sent
mails, while possible (as the script can log the mails to certain file),
have not been added yet.

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

Generalize integration test steps to download last share

Note that the "last link share can be downloaded" step was kept as it
tests the "url" property specific of link shares.

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

Remove unused variables

The step names were adjusted accordingly.

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

Fix disabling send password by Talk without new password in mail shares

When "send password by Talk" was disabled in a mail share it was
possible to keep the same password as before, as it does not pose any
security issue (unlike keeping it when "send password by Talk" is
enabled, as in that case the password was already disclosed by mail).

However, if a mail share is updated but the password is not set again
only the hashed password will be available. In that case it would not
make sense to send the password by mail, so now the password must be
changed when disabling "send password by Talk".

Note that, even if explicitly setting the same password again along with
the "send password by Talk" property would work, this was also prevented
for simplicity.

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

Fix enabling send password by Talk with same password in mail shares

When "send password by Talk" is enabled in a mail share a new password
must be also set. However, when the passwords of the original and the
new share were compared it was not taken into account that the original
password is now hashed, while the new one is not (unless no new password
was sent, in which case the password of the original share was set in
the new share by the controller, but that was already prevented due to
both passwords being literally the same), so it was possible to set the
same password again.

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

Fix enabling send password by Talk with empty password in link shares

When "send password by Talk" is enabled in a link share now a non empty
password is enforced.

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

Fix creating a mail share with a password

When a mail share was created with a password the given password was not
hashed, so it was not possible to open the share with that password.
Moreover, if passwords were enforced the given password was ignored and
a new one was set (although in this case it was hashed so it worked as
expected). Now the given password is properly hashed and not overriden.

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

Extend mail shares unit tests to check the password and mail template

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

[tx-robot] updated from transifex

Merge pull request #21061 from nextcloud/version/18.0.5/RC1

18.0.5 RC1

Merge pull request #21109 from nextcloud/backport/19793/stable18

[stable18] Fix resharing of federated shares that were created out of links

Merge pull request #21127 from nextcloud/fix/make-translation-sanitization-optional-stable18

[stable18] Make the translation sanitization optional

Merge pull request #21132 from nextcloud/backport/20933/stable18

[stable18] Move the password confirmation form template to post

[tx-robot] updated from transifex

Move the password confirmation form template to post

I know we use JS for this. But it triggers some false positives on
testing.

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

Fix devices & sessions sanitization

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

Make the translation sanitization optional

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

Merge pull request #21115 from nextcloud/backport/21106/stable18

[stable18] use the loginname to verify the old password in user password changes

[tx-robot] updated from transifex

use the loginname to verify the old password in user password changes

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

Merge pull request #21096 from nextcloud/backport/21090/stable18

[stable18] Do not read certificate bundle from data dir by default

Fix federated link sharing permissions

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Fix share update test

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

Update public link share permission code

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

Have share permissions on link shares if it is enabled

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

Adjust integration tests to new permissions

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

Remove unneeded test since links have resharing permissions by default

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

Link shares have reshare permission if outgoing federated shares are enabled

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

Set proper share type when converting link shares to federated shares

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

[tx-robot] updated from transifex

Merge pull request #20881 from nextcloud/backport/20841/stable18

[stable18] add locking to resolve concurent move to trashbin conflicts

Do not read certificate bundle from data dir by default

Before the resources/config/ca-bundle.crt was only used when the list of custom
certificates was empty and the instance was not installed. But it should also
be used when the list is empty and the instance is installed.

This is inverting the logic to stop if the instance is not installed to use the
default bundle. And it also does this when the list is empty.

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

Merge pull request #20965 from nextcloud/backport/20033/stable18

[stable18] Enable fseek for files in S3 storage

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

harden seekable http stream a bit against failures

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

[tx-robot] updated from transifex

18.0.5 RC1

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

[tx-robot] updated from transifex

update autoloader

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

add basic tests for s3 seeking and add some error handling if reopen return the wrong range

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

make seekable s3 stream generic

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

Enable fseek for files in S3 storage

Signed-off-by: Lukas Stabe <lukas@stabe.de>

Merge pull request #21053 from nextcloud/backport/21050/stable18

[stable18] Caching and compression for app store requests

Compress the appstore requests by default

In test it reduced the transfered data from 5 MB to 2 MB. This should reduce the load on the appstore significantly.

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

Cache appstore requests for 60 instead of 5 minutes

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

[tx-robot] updated from transifex

Merge pull request #21030 from nextcloud/backport/21003/stable18

[stable18] Remove entries from locales.json incompatible with punic

[tx-robot] updated from transifex

Merge pull request #20996 from nextcloud/backport/20629/stable18

[stable18] Don't load text file preview when text app is available (Fixes: #20615)

Remove entries from locales.json incompatible with punic

As reported at https://github.com/nextcloud/server/issues/20999 the list contains en_US_POSIX as locale but punic is unable to parse such a locale. If you select that locale everyone is confused.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

Merge pull request #20994 from nextcloud/backport/19540/stable18

[stable18] Fix issues with Keystone auth v3 in files_external app

Merge pull request #20992 from nextcloud/backport/20941/stable18

[stable18] update icewind/smb to 3.2.4

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Don't load text file preview when text app is available (Fixes: #20615)

When opening a shared text document, the files_sharing app uses
internal preview support in case no other app to open the text file is
available.

So far, it only checks for the old files_texteditor app, not for the new
new text app. This commit fixes this by checking for both.

Signed-off-by: Jonas Meurer <jonas@freesources.org>

update icewind/smb to 3.2.4

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

Fix issues with Keystone auth v3 in files_external app

Signed-off-by: Julien Lutran <julien.lutran@corp.ovh.com>

Merge pull request #20955 from nextcloud/httpcache-18

[stable18] Proxy server could cache http response when it is not private

Merge pull request #20975 from nextcloud/dependabot/npm_and_yarn/build/stable18/handlebars-4.6.0

[Security] Bump handlebars from 4.5.3 to 4.6.0 in /build

[tx-robot] updated from transifex

[Security] Bump handlebars from 4.5.3 to 4.6.0 in /build

Bumps [handlebars](https://github.com/wycats/handlebars.js) from 4.5.3 to 4.6.0. **This update includes a security fix.**
- [Release notes](https://github.com/wycats/handlebars.js/releases)
- [Changelog](https://github.com/handlebars-lang/handlebars.js/blob/master/release-notes.md)
- [Commits](https://github.com/wycats/handlebars.js/compare/v4.5.3...v4.6.0)

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

[tx-robot] updated from transifex

Fix http cache test

Signed-off-by: Clement Wong <git@clement.hk>

Proxy server could cache http response when it is not private

Signed-off-by: Clement Wong <git@clement.hk>

[tx-robot] updated from transifex

Merge pull request #20574 from nextcloud/backport/16035/stable18

[stable18] dont show remote and email options if we have an exact match for local user email

dont show remote and email options if we have an exact match for local user email

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

Merge pull request #20802 from nextcloud/backport/19124/stable18

[stable18] Exclude groups from sharing: Skip delete groups

Merge pull request #20877 from nextcloud/backport/20719/stable18

[stable18] Fix languages empty array

[tx-robot] updated from transifex

Merge pull request #20924 from nextcloud/backport/20915/stable18

[stable18] Use random_bytes

Merge pull request #20774 from nextcloud/backport/19743/stable18

[stable18] dont try to update storage mtime if we can't get the mtime

Fix languages empty array

Signed-off-by: GretaD <gretadoci@gmail.com>

Use random_bytes

Since we don't care if it is human readbale.
The code is backwards compatible with the old format.

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

Merge pull request #20747 from nextcloud/backport/20718/stable18

[stable18] Add a wrapper to fall back to the share owner on public shares

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #20886 from nextcloud/backport/20853/stable18

[stable18] Fix color-text-maxcontrast not passing WCAG AA

[tx-robot] updated from transifex

Fix color-text-maxcontrast not passing WCAG AA

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>

add locking to resolve concurent move to trashbin conflicts

uses a lock to prevent two requests from moving a file to the trashbin concurrently
(causing sql duplicate key errors)

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