aboutsummaryrefslogtreecommitdiffstats
path: root/apps/settings/lib/SetupChecks
Commit message (Collapse)AuthorAgeFilesLines
* fix(webauthn): do not require bcmath or gmp - not needed anymoreFerdinand Thiessen2025-03-191-3/+1
| | | | | | | | The extensions are not required anymore but only recommended for performance. See also: https://github.com/web-auth/webauthn-framework/issues/213 Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Change SQL query so that it is ansi safeJohan Bernhardsson2025-02-271-3/+3
| | | | Signed-off-by: Johan Bernhardsson <johan.bernhardsson@redpill-linpro.com>
* fix(l10n): Improve english source stringsJoas Schilling2025-02-262-2/+2
| | | | | | | - No leading/trailing whitespace - Use asci single quote Signed-off-by: Joas Schilling <coding@schilljs.com>
* feat(setupcheck): check logging level for validityfeat/setup-check-loggingFerdinand Thiessen2025-02-221-0/+55
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(setupcheck): Update setup check for PHP version to be more accuratebugfix/noid/bump-php-dependency-update-versionsJoas Schilling2025-02-101-1/+12
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #50281 from ↵Andy Scherzinger2025-01-301-1/+3
|\ | | | | | | | | nextcloud/fix/fix-email-setupcheck-with-null-smtpmode fix(settings): Fix setup check when mail_smptmode is set to "null"
| * fix(settings): Fix setup check when mail_smptmode is set to "null"fix/fix-email-setupcheck-with-null-smtpmodeCôme Chilliet2025-01-201-1/+3
| | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix(settings): Add some context to the PHP memory limit errorjtr-settings-memory-limit-detailsJosh2025-01-261-1/+1
| | | | | | Signed-off-by: Josh <josh.t.richards@gmail.com>
* | fix(SetupChecks): Pass webfinger if a handler is therefix-setupcheck-webfinger-400Josh2025-01-261-1/+1
|/ | | Signed-off-by: Josh <josh.t.richards@gmail.com>
* fix(setupchecks): Binary data can have problems with serializesetupChecksMoveFromBinaryGit'Fellow2025-01-091-2/+2
| | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* perf(settings): Speed up InternetConnectivity setup checkjtr-perf-checks-connectivity-https-protoJosh2025-01-071-10/+10
| | | | | Specify default protocol (https://) rather than let default handling test both http:// and https:// Signed-off-by: Josh <josh.t.richards@gmail.com>
* fix(setupcheck): Make the Memcache setupcheck use the cacheJoas Schilling2024-12-021-0/+37
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix(settings): psalm PhpMaxFileSize setup check warningbackport/49384/masterskjnldsv2024-11-191-6/+6
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* feat(settings): add big file upload setup checksfeat/php-setup-file-uploadskjnldsv2024-11-191-0/+80
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* feat: Update supported PostgreSQL versionsfeat/postgres-13-17Ferdinand Thiessen2024-11-131-2/+12
| | | | | | 12 is now EOL but 17 was released and should be supported and tested. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #48792 from nextcloud/chore/update_mysql_setup_check_i18nBenjamin Gaussorgues2024-11-051-4/+2
|\
| * chore(setupchecks): update translation for MySQL row format checkchore/update_mysql_setup_check_i18nBenjamin Gaussorgues2024-10-301-4/+2
| | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | chore(i18n): Fix grammarrakekniven2024-11-031-1/+1
|/ | | | | Reported at Transifex Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
* Merge pull request #48547 from nextcloud/feat/row_format_checkBenjamin Gaussorgues2024-10-181-0/+72
|\
| * feat(setupchecks): add row format setup check for MySQL databasesfeat/row_format_checkBenjamin Gaussorgues2024-10-081-0/+72
| | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | chore(apps): Apply new rector configuration to autouse classesCôme Chilliet2024-10-151-1/+2
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* chore(deps): Update nextcloud/coding-standard to v1.3.1provokateurin2024-09-198-17/+17
| | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(setupChecks): Having transactional file locking disabled is not supportedfix-setupcheck-filelockingJosh Richards2024-09-191-3/+3
| | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* feat: Make `CheckServerResponseTrait` public and provide as ↵feat/make-setup-check-trait-publicFerdinand Thiessen2024-09-168-164/+12
| | | | | | | | | `OCP\SetupCheck\CheckServerResponseTrait` This trait is used by other apps for creating setup checks, so we should provide it instead apps using private API. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(a11y): Add OTF font loading checkbugfix/noid/fix-otf-loadingJoas Schilling2024-09-161-6/+21
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* refactor(settings): `CheckServerResponseTrait` always expect absolute pathFerdinand Thiessen2024-09-133-32/+38
| | | | | | Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(setup-checks): Ensure URL with webroot worksFerdinand Thiessen2024-09-133-25/+27
| | | | | | | | | | | We basically mock the way `URLGenerator::getAbsoluteURL` works, so we must make sure that the URL might already contain the webroot. Because `baseURL` and `cliURL` also contain the webroot we need to remove the webroot from the URL first. Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Daniel <mail@danielkesselberg.de> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(setupchecks): Test overwrite.cli url first, then generated one, andCôme Chilliet2024-09-051-16/+30
| | | | | | trusted domains as last fallback. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* fix: add option to remove the webroot for setup checks and don't check ↵Daniel Kesselberg2024-09-053-13/+38
| | | | | | | | | | trusted_domains. 1) The checks for well-known urls should always run against the root domain and therefore the option to remove the webroot. 2) For trusted domains, the available protocol is unknown, and thus some guesswork would be needed to make that work. I've decided for now to not consider them anymore to reduce false-positives. Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* feat(settings): Add setup check for apcu cache expungeJulius Härtl2024-08-271-0/+70
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* style: update codestyle for coding-standard 1.2.3Daniel Kesselberg2024-08-255-10/+10
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* fix(SetupCheck): Properly check public access to data directoryFerdinand Thiessen2024-08-081-4/+12
| | | | | | | | | | | | | When checking for public (web) access to the data directory the status is not enough as you might have a webserver that forwards to e.g. a login page. So instead check that the content of the file matches. For this the `.ncdata` file (renamed from `.ocdata`¹) has minimal text content to allow checking. ¹The file was renamed from the legacy `.ocdata`, there is a repair step to remove the old one. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #46353 from nextcloud/skipOPcacheCLIGit'Fellow2024-08-011-0/+5
|\ | | | | fix(setupchecks): Skip checking for OPcache settings if running checks from CLI
| * fix: typoGit'Fellow2024-07-081-1/+1
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * fix: check both CLI and valueGit'Fellow2024-07-081-1/+1
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * fix(setupchecks): Skip checking for OPcache if running from CLIGit'Fellow2024-07-081-0/+5
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | feat(security): Add public API to allow validating IP Ranges and checking ↵Joas Schilling2024-07-191-14/+11
| | | | | | | | | | | | | | for "in range" Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | feat(security): restrict admin actions to IP rangesBenjamin Gaussorgues2024-07-191-0/+66
| | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | fix(caldav): lower scheduling table size warningAnna Larch2024-07-171-2/+6
|/ | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* fix(IntegrityCheck): Ensure the check is run if no results are availableFerdinand Thiessen2024-07-031-1/+8
| | | | | | | | | If there are no cached results the current implementation was also returning an empty array, but this was the same as when there was a successful run. So to distinguish this we return `null` if there are *no* results. In this case we need to rerun the integrity checker. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #46190 from ↵Daniel2024-06-291-2/+6
|\ | | | | | | | | nextcloud/bug/45047/skip-check-when-disk-free-space-disabled fix(setupchecks): skip check when disk_free_space is disabled
| * fix(setupchecks): skip check when disk_free_space is disabledDaniel Kesselberg2024-06-281-2/+6
| | | | | | | | | | | | Make it easier to discover that the check failed because disk_free_space is disabled. Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | refactor(SetupCheck): Make mariadb and mysql version variablesFerdinand Thiessen2024-06-271-5/+37
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix(settings): Adjust SetupCheck for supported database versionsFerdinand Thiessen2024-06-271-8/+10
|/ | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(settings): make trailing slash for caldav/carddav redirects optionalMichaIng2024-06-241-1/+1
| | | | | | | | | | | | | | #43939 moved the CalDAV/CardDAV redirect checks from the frontend to a new backend API. Since the backend does not send an authentication header, checking for the expected response code 207 of the DAV endpoint does not work anymore, hence the URL of the last redirect is checked instead. This URL is expected to contain a trailing slash, which was not required before, since the DAV endpoint works properly without it (when authenticated). While a trailing slash in the redirect does no harm, it causes many setups to throw an admin panel warning, while in fact the redirects work properly. Furthermore, the proposed "/.well-known/carddav" => "/remote.php/dav/" redirect leads to double slashes, when doing a request to "/.well-known/carddav/", which seems more wrong then right. This change makes the trailing slash optional, hence old and adjusted setups won't throw the warning anymore, and the DAV endpoint works well in both cases. Signed-off-by: MichaIng <micha@dietpi.com>
* feat: add setup check for needed mimetype migrationsRobin Appelman2024-06-181-0/+44
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* chore: Add SPDX headerAndy Scherzinger2024-06-0346-882/+92
| | | | Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
* fix(caldav): automatically delete outdated scheduling objectsAnna Larch2024-05-311-0/+48
| | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* Merge pull request #45262 from nextcloud/check-HttpsUrlGenerationCôme Chilliet2024-05-131-5/+12
|\ | | | | fix(SetupChecks): Detect CLI mode in HTTPS / URL generator check
| * fix(SetupChecks): Detect CLI mode in HTTPS / URL generator checkJosh Richards2024-05-121-5/+12
| | | | | | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
generated by cgit v1.2.3 (git 2.39.1) at 2025-05-09 03:42:13 +0000