Rename renderHTML to renderHtml

* fixes #4383
* improves consistency

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

Merge pull request #4387 from nextcloud/fix-little-glitch

Fix font-weight of settings button

Merge pull request #4388 from nextcloud/remove-unused-stuff

Remove unused CSS styles

Merge pull request #4386 from nextcloud/improve-menu-css

Improve menu CSS

Improve menu CSS

* fix mess with menus and actions in the files app
* reduces amount of !important usages
* keeps the behaviour on mobile as well as on desktop

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

Merge pull request #4385 from nextcloud/remove-unused-css

Remove unused CSS

Merge pull request #4395 from nextcloud/fix_4394

Do not show empty admin sections

Do not show empty admin sections

Fixes #4393

It is far from efficient code. But then again it is easy to understand
and I doubt admins will browse it 24/7

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

Merge pull request #4382 from nextcloud/use-proper-reply-to

Add "Reply-To" on ShareByMailProvider mails

Fix indentation

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

Merge pull request #4350 from nextcloud/adjust-old-bruteforce-protection-annotations

Adjust existing bruteforce protection code

Remove unused CSS styles

* could not find an traces of .popup and .arrow anywhere else

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

Fix font-weight of settings button

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

Remove unused CSS

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

Merge pull request #4371 from nextcloud/dont-allow-dot-usernames

Better validation of allowed user names

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #4369 from nextcloud/fix-translations

Fix translations

Merge pull request #3585 from nextcloud/popover-clickable-area

expand clickable area of popover menu entries to full width

Merge pull request #4379 from nextcloud/nicely-designed-confirmation-mail

Beautify test email

expand clickable area of popover menu entries to full width

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

Fix translations

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

Merge pull request #4380 from nextcloud/show-instance-name-in-from

Add instance name to default sender

Beautify test email

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

Merge pull request #4372 from nextcloud/smtp-password

Don't put the SMTP password into the HTML code

[tx-robot] updated from transifex

Add "Reply-To" on ShareByMailProvider mails

Fixes https://github.com/nextcloud/server/issues/4209

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

Fix unit tests

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

Add instance name to default sender

Otherwise your mail program shows "foo@mail.com" instead of "Nextcloud" or whatever your instance name is.

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

Merge pull request #4376 from nextcloud/add-bruteforce-protection-to-change-self-password

Add bruteforce protection to changePersonalPassword

Add bruteforce protection to changePersonalPassword

While the risk is actually quite low because one would already have the user session and could potentially do other havoc it makes sense to throttle here in case of invalid previous password attempts.

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

Fix some more stuff

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

Fix tests

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

Don't put the SMTP password into the HTML code

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

Better validation of allowed user names

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

[tx-robot] updated from transifex

[tx-robot] updated from transifex

[tx-robot] updated from transifex

Merge pull request #2834 from nextcloud/accesListToShareManager

Access list to share manager

[tx-robot] updated from transifex

Adjust existing bruteforce protection code

- Moves code to annotation
- Adds the `throttle()` call on the responses on existing annotations

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

Merge pull request #4330 from nextcloud/activities-for-password-mail-change

Add activities when email or password is changed

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

Make BruteForceProtection annotation more clever

[tx-robot] updated from transifex

Fix autoloader

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

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

Mail shares trigger the public key

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

Update autoloader

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

Fix server container registration

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

Don't load handlebars in template

It's already loaded in core.json

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

Not needed in the DIContainer anymore

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

Do not set full path if not currentAccess

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

Add samples to the docs

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

Add more tests for the share helper

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

Catch exceptions and use as many results as possible

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

Activate the test

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

Adjust docs and make !$currentAccess simpler

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

Add unit tests for sharebymail provider

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

Fix share manager test

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

Fix default share provider

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

Fix tests for ShareHelper

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

Fix federated file sharing

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

Fix returned paths for remote shares

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

Return the token as well

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

Also return the token

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

Return the paths for the users without setting them all up

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

Fix comments

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

Add shareHelper test

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

Add mail element to access list

* Each provider just returns what they have so adding an element won't
require changing everything
* Added tests

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

Offload acceslist creation to providers

* This allows for effective queries.
* Introduce currentAccess parameter to speciy if the users needs to have
currently acces (deleted incomming group share). (For notifications)

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

Added ShareHelper

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

Fix tests

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

Port Encryption/file to new getAccessList

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

Added to public interface

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

Add owner to access list

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

Add getAccessList to ShareManager

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

No need for those emails anymore

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

Send an email to the old address on email change

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

Allow to read the old email on the hook as well

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

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

Change LDAP method names

Move logic into new class with DI

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

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>