aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | fix: Remove support for app.php loadingCôme Chilliet2025-04-142-48/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It has been deprecated for a long time, and the last known active application to use it (user_saml) is now migrated the modern API. Presence of the file is still checked in order to log an error. This behavior may be removed as well in a few versions. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | | fix(Log): revert swich to ?? operatorfix/noid/log-false-userArthur Schiwon2025-04-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | false is expected, not null. The changed caused "user" in the log files to be false instead of "--", which is breaking behaviour. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | | | fix(mail): use matching text color for primary elementsfix/use-also-default-textFerdinand Thiessen2025-04-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | | | | Merge pull request #50465 from nextcloud/zorn-v-patch-1John Molakvoæ2025-04-171-1/+1
|\ \ \ \ \ \
| * | | | | | Check that user actually can validate password for jszorn-v-patch-1zorn-v2025-04-161-1/+1
| | |/ / / / | |/| | | | | | | | | | Signed-off-by: zorn-v <zorn7@yandex.ru>
* / | | | | feat(ip): add configurable IPv6 subnet for BFP and throttlingfeat/add-configurable-ipv6-subnetBenjamin Gaussorgues2025-04-171-4/+11
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | | | | Merge pull request #51020 from nextcloud/artonge/fix/transfer_ownershipLouis2025-04-141-1/+12
|\ \ \ \ \
| * | | | | fix: Transfer ownership with S3 as primaryartonge/fix/transfer_ownershipLouis Chemineau2025-04-011-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 #52086 from nextcloud/fix/cleanup-user-backendsCôme Chilliet2025-04-142-37/+18
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Cleanup user backends related methods
| * | | | | fix: Deprecate OC_User methods related to user backendsCôme Chilliet2025-04-102-37/+18
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | | fix(setup): ignore missing theming appupstream/52135/52135-masterMaxence Lange2025-04-121-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | | | Merge pull request #52085 from ↵Anna2025-04-101-2/+5
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | nextcloud/bugfix/noid/fix-cached-return-of-display-name fix(federation): Fix returning "no display name" after cache result
| * | | | | fix(federation): Fix returning "no display name" after cache resultbugfix/noid/fix-cached-return-of-display-nameJoas Schilling2025-04-101-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | | Merge pull request #52050 from nextcloud/fix/noid/taskprocessing-appapiAlexander Piskun2025-04-101-1/+45
|\ \ \ \ \ \ | |/ / / / / |/| | | | | fix(taskprocessing): use the event for AppAPI to get list of AI providers
| * | | | | fix(taskprocessing): use the event for AppAPI to get list of AI providersfix/noid/taskprocessing-appapiOleksander Piskun2025-04-091-1/+45
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Oleksander Piskun <oleksandr2088@icloud.com>
* | | | | | Merge pull request #52082 from nextcloud/default-navigation-reuseKate2025-04-101-1/+2
|\ \ \ \ \ \
| * | | | | | fix: reuse default navigation entry when updating navigation entriesdefault-navigation-reuseRobin Appelman2025-04-091-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | | | fix(files_sharing): fix share creation error handlingskjnldsv2025-04-091-1/+5
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
* | | | | | fix(federation): Don't load the addressbook when resolving a cloud IDperf/noid/dont-load-addressbook-on-resolving-cloudidJoas Schilling2025-04-092-28/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead we delay the lookup of the display name until it is actually used Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | | Revert "refactor(RichObjectStrings): Only log error if key or value is not ↵revert/52035Joas Schilling2025-04-091-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | string in validator" This reverts commit fd156d3408a030c6bb98d9cdaf3790375d766eb5. Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | | Merge pull request #51838 from nextcloud/feat/cleanup-oc-util-methodsJohn Molakvoæ2025-04-087-142/+135
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Mark all methods of OC_Util as deprecated
| * | | | | fix: add more details in documented alternativesfeat/cleanup-oc-util-methodsCôme Chilliet2025-04-081-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Louis <louis@chmn.me> Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
| * | | | | fix: Fix types for sharingDisabledForUserCôme Chilliet2025-04-073-9/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and use the non-deprecated version whenever possible Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | fix: Put back isHtaccessWorking method in Setup where it is usedCôme Chilliet2025-04-071-2/+84
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | fix: Remove some call and references to deprecated OC_Util classCôme Chilliet2025-04-073-10/+6
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | chore(legacy): Flag methods as deprecated, with appropriate replacementCôme Chilliet2025-04-071-34/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when possible. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | chore(legacy): Remove unused protected method in OC_UtilCôme Chilliet2025-04-071-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | chore(legacy): Officially deprecate OC_Util and remove non-static methodsCôme Chilliet2025-04-071-84/+3
| | |_|_|/ | |/| | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | Merge pull request #52013 from ↵Ferdinand Thiessen2025-04-081-1/+6
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/handle-missing-share-providers-when-promoting-reshares fix: Handle missing share providers when promoting reshares
| * | | | | fix: Handle missing share providers when promoting reshareshandle-missing-share-providers-when-promoting-resharesDaniel Calviño Sánchez2025-04-081-1/+6
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The provider for mail shares is not available when the "sharebymail" app is disabled, and in that case a "ProviderException" is thrown when trying to get it. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | Merge pull request #52008 from nextcloud/fix/cache-hit-getFirstNodeByIdRobin Appelman2025-04-081-6/+10
|\ \ \ \ \ | | | | | | | | | | | | fix: Proper order for checking path prefix for getting file by id from cache
| * | | | | fix: Catch old cached paths and fetch the new onefix/cache-hit-getFirstNodeByIdJulius Knorr2025-04-081-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
| * | | | | fix: Proper order for checking path prefix for getting file by id from cacheJulius Knorr2025-04-071-1/+1
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | | | | Merge pull request #52014 from nextcloud/fix/fix-movie-preview-constructKate2025-04-082-47/+39
|\ \ \ \ \ | | | | | | | | | | | | fix(preview): Fix constructor parameter name and default value
| * | | | | fix(previews): Cleanup Movie provider code and remove deprecated static varsfix/fix-movie-preview-constructCôme Chilliet2025-04-082-45/+37
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| * | | | | fix(preview): Fix constructor parameter name and default valueCôme Chilliet2025-04-081-2/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | This should fix tests for movie preview provider Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | | refactor(RichObjectStrings): Only log error if key or value is not string in ↵fix/richobjectstrings/validator-string-key-value-errorprovokateurin2025-04-081-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | validator Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | | feat(ip): use larger IPv6 range by defaultfeat/larger_ipv6_rangeBenjamin Gaussorgues2025-04-081-4/+4
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | | Merge pull request #51712 from invario/masterStephan Orbaugh2025-04-071-7/+49
|\ \ \ \ | | | | | | | | | | Enhancement: Better previews for HDR video (attempt #2)
| * | | | feat(previews): fix casing and commentsinvario2025-04-061-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: invario <67800603+invario@users.noreply.github.com>
| * | | | feat(previews): Clean up code for HDR video previewsinvario2025-04-031-16/+10
| | | | | | | | | | | | | | | | | | | | Signed-off-by: invario <67800603+invario@users.noreply.github.com>
| * | | | feat(previews): Use proper colorspace and apply tonemap for HDR videosinvario2025-04-031-4/+51
| | |/ / | |/| | | | | | | | | | Signed-off-by: KT <invario@gmail.com>
* | | | Merge pull request #51927 from nextcloud/fix/tags-eventsFerdinand Thiessen2025-04-071-0/+13
|\ \ \ \ | | | | | | | | | | fix(systemtags): emit assign and unassign bulk tagging events
| * | | | fix(systemtags): emit assign and unassign bulk tagging eventsfix/tags-eventsJohn Molakvoæ (skjnldsv)2025-04-041-0/+13
| | | | | | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | | | Merge pull request #51982 from nextcloud/revert/email-setting-migrationKate2025-04-071-1/+2
|\ \ \ \ \ | | | | | | | | | | | | Revert "refactor: add migration for email setting"
| * | | | | Revert "refactor: add migration for email setting"revert/email-setting-migrationprovokateurin2025-04-071-1/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e7859f0dacbda051c2decb2e0d62345435ac2fd9. Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | | Merge pull request #51942 from nextcloud/fix/require-update-if-mtime-is-nullFerdinand Thiessen2025-04-071-1/+1
|\ \ \ \ \ | | | | | | | | | | | | fix(cache): always require updates if mtime is null
| * | | | | fix(cache): always require updates if mtime is nullfix/require-update-if-mtime-is-nullFerdinand Thiessen2025-04-041-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Resolves https://github.com/nextcloud/server/issues/51941 Due to strong typings we introduced the parameter needs to be an integer. Previously it was `null` which was equal to `0`. So if there is no storage mtime we need to update the cache. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | | Merge pull request #51933 from nextcloud/fix/master-template-layoutKate2025-04-071-13/+4
|\ \ \ \ \ | | | | | | | | | | | | fix: ensure enabled (enforced) themes are always set on template layout
| * | | | | fix: ensure enabled (enforced) themes are always set on template layoutfix/master-template-layoutFerdinand Thiessen2025-04-041-13/+4
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>