aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #46678 from nextcloud/backport/44295/stable28Andy Scherzinger2024-07-253-3/+11
|\ | | | | [stable28] Allow injecting the user temporarily for direct editing
| * fix(logging): Restore the option to log with debug levelJoas Schilling2024-07-221-2/+2
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * fix: Implement option to temporarily set the user sessionJulius Härtl2024-07-222-1/+11
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| * fix: Do not use incognito mode for direct editingJulius Härtl2024-07-221-2/+0
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #46683 from nextcloud/backport/46674/stable28Andy Scherzinger2024-07-251-12/+21
|\ \ | | | | | | [stable28] feat: add config flag to toggle persistent redis connections
| * | feat: add config flag to toggle persistent redis connectionsRobin Appelman2024-07-221-12/+21
| |/ | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | feat: add additional logging for database errorsRobin Appelman2024-07-252-4/+71
| | | | | | | | | | | | including the stack trace of the current database transaction Signed-off-by: Robin Appelman <robin@icewind.nl>
* | fix: FileInfo from `View` should have the correct name of a mountpointFerdinand Thiessen2024-07-241-0/+4
|/ | | | | | | | | | If a mountpoint is returned from a `View` the name must match the view scoped name and not the global name. For example group folders have a name like `1` or `2` (the ID), but the name of the FileInfo returned fro mthe View should be the mount point name, like `First groupfolder` or similar. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #46628 from nextcloud/backport/46419/stable28Joas Schilling2024-07-224-2/+116
|\ | | | | [stable28] fix(mail): Fix big logos in mail templates for Outlook
| * fix(mail): Fix big logos in mail templates for OutlookJoas Schilling2024-07-194-2/+116
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #46577 from nextcloud/backport/46555/stable28Git'Fellow2024-07-191-0/+4
|\ \ | | | | | | [stable28] fix(appstore): Return an empty array in case of fail
| * | chore: add warning in case of failureGit'Fellow2024-07-171-0/+1
| | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * | fix(appstore): Return an empty array in case of failGit'Fellow2024-07-171-0/+3
| |/ | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | Merge pull request #46520 from nextcloud/backport/46475/stable28Git'Fellow2024-07-191-1/+1
|\ \ | | | | | | [stable28] fix(redis): Require version >=4 for unlink
| * | fix(redis): Require version >=4 for unlinkGit'Fellow2024-07-151-1/+1
| |/ | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* / fix(TextProcessing): use error instead of info for exception loggingAnupam Kumar2024-07-191-1/+1
|/ | | | Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
* fix(Session): avoid race conditions on clustered setupsArthur Schiwon2024-07-113-46/+19
| | | | | | | | - re-stablishes old behaviour with cache to return null instead of throwing an InvalidTokenException when the token is cached as non-existing - token invalidation and re-generation are bundled in a DB transaction now Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #46394 from nextcloud/backport/46320/stable28Joas Schilling2024-07-111-0/+1
|\ | | | | [stable28] perf: Set session.cache_limiter at runtime to avoid clients caching static assets served by PHP
| * perf: Set session.cache_limiter at runtime to avoid clients caching static ↵Julius Härtl2024-07-101-0/+1
| | | | | | | | | | | | | | | | | | | | | | assets served by PHP By default there is a Pragma: no-cache header set due to the default value `no-cache` of session.cache-limiter, which will cause Chrome and iOS to not cache even with a different Cache-Control header set on the response. Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #46067 from nextcloud/backport/45877/stable28Andy Scherzinger2024-07-101-9/+9
|\ \ | | | | | | [stable28] fix: don't use custom certificate bundle if no customer certificates are configured
| * | fix: don't use custom certificate bundle if no customer certificates are ↵Robin Appelman2024-06-241-9/+9
| | | | | | | | | | | | | | | | | | configured Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #45920 from nextcloud/backport/45582/stable28Andy Scherzinger2024-07-101-1/+1
|\ \ \ | |_|/ |/| | [stable28] delete background jobs by id when cleaning up
| * | fix: delete background jobs by id when cleaning upRobin Appelman2024-06-171-1/+1
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #46297 from nextcloud/backport/46174/stable28Git'Fellow2024-07-091-7/+9
|\ \ \ | | | | | | | | [stable28] fix(IntegrityCheck): Ensure the check is run if no results are available
| * | | fix(IntegrityCheck): Ensure the check is run if no results are availableFerdinand Thiessen2024-07-051-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | | fix(s3): Don't wait indefinitely for S3 to returnGit'Fellow2024-07-091-29/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: lint Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: use AwsException Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Throw on connection failure Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Wrap all in try catch block Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: use RequestTimeout error message Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> log: use OCP Server class Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Handle connect timeout only Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: Handle errors more generically Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | | | feat: log file id and path when extracting the mp3 cover failsDaniel Kesselberg2024-07-081-4/+4
|/ / / | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Merge pull request #46141 from nextcloud/backport/46097/stable28Louis2024-07-012-21/+30
|\ \ \ | | | | | | | | [stable28] fix(files_sharing): Also set the expiration date timezone during validation
| * | | fix(files_sharing): Store the expiration date relative to the server's timezoneLouis Chemineau2024-06-272-21/+30
| | | | | | | | | | | | | | | | | | | | | | | | This is needed as we want to store the difference between the server's and the user's timezones. Signed-off-by: Louis Chemineau <louis@chmn.me>
* | | | fix: removed unused variable and check if tag ids are empty before starting ↵yemkareems2024-06-271-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | the inserts Signed-off-by: yemkareems <yemkareems@gmail.com>
* | | | fix: filtering tags based on existing tags and doing a transaction around ↵yemkareems2024-06-271-11/+14
| | | | | | | | | | | | | | | | | | | | | | | | the select and insert Signed-off-by: yemkareems <yemkareems@gmail.com>
* | | | fix: do a select in systemtag_object_mapping to see if tag exists already in ↵yemkareems2024-06-271-6/+20
|/ / / | | | | | | | | | | | | | | | db. if it does not exist alone insert the same or else do nothing Signed-off-by: yemkareems <yemkareems@gmail.com>
* | / fix(Token): add FILESYSTEM scope with SCOPE_SKIP_PASSWORD_VALIDATIONArthur Schiwon2024-06-251-1/+4
| |/ |/| | | | | | | | | | | | | | | | | The scope design requires scopes to be either not specified, or specified explicitely. Therefore, when setting the skip-password-validation scope for user authentication from mechanisms like SAML, we also have to set the filesystem scope, otherwise they will lack access to the filesystem. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | fix: set mimetype_version during upgrade to the old versionRobin Appelman2024-06-212-6/+8
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | feat: add setup check for needed mimetype migrationsRobin Appelman2024-06-212-23/+83
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | fix: move repair mimetype repair step to the expensive stepsRobin Appelman2024-06-211-1/+1
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* refactor(shareManager): Use constructor property promotionfenn-cs2024-06-131-75/+23
| | | | Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* fix(shareManager): Respect empty `expireDate` in serverfenn-cs2024-06-132-79/+106
| | | | | | If `expireDate` is an empty string and not `null` then the server should not set a default. Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
* refactor: make share manager buildableRobin Appelman2024-06-132-29/+6
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #45839 from nextcloud/backport/45649/stable28Andy Scherzinger2024-06-131-1/+1
|\ | | | | [stable28] fix(ObjectStore): Fix regression when hostname is empty
| * fix(ObjectStore): handle empty S3 hostnameJosh2024-06-131-1/+1
| | | | | | | | | | | | | | | | Fixes #45637 The support for s3-accelerate added in #44496 introduced a regression in AWS S3 environments when `hostname` is blank (which is a valid configuration w/ AWS since the hostname gets auto-generated). Signed-off-by: Josh <josh.t.richards@gmail.com>
* | Merge pull request #45857 from nextcloud/backport/45172/stable28Andy Scherzinger2024-06-131-1/+5
|\ \ | | | | | | [stable28] feat: add option to disable scanner transactions
| * | feat: add option to disable scanner transactionsRobin Appelman2024-06-131-1/+5
| |/ | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* / fix: get child ids for folder in a separate query during moveRobin Appelman2024-06-131-4/+21
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #45807 from nextcloud/backport/45768/stable28Arthur Schiwon2024-06-131-2/+6
|\ | | | | [stable28] Use isRetryable to catch retryable exceptions
| * fix(files): Use isRetryable to catch retryable exceptionsLouis Chemineau2024-06-121-2/+6
| | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* | Merge pull request #45704 from nextcloud/backport/43942/stable28Arthur Schiwon2024-06-126-41/+76
|\ \ | |/ |/| [stable28] fix(Session): avoid password confirmation on SSO
| * fix(Token): make new scope future compatibleArthur Schiwon2024-06-123-3/+3
| | | | | | | | | | | | | | - "password-unconfirmable" is the effective name for 30, but a draft name was backported. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * fix(Session): avoid password confirmation on SSOArthur Schiwon2024-06-126-41/+76
| | | | | | | | | | | | | | | | | | | | | | SSO backends like SAML and OIDC tried a trick to suppress password confirmations as they are not possible by design. At least for SAML it was not reliable when existing user backends where used as user repositories. Now we are setting a special scope with the token, and also make sure that the scope is taken over when tokens are regenerated. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | fix(Mailer): Allow to enforce strict email formatFerdinand Thiessen2024-06-111-1/+4
| | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>