Merge pull request #21197 from nextcloud/bugfix/noid/app-screenshots

Check if screenshot is available before showing it

Check if screenshot is available before showing it

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

Merge pull request #21225 from nextcloud/techdebt/humanfilesize-getcanonicallocale-package

Move humanFileSize and OC.getCanonicalLocale to npm packages

Merge pull request #21229 from nextcloud/fix-empty-event-reminder-notifications

Fix empty event UUID reminder notifications

Move humanFileSize and OC.getCanonicalLocale to npm packages

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

[tx-robot] updated from transifex

Merge pull request #20914 from nextcloud/enh/deprecated_event_log_usage

Log usage of a deprecated event instead of emitting

Merge pull request #21232 from nextcloud/bug/20427/use-OC-CLI-instead-of-PHP-SAPI

Use \OC::$CLI instead of PHP_SAPI

Merge pull request #21165 from nextcloud/dependabot/npm_and_yarn/babel/plugin-proposal-class-properties-7.10.1

Bump @babel/plugin-proposal-class-properties from 7.8.3 to 7.10.1

Merge pull request #21191 from nextcloud/td/js/use_router

OC.routing is just a @nextcloud/routing proxy

Use \OC::$CLI instead of PHP_SAPI

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

Fix empty event UUID reminder notifications

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

OC.routing is just a @nextcloud/routing proxy

So lets kill the duplicate code!

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

Merge pull request #21199 from nextcloud/bugfix/noid/prevent-harder-sharing-your-root

Prevent harder to share your root

[tx-robot] updated from transifex

Merge pull request #20938 from nextcloud/fix/share-talk-verification

Fix the Talk verification

Bump @babel/plugin-proposal-class-properties from 7.8.3 to 7.10.1

Bumps [@babel/plugin-proposal-class-properties](https://github.com/babel/babel/tree/HEAD/packages/babel-plugin-proposal-class-properties) from 7.8.3 to 7.10.1.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.1/packages/babel-plugin-proposal-class-properties)

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

Merge pull request #21168 from nextcloud/dependabot/npm_and_yarn/babel/preset-env-7.10.2

Bump @babel/preset-env from 7.9.6 to 7.10.2

Prevent harder to share your root

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

Bump @babel/preset-env from 7.9.6 to 7.10.2

Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.9.6 to 7.10.2.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.2/packages/babel-preset-env)

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

Merge pull request #21166 from nextcloud/dependabot/npm_and_yarn/babel/core-7.10.2

Bump @babel/core from 7.9.6 to 7.10.2

Merge pull request #21167 from nextcloud/dependabot/npm_and_yarn/vue-router-3.3.2

Bump vue-router from 3.2.0 to 3.3.2

Merge pull request #21091 from nextcloud/sftp-normalize-path

normalize sftp path in read and write stream

Merge pull request #21181 from nextcloud/fix/remote-share-remove-error-handling

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

[tx-robot] updated from transifex

Merge pull request #21190 from nextcloud/td/js/linter

Fix some linter warning

Fix some linter warning

Part of my ongoing battle against deprecation messages.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Bump vue-router from 3.2.0 to 3.3.2

Bumps [vue-router](https://github.com/vuejs/vue-router) from 3.2.0 to 3.3.2.
- [Release notes](https://github.com/vuejs/vue-router/releases)
- [Changelog](https://github.com/vuejs/vue-router/blob/dev/CHANGELOG.md)
- [Commits](https://github.com/vuejs/vue-router/compare/v3.2.0...v3.3.2)

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

Merge pull request #21164 from nextcloud/dependabot/npm_and_yarn/nextcloud/dialogs-1.3.1

Bump @nextcloud/dialogs from 1.3.0 to 1.3.1

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

Restore previous state when failing to change video verification state

When video verification can not be enabled or disabled the previous
state is set again in the JavaScript share object. This ensures that the
UI will not reflect a misleading state.

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

Fix the Talk verification

When enabling or disabling Talk verification in mail shares the server
expects also a new password to be set. As we always just update one
property at a time this means the Talk verification was impossible to
activate or deactivate. With this patch, we send the talk option AND the
new password. If there is no new password, the Talk option is disabled
(in mail shares; in link shares it is possible to enable or disable the
video verification without changing the password).

When we finally have descriptive text on ActionCheckbox'es we should
definitely add some explanatory text for the user. Right now this is as
good as it gets.

We'll have to backport to 18.

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

[tx-robot] updated from transifex

Bump @nextcloud/dialogs from 1.3.0 to 1.3.1

Bumps [@nextcloud/dialogs](https://github.com/nextcloud/nextcloud-dialogs) from 1.3.0 to 1.3.1.
- [Release notes](https://github.com/nextcloud/nextcloud-dialogs/releases)
- [Changelog](https://github.com/nextcloud/nextcloud-dialogs/blob/master/CHANGELOG.md)
- [Commits](https://github.com/nextcloud/nextcloud-dialogs/compare/v1.3.0...v1.3.1)

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

[tx-robot] updated from transifex

Bump @babel/core from 7.9.6 to 7.10.2

Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.9.6 to 7.10.2.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/master/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.10.2/packages/babel-core)

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

Merge pull request #21155 from nextcloud/td/move_to_capabilities_package

Move to @nextcloud/capabilities package

[tx-robot] updated from transifex

Merge pull request #21154 from nextcloud/fix/file_info_load_order

Fix the loading order of the js files_info and files_client

Merge pull request #21156 from nextcloud/do-not-clone-the-whole-talk-repository-for-integration-tests

Do not clone the whole Talk repository for integration tests

Do not clone the whole Talk repository for integration tests

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

Merge pull request #21141 from nextcloud/fix/require-ext-pdo-composer

List ext-pdo as hard requirement for composer

Move to @nextcloud/capabilities package

And kill of oc_capabilities and _oc_capabilities globals

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

Fix the loading order of the js files_info and files_client

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

Merge pull request #20970 from nextcloud/td/js/escape-html-global

Remove the escape-html global

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

Fix password changes in link and mail shares

Remove the escape-html global

Already announced for 19 that it would go. So now it realy does go.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Merge pull request #21118 from nextcloud/td/move_systemtags

Move systemtags to a proper location

Merge pull request #21146 from nextcloud/fix-reference-to-wrong-class-name

Fix reference to wrong class name

[tx-robot] updated from transifex

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 reference to wrong class name

\OCP\IShare does not exist; the right name is \OCP\Share\IShare,
although it is already imported as IShare.

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

Merge pull request #21140 from nextcloud/enhancement/oc-get-all-apps-return-type-doc

Document \OC_App::getAllApps return type

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>

Move the systemtags to core/src

* Move to the handlebar loader from webpack

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

List ext-pdo as hard requirement for composer

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

Document \OC_App::getAllApps return type

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

Merge pull request #20597 from nextcloud/enh/lazy_node_events

Add lazy events for the Node API

Merge pull request #21124 from nextcloud/feature/20612/add-capability-for-file-comments

Add a capability when file comments are enabled

Add a capability when file comments are enabled

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

Add lazy events for the Node API

Right now if you want to get events via the Node API you have to have a
real instance of the Root. Which in turns sets up the whole FS.

We should make sure this is done lazy. Else enabling the preview
generator for example makes you setup the whole FS on each and every
authenticated call.

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

[tx-robot] updated from transifex

Merge pull request #21122 from nextcloud/enhancement/auth-token-exception-reporting

Improve traces of invalid token exceptions

Merge pull request #21123 from nextcloud/fix/noid/fix-retrieval-of-groups

simplify getGroups, fixing wrong chunking logic

Merge pull request #21112 from nextcloud/techdebt/noid/remove-db-prefix

Remove db prefix option from cli install command

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

Make the translation sanitization optional

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>

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 #21079 from nextcloud/dependabot/npm_and_yarn/moment-2.26.0

Bump moment from 2.25.3 to 2.26.0

Merge pull request #21117 from nextcloud/design/text-variables-simplify

Simplify text variables from 4 to 2, map -lighter to -maxcontrast

Log why a token is not valid during password check

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

Improve traces of invalid token exceptions

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

[tx-robot] updated from transifex

Simplify text variables from 4 to 2, map -lighter to -maxcontrast

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

Bump moment from 2.25.3 to 2.26.0

Bumps [moment](https://github.com/moment/moment) from 2.25.3 to 2.26.0.
- [Release notes](https://github.com/moment/moment/releases)
- [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md)
- [Commits](https://github.com/moment/moment/compare/2.25.3...2.26.0)

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

Merge pull request #21106 from nextcloud/fix/10809/user-pwd-change-loginname

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

Remove the prefix setting attempt from our CI

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

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

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

Remove db prefix option from cli install command

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

Merge pull request #21108 from nextcloud/fix/20745/eor-old-php-api

fixes infinitely repeating LDPA search results with PHP <= 7.2

[tx-robot] updated from transifex

fixes infinitely repeating LDPA search results with PHP <= 7.2

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

Merge pull request #21092 from nextcloud/enh/noid/recommended-apps

Update recommended apps

Merge pull request #21090 from nextcloud/bug/noid/do-not-read-cert-from-datadir-by-default

Do not read certificate bundle from data dir by default

Merge pull request #21078 from nextcloud/dependabot/npm_and_yarn/vue-router-3.2.0

Bump vue-router from 3.1.6 to 3.2.0

Bump bundles

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

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>

Update hub bundle and add proxy rule to htaccess

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

normalize sftp path in read and write stream

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

Bump vue-router from 3.1.6 to 3.2.0

Bumps [vue-router](https://github.com/vuejs/vue-router) from 3.1.6 to 3.2.0.
- [Release notes](https://github.com/vuejs/vue-router/releases)
- [Changelog](https://github.com/vuejs/vue-router/blob/dev/CHANGELOG.md)
- [Commits](https://github.com/vuejs/vue-router/compare/v3.1.6...v3.2.0)

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

Merge pull request #20005 from joeried/occ-remove-bruteforce-attempts-by-ip

Implement occ command to reset bruteforce attemps from a given IP address