refactor(certificate manager): remove unused variable

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

fix(certificate manager): migrate certificate bundles from files_external to data

CertificateManager doesn't work propertly if the files_external app is disabled, so let's store
directly in /data/certificate_manager the bundled certificates. This always has to be done on local
disk as curl currently requires a path to the cert bundle.

When we require PHP 8.1 we will be able to simply store the certificate
bundle in database/memory/cache and pass it through the CURLOPT_SSLCERT_BLOB option.

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

Merge pull request #43234 from nextcloud/fix/provisioning-exception-msgs

fix(provisioning_api): Translate exceptions shown in the frontend

Merge branch 'master' of github.com:nextcloud/server

Cleanup versions entity in during versions:clean command

Signed-off-by: Louis Chemineau <louis@chmn.me>

Merge pull request #43210 from nextcloud/fix/use-github-ci-for-s3-tests

chore(ci): Force github CI for tests failing on self-hosted runners

Merge pull request #43203 from nextcloud/fix/cleanup-register-command-with-di

Use Dependency Injection in register_commands.php

fix(provisioning_api): Translate exceptions shown in the frontend + replace some deprecations

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #43215 from nextcloud/rakekniven-patch-3

fix(i18n): Adapted spelling of Unicode

fix: Make Repair object injectable and fix BackgroundRepair test

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #41327 from nextcloud/dont-reuse-metadata-unscanned

dont reuse etag for folders marked explicitly unscanned

also updated parent etags when a changed etag is detected during scanning

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

Merge pull request #41338 from nextcloud/fix/nested-actions-back-focus

fix(files): make sure we focus the parent actions submenu entry

add test for updating etag with unchanged mtime in child folder

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

add test for updating etag with unchanged mtime

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

dont reuse etag for folders marked explicitly unscanned

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

Merge pull request #42913 from nextcloud/fix/settings-apps-stop-reload-force-enabl

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(files): make sure we focus the parent actions submenu entry

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>

Merge pull request #43170 from nextcloud/fix/mysql-ci

fix(CI): MySQL unit tests, use LTS for code coverage and drop EOL

Merge pull request #40594 from lorenz/fix-dav-url

fix(settings): stop reloading after force-enabling an app

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>

Merge pull request #43164 from nextcloud/dependabot/npm_and_yarn/moment-2.30.1

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

chore(deps): Bump moment from 2.29.4 to 2.30.1

Dependabot couldn't find the original pull request head commit, 3cd9244a458dd70796ef87dfd9add77e367650a2.

Merge pull request #43166 from nextcloud/dependabot/npm_and_yarn/marked-11.2.0

chore(deps): Bump marked from 9.1.5 to 11.2.0

dav: fix wrong decoding of pluses in URLs

PHP's urldecode function does not decode URLs according to RFC 3986, but
according to the HTML 4.01 query parameter
encoding. This results in pluses being wrongly decoded to spaces even
though they should not be decoded at all.

Use rawurldecode instead, which follows RFC 3986 properly.

This fixes an issue where files on DAV shares containing pluses were
incorrectly decoded to spaces.

Fixes: #15849
Signed-off-by: Lorenz Brun <lorenz@brun.one>

Merge pull request #38173 from arawa/feature/37677/exclude-some-groups-from-sharing-with-users

New core setting : shareapi_only_share_with_group_members_exclude_gro…

Merge pull request #42933 from nextcloud/fix/db/dirty-read-on-primary

fix(db): Execute dirty reads on the primary node

build(): build assets

Signed-off-by: Baptiste Fotia <fotia.baptiste@hotmail.com>

build(): build assets

Signed-off-by: Baptiste Fotia <fotia.baptiste@hotmail.com>

test(php): Fix the unit tests

I added our context in the unit tests following the advice of Louis.

Link : https://github.com/nextcloud/server/pull/43186#issuecomment-1916571027

Signed-off-by: Baptiste Fotia <fotia.baptiste@hotmail.com>

feat(php,vue): Create a new parameter

Signed-off-by: Baptiste Fotia <fotia.baptiste@hotmail.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #43172 from nextcloud/fix/43149/files--table-header-actions-position

fix(files): selected files actions position on scroll/with readme block

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

chore(deps): Bump marked from 9.1.5 to 11.2.0

