Merge pull request #21523 from nextcloud/backport/21478/stable17

[stable17] Clarify that the email is always shared within the instance

Clarify that the email is always shared within the instance

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21487 from nextcloud/backport/21482/stable17

[stable17] Don't log Keys

Don't log Keys

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

[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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #21335 from nextcloud/backport/21318/stable17

[stable17] clear LDAP cache after user deletion

clear LDAP cache after user deletion

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

[tx-robot] updated from transifex

Merge pull request #21158 from nextcloud/backport/21143/stable17-fix-password-changes-in-link-and-mail-shares

[stable17] Fix password changes in link and mail shares

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #20803 from nextcloud/backport/19124/stable17

[stable17] Exclude groups from sharing: Skip delete groups

Merge pull request #21130 from nextcloud/backport/21123/stable17

[stable17] simplify getGroups, fixing wrong chunking logic

[tx-robot] updated from transifex

Merge pull request #21249 from nextcloud/backport/21229/stable17

[stable17] Fix empty event UUID reminder notifications

Fix empty event UUID reminder notifications

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

[tx-robot] updated from transifex

Merge pull request #21231 from nextcloud/backport/20427/stable17

[stable17] Fix password reset saying Admin changed my password when reset from login page

Merge pull request #21228 from nextcloud/backport/21199/stable17

[stable17] Prevent harder to share your root

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>

Prevent harder to share your root

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

Merge pull request #21202 from nextcloud/backport/21181/stable17

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

Merge pull request #21183 from nextcloud/version/17.0.7/final

17.0.7

[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

17.0.7

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

[tx-robot] updated from transifex

Do not disable send password by Talk without setting new password in UI

Before it was possible to disable "send password by Talk" without
setting a new password, so the UI allowed that. Now that the server
prevents it the UI had to be adjusted to not update the share until a
new password is set when "send password by Talk" is disabled.

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

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
ignored. Now the given password is properly set and, if passwords are
enforced, it is not overriden with the autogenerated password.

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

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

simplify getGroups, fixing wrong chunking logic

pagination is taken care of properly in the search logic in Access class

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

Merge pull request #21062 from nextcloud/version/17.0.7/RC1

17.0.7 RC1

Merge pull request #21110 from nextcloud/backport/19793/stable17

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

[tx-robot] updated from transifex

Merge pull request #21104 from nextcloud/backport/20676/stable17

[stable17] Don't remove last user in ldap group when limit is -1

Merge pull request #21116 from nextcloud/backport/21106/stable17

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

[tx-robot] updated from transifex

Don't remove last user in ldap group when limit is -1

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

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

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

Merge pull request #21097 from nextcloud/backport/21090/stable17

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

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

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 #20986 from nextcloud/backport/20955/stable17

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

Merge pull request #20966 from nextcloud/backport/20033/stable17

[stable17] Enable fseek for files in S3 storage

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Fix http testCacheSeconds

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

[tx-robot] updated from transifex

harden seekable http stream a bit against failures

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

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>

[tx-robot] updated from transifex

17.0.7 RC1

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

[tx-robot] updated from transifex

Merge pull request #21055 from nextcloud/backport/21050/stable17

[stable17] 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

[tx-robot] updated from transifex

Merge pull request #20993 from nextcloud/backport/20941/stable17

[stable17] update icewind/smb to 3.2.4

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

update icewind/smb to 3.2.4

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

Merge pull request #20882 from nextcloud/backport/20841/stable17

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

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>

Merge pull request #20977 from nextcloud/dependabot/npm_and_yarn/build/stable17/handlebars-4.6.0

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

[tx-robot] updated from transifex