Skip to content

fix: Prevent DiscoverySingleLocalDirectoryJob from hanging when target is file - #10434

Merged
claucambra merged 3 commits into
masterfrom
fix/file-dir-scan
Jul 28, 2026
Merged

fix: Prevent DiscoverySingleLocalDirectoryJob from hanging when target is file#10434
claucambra merged 3 commits into
masterfrom
fix/file-dir-scan

Conversation

@claucambra

Copy link
Copy Markdown
Collaborator

Resolves

#10071

Summary

Should allow the sync process to finish.

Checklist

AI (if applicable)

@camilasan

Copy link
Copy Markdown
Member

@claucambra you just need to fix the compilation of the test on Windows.

@mgallien
mgallien force-pushed the fix/file-dir-scan branch from 5021ac3 to 1b9541e Compare July 23, 2026 13:58
@mgallien
mgallien enabled auto-merge July 23, 2026 13:58
@claucambra
claucambra force-pushed the fix/file-dir-scan branch 3 times, most recently from 439f849 to f605ed4 Compare July 27, 2026 15:04
@claucambra claucambra added this to the 34.0.1 milestone Jul 27, 2026
Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
@claucambra

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0 please

@github-actions

Copy link
Copy Markdown
Contributor

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

Digest: sha256:1fd9dbd9e5df08e36963d724e379ea42061795582564ec2300a288605888df37

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.

@claucambra
claucambra disabled auto-merge July 28, 2026 07:55
@claucambra
claucambra merged commit f402d92 into master Jul 28, 2026
20 of 21 checks passed
@claucambra
claucambra deleted the fix/file-dir-scan branch July 28, 2026 08:20
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)
1 New Bugs (required ≤ 0)
E Reliability Rating on New Code (required ≥ A)
33 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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants