aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
* fix: ensure parent folder exists when writing a file to object storageRobin Appelman2024-09-191-0/+4
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: verify that parent exists in cache when insertingRobin Appelman2024-09-191-0/+3
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: fix object store id for test object storeRobin Appelman2024-09-191-2/+2
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: cleanup objectstore file_put_contentRobin Appelman2024-09-191-7/+4
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: create intermediate directories for objectstore moveFromStorageRobin Appelman2024-09-191-0/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* fix: get source file before moving the cache item in objectstore moveFromStorageRobin Appelman2024-09-191-1/+5
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #48104 from nextcloud/backport/46218/stable29John Molakvoæ2024-09-172-1/+2
|\
| * chore: improve hash_file php usage in Local Storagebackport/46218/stable29John Molakvoæ2024-09-161-8/+1
| | | | | | | | Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * fix(files): Ensure that the hash method does not return nullLouis Chemineau2024-09-162-2/+10
| | | | | | | | | | | | | | | | To match https://github.com/nextcloud/server/blob/beececf66068f57c416225efcde9b44ce5c2e835/lib/private/Files/View.php#L1050 - Fix https://github.com/nextcloud/server/issues/44110 Signed-off-by: Louis Chemineau <louis@chmn.me>
* | fix: switch from explode to substr (faster)backport/46140/stable29Josh Richards2024-09-161-1/+2
| | | | | | | | Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* | fix(config): Add missing handling for envCache in getKeys()Josh Richards2024-09-161-5/+12
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | NC_ env variable overrides were not appearing in the output of `occ config:list system` nor `occ config:system:get xxx`. This was creating nearly impossible to diagnose configuration/ behavior disprepancies. - Refactored readData() so that we aren't saving the entire environment in the envCache anymore (only those prefixed "NC_") and so that we save NC_ provided config values under their real key. - Refactored getValue() to accommodate readData() refactor - Fixed getKeys() to properly return envCache keys too Environment provided config variables now appear in `occ config:list system` as expected. Environment provided config variables now appear when queried via `occ config:system:get KEY` envCache is now free of non-NC stuff. Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
* Merge pull request #47983 from nextcloud/backport/47928/stable29John Molakvoæ2024-09-161-1/+1
|\
| * fix(a11y): Add OTF font loading checkbackport/47928/stable29Joas Schilling2024-09-161-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #47862 from nextcloud/backport/47860/stable29John Molakvoæ2024-09-161-3/+1
|\ \
| * | fix: Only write once to template instead of create/copybackport/47860/stable29Julius Knorr2024-09-101-3/+1
| | | | | | | | | | | | Signed-off-by: Julius Knorr <jus@bitgrid.net>
* | | Merge pull request #47840 from nextcloud/backport/47640/stable29John Molakvoæ2024-09-161-1/+1
|\ \ \ | |_|/ |/| |
| * | fix: Run migration sql as statement so that the primary db node is usedbackport/47640/stable29Julius Härtl2024-09-091-1/+1
| |/ | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #47941 from nextcloud/backport/47933/stable29Joas Schilling2024-09-161-3/+3
|\ \ | | | | | | [stable29] fix(config): Throw PreconditionException always when it didn't match
| * | fix(config): Throw PreconditionException always when it didn't matchbackport/47933/stable29Joas Schilling2024-09-131-3/+3
| |/ | | | | | | | | | | | | | | Previously even when the precondition did not match, the call "passed" when the after value was the expected one. This however can lead to race conditions, duplicate code excutions and other things. Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #47936 from nextcloud/backport/47854/stable29Andy Scherzinger2024-09-151-1/+2
|\ \ | | | | | | [stable29] fix(appstore): Ensure returned apps from AppStore are valid
| * | fix: check if key is setbackport/47854/stable29Git'Fellow2024-09-131-1/+1
| | | | | | | | | | | | Co-authored-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com> Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * | fix(appstore): Ensure returned apps from AppStore are validGit'Fellow2024-09-131-1/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: lint chore: remove space Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: check if response array is null Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> chore: Add log
* | Merge pull request #47867 from nextcloud/backport/47858/stable29John Molakvoæ2024-09-151-0/+3
|\ \ | | | | | | [stable29] fix(preview): gracefully handle file not being opened in ProviderV2
| * | fix(preview): gracefully handle file not being opened in ProviderV2backport/47858/stable29Richard Steinmetz2024-09-101-0/+3
| |/ | | | | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* | Merge pull request #48033 from nextcloud/backport/48013/stable29John Molakvoæ2024-09-151-1/+3
|\ \ | | | | | | [stable29] fix(BaseResponse): Cast XML element values to string
| * | fix(BaseResponse): Cast XML element values to stringbackport/48013/stable29provokateurin2024-09-151-1/+3
| | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | Merge pull request #47871 from nextcloud/backport/47834/stable29John Molakvoæ2024-09-151-2/+9
|\ \ \ | | | | | | | | [stable29] fix(appstore): return if appstore is manually disabled
| * | | fix(appstore): disabled appstore log level bump from debug to infobackport/47834/stable29John Molakvoæ2024-09-101-1/+1
| | | | | | | | | | | | | | | | Co-authored-by: Joas Schilling <213943+nickvergessen@users.noreply.github.com> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
| * | | fix(appstore): return if appstore is manually disabledGit'Fellow2024-09-101-2/+9
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: camel case Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: log a debug log Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: typing Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: typo fix: typing
* | | Merge pull request #47795 from nextcloud/backport/47756/stable29John Molakvoæ2024-09-151-1/+2
|\ \ \ | | | | | | | | [stable29] fix(files): Check if target path is a descendant of the shared folder
| * | | fix(files): Check if the target path is a descendant of the shared folder pathbackport/47756/stable29Git'Fellow2024-09-061-1/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: fix tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: add tests Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com> fix: tests
* | | Merge pull request #47873 from nextcloud/backport/47853/stable29John Molakvoæ2024-09-151-4/+4
|\ \ \ | | | | | | | | [stable29] fix: make swift connect exception message more informative
| * | | fix: make swift connect exception message more informativebackport/47853/stable29Robin Appelman2024-09-101-4/+4
| |/ / | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #48034 from nextcloud/backport/48016/stable29John Molakvoæ2024-09-151-1/+1
|\ \ \ | | | | | | | | [stable29] fix(S3ConfigTrait): Allow proxy field to take false
| * | | fix(S3ConfigTrait): Allow proxy field to take falsebackport/48016/stable29provokateurin2024-09-151-1/+1
| | |/ | |/| | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | Merge pull request #48026 from nextcloud/backport/48014/stable29Kate2024-09-151-2/+1
|\ \ \
| * | | fix(Tags): Return boolean for userHasTags()backport/48014/stable29provokateurin2024-09-151-2/+1
| | | | | | | | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | | | Merge pull request #48020 from nextcloud/backport/48015/stable29John Molakvoæ2024-09-151-1/+1
|\ \ \ \ | |_|/ / |/| | |
| * | | fix(Storage\Local): Do not call getSourcePath() on SplFileInfobackport/48015/stable29provokateurin2024-09-151-1/+1
| |/ / | | | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* / / fix(Comment): Initialize childrenCount as integerbackport/48017/stable29provokateurin2024-09-151-1/+1
|/ / | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix: Do not try to set HTTP response code on already closed connectionbackport/47971/stable29Côme Chilliet2024-09-141-1/+2
| | | | | | | | | | | | | | | | | | This avoids a PHP warning in the logs about trying to set the response code while the output already started. It’s useless to try to print an error page anyway in this situation because the connection was closed already. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | fix(files): Adjust Cache::searchQuery() parameter name to match interfacebackport/47954/stable29provokateurin2024-09-132-4/+4
|/ | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* fix: Use sha256 to hash arguments of background jobsLouis Chemineau2024-09-051-5/+4
| | | | | | This is to prevent collision as we are sometime hashing user input, yet using that hash to target the background job in the database. Signed-off-by: Louis Chemineau <louis@chmn.me>
* Merge pull request #47486 from nextcloud/backport/46013/stable29Andy Scherzinger2024-09-051-0/+25
|\ | | | | [stable29] fix: write object to the correct urn when moving from another storage to object store
| * fix: write object to the correct urn when moving from another storage to ↵Robin Appelman2024-08-261-0/+25
| | | | | | | | | | | | object store Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #47284 from nextcloud/backport/47038/stable29Arthur Schiwon2024-09-051-1/+2
|\ \ | | | | | | [stable29] fix: check for correct storage class when checking for link parent
| * | fix: check for correct storage class when checking for link parentRobin Appelman2024-09-041-1/+2
| | | | | | | | | | | | | | | | | | fixes #39123 Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | fix(appstore): Cache apps.json also on dev instancesMarcel Müller2024-09-051-8/+9
|/ / | | | | | | Signed-off-by: Marcel Müller <marcel-mueller@gmx.de>
* | fix(files): Create non-existent parents of mountpointsprovokateurin2024-09-041-0/+9
| | | | | | | | Signed-off-by: provokateurin <kate@provokateurin.de>
* | fix: lintGit'Fellow2024-09-041-1/+1
| | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>