Merge pull request #4346 from nextcloud/properly-do-bruteforce-protection-via-annotation

Make BruteForceProtection annotation more clever

[tx-robot] updated from transifex

Make BruteForceProtection annotation more clever

This makes the new `@BruteForceProtection` annotation more clever and moves the relevant code into it's own middleware.

Basically you can now set `@BruteForceProtection(action=$key)` as annotation and that will make the controller bruteforce protected. However, the difference to before is that you need to call `$responmse->throttle()` to increase the counter. Before the counter was increased every time which leads to all kind of unexpected problems.

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

Merge pull request #4326 from nextcloud/downstream-27562

Reorder the entries of the log for easier reading

Merge pull request #4308 from nextcloud/lost-password-email

Update email template for lost password email

Merge pull request #2424 from nextcloud/fix-login-controller-test-consolidate-login

Fix login controller test and consolidate login

Improve PHPDoc

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

Merge pull request #4337 from nextcloud/fix-adblock-share-icon

Fix AdBlock blocking share icon, ref #866

Use constants

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

Merge pull request #4341 from nextcloud/local-link-explanation

Explain local link

Explain local link

The local link is a clever thing and the clients should support this imho but it might not be clear to all users. For one, the term 'local link' is a bit odd. Local with respect to what? It links directly to the file or folder, so direct link seems to make more sense to me. And we should explain the difference with a public link. So this PR:
* renames local link to direct link
* adds a short explanation, noting it only works for users who have access to this file/folder.

As other links are called public link you could also consider calling this 'private link', I suppose. But the links we sent by mail to ppl could also be called 'private link' (they are for one user, who git it by email) so I think it might be confusing. What do @nextcloud/designers think?

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

Merge pull request #4336 from nextcloud/add-ratelimiting

Add support for ratelimiting via annotations

Merge pull request #4342 from nextcloud/update-certificate-bundle

Update CA bundle

Merge pull request #4344 from nextcloud/dont-load-handlebars

Don't load handlebars in template

Don't load handlebars in template

It's already loaded in core.json

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

Merge pull request #4313 from nextcloud/ldap-name-change

Change LDAP method names

Remove expired attempts

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

Add tests for multiple parameters

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

Enable testing app in CI step

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

Update static autoloadermap

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

Move to dedicated MiddleWare

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

Fix .drone.yml

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

Add isset() as it can be an empty result

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

Register in ServerContainer

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

Add support for ratelimiting via annotations

This allows adding rate limiting via annotations to controllers, as one example:

```
@UserRateThrottle(limit=5, period=100)
@AnonRateThrottle(limit=1, period=100)
```

Would mean that logged-in users can access the page 5 times within 100 seconds, and anonymous users 1 time within 100 seconds. If only an AnonRateThrottle is specified that one will also be applied to logged-in users.

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

Update CA bundle

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

Merge pull request #4340 from nextcloud/remove-invalid-route

Remove invalid route

Remove invalid route

Said file doesn't exist anymore and accessing it would just return a 500

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

Merge pull request #3966 from nextcloud/downstream-26570

Override config.php values through environment variables

Merge pull request #3043 from nextcloud/issue-3038-no-logentry-on-email-login

Dont create a log entry on email login

[tx-robot] updated from transifex

Merge pull request #4338 from nextcloud/handlebars-min

Move to handlebars.min.js

Move to handlebars.min.js

core.js is now below 1mb!

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

Update email template for lost password email

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

Merge pull request #4307 from nextcloud/sharing-emails

New emails for sharebymail

Merge pull request #4335 from nextcloud/jquery-ui-min

Moved to jquery-ui.custom.min.js

Merge pull request #4282 from nextcloud/oci-connect-string

use the same oci connectstring in all code paths

Moved to jquery-ui.custom.min.js

Shaves off about 200kb of the default loaded js libs

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

Merge pull request #4306 from nextcloud/followup-3905

Favorites integration tests for shared files

Merge pull request #4331 from nextcloud/query-builder-sum

Add sum() to the query function builder

Fix unit tests

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

Favorites integration tests for shared files

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

Merge pull request #4329 from nextcloud/move-out-shared-folder

Fix moving files out of a shared folder

Dont add click-button text for plaintext mails

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