Bumps [marked](https://github.com/markedjs/marked) from 9.1.5 to 11.2.0.
- [Release notes](https://github.com/markedjs/marked/releases)
- [Changelog](https://github.com/markedjs/marked/blob/master/.releaserc.json)
- [Commits](https://github.com/markedjs/marked/compare/v9.1.5...v11.2.0)

---
updated-dependencies:
- dependency-name: marked
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

Revert "switching to new IAppConfig"

This reverts commit c693c58d253168e712cde57b2dd8ea3262b900e8.

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

switching to new IAppConfig

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

chore(assets): Recompile assets

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

fix(files): selected files actions position on scroll/with readme block

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>

Merge pull request #43171 from nextcloud/fix/vue-8-5-1

chore: Update `@nextcloud/vue` to 8.5.1

chore: Update `@nextcloud/vue` to 8.5.1

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix(i18n): Adapted spelling of Unicode

Reported at Transifex

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>

chore: Fix DI for integrity checker before installation

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: use DI for \OC\Repair

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: Revert using DI for IntegrityCodeChecker as it uses a special service name

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: use DI for Command\Security\ListCertificates

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: use long namespaced class instead of putting it inline

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: Fix DI for Encryption\DecryptAll command

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: Use DI in register_command instead of new calls

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #43194 from nextcloud/fix/fix-psalm-missing-template-parameter

chore(ci): Force github CI for tests failing on self-hosted runners

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore(codeowners): add frontend files expertise

Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>

Merge pull request #42936 from nextcloud/fix/files_reference_widget_inaccessible_wording

fix(files): Improve inaccessible files message in file reference widget

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

fix(db): Execute dirty reads on the primary node

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

Merge pull request #43185 from nextcloud/feat/db/dirty-queries-production

feat(db): Make dirty query logging available in production

fix(files): Improve inaccessible files message in file reference widget

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

Merge pull request #43191 from nextcloud/fix/use-di-for-setup-class

chore: Fix twofactor_backupcodes tests

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

fix: Fix template parameter for some listeners

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

fix(twofactor_backupcodes): Mirate away from deprecated event

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore: Fix missing template parameter for IEventListener

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

feat(db): Make dirty query logging available in production

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

Merge pull request #43188 from nextcloud/fix/ci-disable-coverage-on-pr

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #43031 from nextcloud/fix/fix-throw-phpdoc-in-setupresult

Merge pull request #42331 from nextcloud/fix/auth-token-settings

fix(settings): Disable renaming for tokens marked to be wiped

Merge pull request #42794 from nextcloud/fix/invalid-trusted-proxies

fix(Request): Catch exceptions in `isTrustedProxy`

chore: Cleanup Installer class as well

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

fix: Use DI for Setup class and move away from deprecated methods

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

chore(ci): Turn off codecov patch CI

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Document throwing behavior of SetupResult upon invalid rich object

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #42942 from nextcloud/enh/migrate-debugmode-setupcheck

chore: Compile assets

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix(settings): Disable renaming for tokens marked to be wiped

This otherwise a WipeException will be thrown when trying to rename

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix(CI): MySQL unit tests, use LTS for code coverage and drop EOL

Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

chore(ci): Disable code coverage upload on pull request

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Migrate debug mode check to new SetupCheck API

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Merge pull request #42930 from nextcloud/fix/6914/overwritecond

Fix: config param 'overwritecondaddr' not working

Merge pull request #43143 from nextcloud/dependabot/composer/scssphp/scssphp-1.12.1

chore(deps): Bump scssphp/scssphp from 1.11.0 to 1.12.1

chore(deps): Bump scssphp/scssphp from 1.11.0 to 1.12.1

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

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Fix(l10n): Update translations from Transifex

Signed-off-by: Nextcloud bot <bot@nextcloud.com>

Merge pull request #43147 from nextcloud/fix/files-unify-folder-before-files

fix(files): Consistent sorting for folders before files (user configurable)

test(unit): fix RequestTest

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

code style: ommited space, reverted [code review]

fixing bug #6914: Config Param 'overwritecondaddr' not working

- just ignoring/removing extra parameter 'protocol' as suggested by
  blizzz

Signed-off-by: Pavel Kryl <pavel@kryl.eu>

fix(Request): Catch exceptions in `isTrustedProxy`

The function fails if the configured trusted proxies contain invalid characters and the underlying IpUtils will throw.
But as it is used by `getRemoteAddress` which is used by logging / templating, thrown errors are not reported but silently fail with error 500.

Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

fix(cypress): Adjust files sorting tests

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

chore: Compile assets

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #43159 from nextcloud/dependabot/github_actions/actions/checkout-4.1.1

Merge pull request #43161 from nextcloud/dependabot/github_actions/peter-evans/create-or-update-comment-4.0.0

fix(files): Consistent sorting for folders before files (user configurable)

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Merge pull request #43163 from nextcloud/dependabot/npm_and_yarn/core-js-3.35.1

Merge pull request #43162 from nextcloud/dependabot/github_actions/dorny/paths-filter-3.0.0

chore(deps): Bump dorny/paths-filter from 2.11.1 to 3.0.0

chore(assets): Recompile assets

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

Merge pull request #43152 from nextcloud/fix/unshare