Skip to content

fix(lscoljob): disable Qt's decompression check to allow listing of really large folders again - #9988

Merged
mgallien merged 1 commit into
masterfrom
bugfix/noid/propfind-decompression
Jun 1, 2026
Merged

fix(lscoljob): disable Qt's decompression check to allow listing of really large folders again#9988
mgallien merged 1 commit into
masterfrom
bugfix/noid/propfind-decompression

Conversation

@nilsding

@nilsding nilsding commented May 7, 2026

Copy link
Copy Markdown
Member

the proper fix would be to rework LsColJob to make use of Nextcloud 31's paginated propfind functionality

Resolves

#

Summary

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@mgallien
mgallien force-pushed the bugfix/noid/propfind-decompression branch from 6067935 to f5d339e Compare May 21, 2026 14:33
@nilsding
nilsding marked this pull request as ready for review May 21, 2026 14:58
@nilsding
nilsding requested review from camilasan and i2h3 as code owners May 21, 2026 14:58
…eally large folders again

the proper fix is to rework LsColJob to make use of Nextcloud 31's
paginated propfind functionality

Signed-off-by: Jyrki Gadinger <[email protected]>
@mgallien
mgallien force-pushed the bugfix/noid/propfind-decompression branch from f5d339e to 1c0237b Compare June 1, 2026 12:04
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-9988.zip

Digest: sha256:daccb6df3e47af43d66c3eb2358e13c003f171233d70f88aeb0fd2a8c6b66967

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@mgallien
mgallien enabled auto-merge June 1, 2026 12:21
@mgallien
mgallien merged commit 3f5fae8 into master Jun 1, 2026
21 checks passed
@mgallien
mgallien deleted the bugfix/noid/propfind-decompression branch June 1, 2026 13:07
@sonarqubecloud

sonarqubecloud Bot commented Jun 1, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
21 Security Hotspots
E Maintainability Rating on New Code (required ≥ A)
14 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@nilsding

Copy link
Copy Markdown
Member Author

/backport to stable-33.0 please

@nilsding

Copy link
Copy Markdown
Member Author

/backport to stable-4.0 please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants