Skip to content

[stable-33.0] fix(upload): skip locked files during upload to avoid errors#9917

Merged
mgallien merged 5 commits intostable-33.0from
backport/9810/stable-33.0
Apr 27, 2026
Merged

[stable-33.0] fix(upload): skip locked files during upload to avoid errors#9917
mgallien merged 5 commits intostable-33.0from
backport/9810/stable-33.0

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented Apr 27, 2026

Backport of PR #9810

@backportbot backportbot Bot requested a review from Aiiaiiio as a code owner April 27, 2026 11:57
@backportbot backportbot Bot requested a review from nilsding April 27, 2026 11:57
@backportbot backportbot Bot added this to the 33.0.3 milestone Apr 27, 2026
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
will ensure we watch them and sync them as soon as they are no longer
locked

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
fix behavior to match expectations from existing automated tests

ignoring locked files that would have been uploaded (i.e. not
downloaded)

ignoring locked files that have not match another ignore rule

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien force-pushed the backport/9810/stable-33.0 branch from ef4d70a to 22e70fc Compare April 27, 2026 12:03
@mgallien mgallien enabled auto-merge April 27, 2026 12:05
@github-actions
Copy link
Copy Markdown

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

Digest: sha256:618365302a056101c99c511a153027893631c7414df8d10b8733813b765dfbb1

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 merged commit b50743c into stable-33.0 Apr 27, 2026
17 checks passed
@mgallien mgallien deleted the backport/9810/stable-33.0 branch April 27, 2026 12:18
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
33.3% Coverage on New Code (required ≥ 80%)
3 New Code Smells (required ≤ 0)
B Maintainability Rating on New Code (required ≥ A)

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.

1 participant