Skip to content

fix(file-provider): Deletion of server-side item for items excluded from sync - #10534

Merged
Rello merged 9 commits into
masterfrom
fix/exclusion-deletion
Aug 5, 2026
Merged

fix(file-provider): Deletion of server-side item for items excluded from sync#10534
Rello merged 9 commits into
masterfrom
fix/exclusion-deletion

Conversation

@claucambra

@claucambra claucambra commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Resolves

#10521 (comment)

Summary

Returning NSFileProviderError(.excludedFromSync) from an item creation or modification callback starts a system-managed removal sequence. It does not only report that the operation was rejected. This currently causes an item's standard deletion pipeline to be called, which culminates in a server deletion.

This PR adds a marker for items excluded from sync to prevent this from happening.

Checklist

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
…rker

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
…ed item

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
…te to server

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
…dification

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
…n modifying bundle excluded from sync

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
@claucambra claucambra self-assigned this Aug 5, 2026
@claucambra
claucambra requested a review from camilasan as a code owner August 5, 2026 02:20
@claucambra claucambra added the bug label Aug 5, 2026
@claucambra claucambra added os: 🍎 macOS Apple macOS, formerly also known as OS X confirmed bug approved by the team feature: 📁 file provider macOS File Provider Extension, more general also known as virtual file system. labels Aug 5, 2026
@claucambra claucambra added this to the 34.0.1 milestone Aug 5, 2026
@claucambra

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0 please

@claucambra claucambra modified the milestones: 34.0.1, 34.0.2 Aug 5, 2026
@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

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

Digest: sha256:bf5678fb4feca8d0b8792cb7df8f89fd291c738629b03cb4694e75e444ff71f5

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.

@Rello

Rello commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

gave it another test with my error example - works nicely

@Rello
Rello merged commit 38a56db into master Aug 5, 2026
23 checks passed
@Rello
Rello deleted the fix/exclusion-deletion branch August 5, 2026 22:24
@github-actions

Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

@i2h3

i2h3 commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Please note that this is not waterproof because of #10106.

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

Labels

bug confirmed bug approved by the team feature: 📁 file provider macOS File Provider Extension, more general also known as virtual file system. feedback-requested os: 🍎 macOS Apple macOS, formerly also known as OS X

Projects

None yet

Development

Successfully merging this pull request may close these issues.

macOS client 34.x issues a **server-side** DELETE for .app bundles it refuses to sync — permanent data loss when the item is a dataless placeholder

5 participants