aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | fix: Pass the mountpoint target user to storages without ownerJulius Härtl2024-07-222-0/+17
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Storages that do not have a dedicated owner (e.g. groupfolders, external storages) currently always assume the current session user as the owner. This leads to several issues when there is no user session but a node is obtained through a user folder. In order to have the correct user available we need to pass the user that is used to setup a mountpoint along to the storage layer as we generally assume that an owner is available for those. Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | / / feat: add additional logging for database errorsRobin Appelman2024-07-252-4/+51
| |/ / |/| | | | | | | | | | | | | | 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 #46636 from nextcloud/backport/46608/stable29Anupam Kumar2024-07-241-1/+1
|\ \ \ | |/ / |/| | [stable29] fix(TextProcessing): use error instead of info for exception logging
| * | fix(TextProcessing): use error instead of info for exception loggingAnupam Kumar2024-07-191-1/+1
| | | | | | | | | | | | Signed-off-by: Anupam Kumar <kyteinsky@gmail.com>
* | | Merge pull request #46569 from nextcloud/backport/46342/stable29Josh2024-07-191-3/+3
|\ \ \ | | | | | | | | [stable29] fix(previews): Stop returning true when `getimagesize()` fails
| * | | fix(previews): Stop returning true when getimagesize() failsJosh Richards2024-07-161-3/+3
| | |/ | |/| | | | | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* | | Merge pull request #46626 from nextcloud/backport/46448/stable29Andy Scherzinger2024-07-191-3/+8
|\ \ \ | |_|/ |/| | [stable29] fix(dashboard): Unify widget icon colors and document it's behaviour
| * | fix(dashboard): Document expected icon behaviourJoas Schilling2024-07-191-3/+8
| |/ | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com> [skip ci]
* / fix(mail): Fix big logos in mail templates for OutlookJoas Schilling2024-07-194-2/+116
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.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>
* perf: Set session.cache_limiter at runtime to avoid clients caching static ↵Julius Härtl2024-07-111-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 #45919 from nextcloud/backport/45582/stable29Andy Scherzinger2024-07-101-1/+1
|\ | | | | [stable29] delete background jobs by id when cleaning up
| * fix: delete background jobs by id when cleaning upRobin Appelman2024-07-101-1/+1
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #46069 from nextcloud/backport/45877/stable29Andy Scherzinger2024-07-101-9/+9
|\ \ | | | | | | [stable29] 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 #46035 from nextcloud/backport/46017/stable29Andy Scherzinger2024-07-101-1/+1
|\ \ \ | |_|/ |/| | [stable29] fix: also use optimized getFirstNodeyIdInPath for Folder::getFirstNodeById
| * | fix: also use optimized getFirstNodeyIdInPath for Folder::getFirstNodeByIdRobin Appelman2024-06-211-1/+1
| |/ | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #46298 from nextcloud/backport/46174/stable29Ferdinand Thiessen2024-07-101-5/+10
|\ \ | | | | | | [stable29] 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-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | | Merge pull request #46306 from nextcloud/backport/46276/stable29Daniel2024-07-061-4/+4
|\ \ \ | | | | | | | | [stable29] chore(mp3info): apply upstream patch for invalid array access
| * | | feat: log file id and path when extracting the mp3 cover failsDaniel Kesselberg2024-07-061-4/+4
| |/ / | | | | | | | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de> [skip ci]
* | | Merge pull request #46163 from nextcloud/backport/45876/stable29Daniel2024-07-061-0/+16
|\ \ \ | |/ / |/| | [stable29] fix: Avoid throwing errors for teams are unavailable
| * | fix: Avoid throwing errors for teams are unavailableJulius Härtl2024-06-281-0/+16
| | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | fix(preview): don't create folder structure when previews are disabledDaniel Kesselberg2024-07-021-4/+17
| | | | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | fix(files_sharing): Store the expiration date relative to the server's timezoneLouis Chemineau2024-07-012-20/+29
|/ / | | | | | | | | | | 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(federation): Fix missing protocol on CloudID remoteJoas Schilling2024-06-272-4/+14
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #45995 from nextcloud/backport/45580/stable29Louis2024-06-251-0/+22
|\ \ | | | | | | [stable29] fix: avoid duplicate tag inserts by checking if the mapping exists already in db
| * | fix: removed unused variable and check if tag ids are empty before starting ↵yemkareems2024-06-201-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-201-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-201-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-241-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-192-6/+8
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | feat: add setup check for needed mimetype migrationsRobin Appelman2024-06-192-25/+85
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | fix: move repair mimetype repair step to the expensive stepsRobin Appelman2024-06-191-1/+1
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* 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>
* 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 #45809 from nextcloud/fix/noid/fix-new-token-scope-nameArthur Schiwon2024-06-123-3/+3
|\ | | | | [stable29] fix(Token): make new scope future compatible
| * 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(files): Use isRetryable to catch retryable exceptionsLouis Chemineau2024-06-121-2/+6
|/ | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* fix(Session): avoid password confirmation on SSOArthur Schiwon2024-06-116-40/+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-071-1/+4
| | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* Merge pull request #45379 from nextcloud/backport/45374/stable29John Molakvoæ2024-06-071-0/+1
|\
| * feat: Add `forbidden_filename_characters` to JSConfig for use in frontend ↵Ferdinand Thiessen2024-05-161-0/+1
| | | | | | | | | | | | | | | | | | libraries Add the filename restrictions to our JS config so we can create a common frontend library function to check filename validity (de-duplicate code). Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | Merge pull request #45476 from nextcloud/backport/45459/stable29John Molakvoæ2024-06-071-0/+3
|\ \
| * | fix: register DI alias for IFileAccessRobin Appelman2024-05-231-0/+3
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #45590 from nextcloud/backport/45413/stable29John Molakvoæ2024-06-071-1/+1
|\ \ \
| * | | Try create folder first and check if it exists afterGit'Fellow2024-05-301-1/+1
| | | | | | | | | | | | | | | | Co-authored-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com> Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>