aboutsummaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #52256 from nextcloud/backport/52224/stable31Joas Schilling20 hours1-0/+29
|\ | | | | [stable31] fix(usermanager): Don't throw when checking if a too long user id is an existing user
| * fix(usermanager): Don't throw when checking if a too long user id is an ↵backport/52224/stable31Joas Schilling5 days1-0/+29
| | | | | | | | | | | | existing user Signed-off-by: Joas Schilling <coding@schilljs.com>
* | fix(mail): use matching text color for primary elementsbackport/52240/stable31Ferdinand Thiessen5 days1-12/+12
|/ | | | | | | when we use the default primary as background color we have to use the default primary text color as well. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Transfer ownership with S3 as primarybackport/51020/stable31Louis Chemineau6 days1-0/+18
| | | | | | | | | | | | | | | | | | | | | | When using S3 as primary storage, transferring ownership with the `--move` option fail with the following error: `SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '8-45b963397aa40d4a0063e0d85e4fe7a1' for key 'fs_storage_path_hash'` The `--move` option moves the entire home folder from one account to another. The error means that the move failed because the destination folder already exist in `oc_filecache`. - With S3 as primary storage, folders only exists as entries in `oc_filecache`. - With S3 as primary storage, `moveFromStorage(...)` only moves the cache entry, as nothing needs to be moved on disk. This cache move does not delete potentially pre-existing destination folder. - With Local storage, `moveFromStorage(...)` calls `rename(...)` which delete pre-existing folder. - `transfer(...)`: https://github.com/nextcloud/server/blob/687a4d9ac7fcdbd935f81a0def567a1092306f7a/apps/files/lib/Service/OwnershipTransferService.php#L112 - `oneTimeUserSetup(...)`: https://github.com/nextcloud/server/blob/687a4d9ac7fcdbd935f81a0def567a1092306f7a/lib/private/Files/SetupManager.php#L261-L262 - `mkdir(...)`: https://github.com/nextcloud/server/blob/687a4d9ac7fcdbd935f81a0def567a1092306f7a/lib/private/Files/ObjectStore/ObjectStoreStorage.php#L91-L135 - `moveFromStorage(...)`: https://github.com/nextcloud/server/blob/687a4d9ac7fcdbd935f81a0def567a1092306f7a/lib/private/Files/ObjectStore/ObjectStoreStorage.php#L635-L636 Delete pre-existing folder in `moveFromStorage(...)` Signed-off-by: Louis Chemineau <louis@chmn.me>
* Merge pull request #52068 from nextcloud/backport/52066/stable31Joas Schilling11 days2-14/+67
|\ | | | | [stable31] fix(federation): Don't load the addressbook when resolving a cloud ID
| * fix(federation): Fix returning "no display name" after cache resultbackport/52066/stable31Joas Schilling11 days1-0/+28
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * fix(federation): Don't load the addressbook when resolving a cloud IDJoas Schilling11 days2-14/+39
| | | | | | | | | | | | Instead we delay the lookup of the display name until it is actually used Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #52078 from nextcloud/backport/52075/stable31John Molakvoæ11 days1-1/+1
|\ \ | |/ |/|
| * fix(files_sharing): fix share creation error handlingskjnldsv13 days1-1/+1
| | | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | fix(taskprocessing): use the event for AppAPI to get list of AI providersbackport/52050/stable31Oleksander Piskun11 days1-1/+319
|/ | | | Signed-off-by: Oleksander Piskun <oleksandr2088@icloud.com>
* Revert "refactor(RichObjectStrings): Only log error if key or value is not ↵revert/52038Joas Schilling13 days1-0/+5
| | | | | | | | string in validator" This reverts commit 23a61a2b42ce9b48ced31d05dba3dff281d37cf7. Signed-off-by: Joas Schilling <coding@schilljs.com>
* refactor(RichObjectStrings): Only log error if key or value is not string in ↵backport/52035/stable31provokateurin14 days1-5/+0
| | | | | | validator Signed-off-by: provokateurin <kate@provokateurin.de>
* feat(ip): use larger IPv6 range by defaultBenjamin Gaussorgues14 days1-5/+5
| | | | | | | Some providers assign `/48` IPv6 blocks instead of `/64` so it sounds safer to use this mask by default. Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* fix: ensure enabled themes are set on the templateFerdinand Thiessen2025-04-041-0/+6
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(session): Only mark sessions of permanent tokens as app passwordsbackport/51905/stable31Christoph Wurst2025-04-031-0/+40
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* fix(oauth2): retain support for legacy ownCloud clientsbackport/50858/stable31Richard Steinmetz2025-04-021-2/+8
| | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* perf(cron): Delay (re)checking timed jobsbackport/50768/stable31Christoph Wurst2025-03-313-17/+52
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* test: Adapt tests to properly use valid email addressesbackport/51707/stable31Julius Knorr2025-03-281-25/+25
| | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* fix(IFilenameValidator): correctly handle case insensitivitybackport/51611/stable31Ferdinand Thiessen2025-03-281-1/+91
| | | | | | | | - forbidden names and forbidden base names are case **insensitive** so we need to check all lowercase here. - add test that config value is also read case insensitive. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Do not build encrypted password if there is nonebackport/51130/stable31Julius Knorr2025-03-281-0/+40
| | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* fix: add event status and participant statusbackport/51501/stable31SebastianKrupinski2025-03-283-0/+7
| | | | Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
* fix(database): Cast users count to integerbackport/51490/stable31Git'Fellow2025-03-251-0/+13
| | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* test: fix CI when running Karma testsFerdinand Thiessen2025-03-201-3/+11
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(cardav): only show useres from enabled addressBooks in contacts menubackport/51380/stable31Hamza Mahjoubi2025-03-191-18/+64
| | | | Signed-off-by: Hamza Mahjoubi <hamzamahjoubi221@gmail.com>
* fix(RichObjectStrings): Make exception messages for invalid parameters more ↵backport/51442/stable31provokateurin2025-03-171-2/+2
| | | | | | useful for debugging Signed-off-by: provokateurin <kate@provokateurin.de>
* fix(mime-types): adjust compiled mime types and signbackport/51434/stable31Ferdinand Thiessen2025-03-122-3/+2
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(lookup-server): disable lookup server for non-global scale setupsFerdinand Thiessen2025-03-111-14/+13
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(lookup-server): do not query data by defaultFerdinand Thiessen2025-03-111-4/+4
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* refactor(TempManager): Simplify and unify implementations and remove legacy ↵backport/51194/stable31provokateurin2025-03-031-18/+7
| | | | | | behavior Signed-off-by: provokateurin <kate@provokateurin.de>
* feat: support migrating an instance to shardingbackport/48795/stable31Robin Appelman2025-02-251-1/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #50941 from nextcloud/backport/50053/stable31Andy Scherzinger2025-02-251-1/+1
|\ | | | | [stable31] fix: Change UserAgent to *cloud
| * fix: Change UserAgent to *cloudbackport/50053/stable31Johannes Endres2025-02-211-1/+1
| | | | | | | | | | Co-authored-by: Daniel Kesselberg <mail@danielkesselberg.de> Signed-off-by: Johannes Endres <je@johannes-endres.de>
* | refactor: convert sanitize account properties repair step to background jobbackport/50985/stable31Ferdinand Thiessen2025-02-242-10/+55
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix: Optimize repair step performanceFerdinand Thiessen2025-02-241-1/+26
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix: validate account properties as a repair stepFerdinand Thiessen2025-02-241-0/+89
|/ | | | | | | | Replace `ValidatePhoneNumber` from Nextcloud 21 with a new repair step, `ValidateAccountProperties` which validates and sanitizes all account properties. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: make sure we process mime extensions as stringskjnldsv2025-02-091-0/+35
| | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* fix(FediverseAction): Ensure valid fediverse links are generatedFerdinand Thiessen2025-02-061-0/+186
| | | | | | Harden also for existing values of the profile. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix(AccountManager): Sanitize social media handlesFerdinand Thiessen2025-02-061-50/+200
| | | | | | Ensure to only accept valid X and fediverse handles. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Exclude non accepted shares when computing access listbackport/50519/stable31Louis Chemineau2025-01-301-0/+9
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* Merge pull request #50508 from nextcloud/backport/50436/stable31Andy Scherzinger2025-01-282-5/+10
|\ | | | | [stable31] fix: Harden files scanner for invalid null access
| * fix: Harden files scanner for invalid null accessbackport/50436/stable31Ferdinand Thiessen2025-01-282-5/+10
| | | | | | | | | | | | Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com> Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | fix(sharing): Move permission validation to share managerFerdinand Thiessen2025-01-281-4/+24
|/ | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #50468 from nextcloud/backport/50234/stable31Andy Scherzinger2025-01-273-213/+171
|\ | | | | [stable31] fix(ratelimit): Allow to bypass rate-limit from bruteforce allowlist
| * fix(ratelimit): Allow to bypass rate-limit from bruteforce allowlistbackport/50234/stable31Joas Schilling2025-01-273-213/+171
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | fix(preview): Filter for folders in cleanup old preview jobbackport/48581/stable31Dario Mehlich2025-01-271-0/+31
|/ | | | | | | | | | | | | Fixes #35936. When running `OC\Preview\BackgroundCleanupJob`, the main iteration loop in `run()` expects a folder, however, `getOldPreviewLocations()` currently does not filter by mimetype and therefore can yield a non-folder entry which causes an Exception when constructing the Folder impl. Filtering for `httpd/unix-directory`, as `getNewPreviewLocations()` already does, fixes this issue. Signed-off-by: Dario Mehlich <d.mehlich@gmail.com>
* fix(taskprocessing): Fix testsMarcel Klehr2025-01-251-0/+2
| | | | Signed-off-by: Marcel Klehr <mklehr@gmx.net>
* fix: Metadata field searchbackport/50273/stable31Louis Chemineau2025-01-251-2/+7
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* fix(TemplateLayout): `core` is not an app but the server itselfFerdinand Thiessen2025-01-241-13/+19
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* fix: Correctly return app id and app version for `core` styles and imagesFerdinand Thiessen2025-01-241-0/+103
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Revert "fix(Share20\Manager): Ensure node is still accessible when checking ↵backport/50389/stable31provokateurin2025-01-241-29/+4
| | | | | | | | share" This reverts commit c2ca99e2f641396b6823c1e675afd3ecb486e744. Signed-off-by: provokateurin <kate@provokateurin.de>