No newlines when heading is empty

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

make the plain text footer standard compliant and add a space after '--'. It also adds a line break in front so that there is some spacing between the mail body and the footer

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

Use instance name as alt-text

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

Don't duplicate the sentence with the header

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

Use new email design for sharebymail emails

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

Properly escape heading, body and button text

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

Add addBodyButton to add a single button to email templates

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

Change LDAP method names

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

Add sub() to the query function builder

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

Merge pull request #4325 from nextcloud/downstream-27522

Optimize put - Dont try to fetch filecache for not existing filecache…

Merge pull request #3614 from nextcloud/discover-federatedsharing-endpoints

Discover federatedsharing endpoints

Merge pull request #4323 from nextcloud/downstream-27613

Allow file upload when storage is unlimited

Merge pull request #4324 from nextcloud/downstream-27581-27602

Fix casing of same origin frame option

Fix AdBlock blocking share icon, ref #866

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

use unjailed path when moving files out of a shared storage

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

rename Jail::getSourcePath to getUnjailedPath to fix conflict with Local::getSourcePath

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

Simplify the test

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

Reorder the entries of the log for easier reading

Optimize put - Dont try to fetch filecache for not existing filecache in encription

Fix casing of same origin frame option

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

Allow file upload when storage is unlimited

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

Merge pull request #3905 from nextcloud/downstream-27069

Add integration test for trashbin

Merge pull request #4317 from nextcloud/cleanup-files_trashbin-update

[files_trashbin] Remove unneeded update steps

Merge pull request #4316 from nextcloud/cleanup-files_sharing-update

[files_sharing] Remove unneeded update steps

Merge pull request #4312 from nextcloud/cleanup-name

Update comments to Nextcloud

Merge pull request #4315 from nextcloud/cleanup-files_external-update

[files_external] Remove unneeded update steps

Merge pull request #4309 from nextcloud/remove-unused-code

Removes unused code for link share emails

Merge pull request #4310 from nextcloud/allow-plain-text-email

Allow to set text versions for the plain text email

Merge pull request #4314 from nextcloud/remove-not-needed-update.php

Remove update.php of files app

[files_trashbin] Remove unneeded update steps

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

[files_sharing] Remove unneeded update steps

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

[files_external] Remove unneeded update steps

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

Remove update.php of files app

* contains only update steps for 8.1/8.2 to 9.0 update

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

Update comments to Nextcloud

* based on PR by @Ardinis
* see #4311

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

[tx-robot] updated from transifex

Allow to set text versions for the plain text email

* allows different texts for HTML and text version of the email

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

Removes unused code for link share emails

* now handled by sharebymail app
* see https://github.com/nextcloud/server/pull/657

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

Merge pull request #4304 from nextcloud/add-email-template-to-ocp

Add IEMailTemplate to public OCP API

Add drone.yml config

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

Add integration test for trashbin

Add test for basic deletion.
Add test when deleting from shared folder as recipient.
Add test to check that metadata stays when moving out of shared folder
as recipient.

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

Merge pull request #4302 from nextcloud/test-integration-test

[Integration Tests] Clear opcode cache after config change

Remove constructor from interface

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

Merge pull request #3876 from nextcloud/downstream-26583

Share api expanded by tags

Merge pull request #4285 from nextcloud/bump-md5

[js] Bump blueimp-md5

Clear opcode cache after config change

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

share api expanded by tags (#26583)

* share api expanded by tags

* Modified files_sharing JS Unit tests

* modified tests. renamed request parameter. refactoring

* Update Share20OCS.php

Added missing function description

* Update Helper.php

Added missing function description

* Update Helper.php

implicit boolean conversion to !empty()

* Update Share20OCSTest.php

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

Merge pull request #4293 from nextcloud/fix_4248

Do reload shared views

Adjust class references

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

Add IEMailTemplate to public OCP API

Also adds `\OCP\Mail\IMailer::createEMailTemplate` as helper so the functionality can easily be used within apps.

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

Update autoloader

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

Do proper DI

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

give old end-points a seperate name

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

fix constructor

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

remove discovery manager in favour of the OCSDiscoveryService

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

add tests for discovery service

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>

adjust existing tests

Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>