Skip to content

Check for file provider upload size mismatch - #10369

Merged
i2h3 merged 2 commits into
masterfrom
i2h3/file-provider-upload-size-check
Jul 16, 2026
Merged

Check for file provider upload size mismatch#10369
i2h3 merged 2 commits into
masterfrom
i2h3/file-provider-upload-size-check

Conversation

@i2h3

@i2h3 i2h3 commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

This is part of reliability improvements derived from issues in use with Adobe Creative Cloud.

Previously, we have not even verified for the upload item to be of the same size as the local item. Further paths with checks based on checksums are out of scope for this release because they require significantly more effort including on the server side.

Checklist

AI (if applicable)

Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
@i2h3 i2h3 added this to the 34.0.0 milestone Jul 14, 2026
@i2h3 i2h3 self-assigned this Jul 14, 2026
@i2h3 i2h3 added the bug label Jul 14, 2026
@i2h3
i2h3 requested a review from camilasan as a code owner July 14, 2026 14:21
@i2h3 i2h3 added the os: 🍎 macOS Apple macOS, formerly also known as OS X label Jul 14, 2026
@i2h3
i2h3 requested review from mgallien and nilsding as code owners July 14, 2026 14:21
@i2h3 i2h3 added the feature: 📁 file provider macOS File Provider Extension, more general also known as virtual file system. label Jul 14, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 💻 Desktop Clients team Jul 14, 2026
@i2h3

i2h3 commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0

@i2h3 i2h3 moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 💻 Desktop Clients team Jul 14, 2026
MockRemoteInterfaceTests.testEnumerate compared the enumerated root file's date against the item's creationDate, but toNKFile() maps modificationDate onto NKFile.date. MockRemoteItem defaults creationDate and modificationDate to two separate Date() calls that are only sometimes identical, so the assertion flaked on CI when the clock ticked between them. Compare against modificationDate, the field the value is actually derived from.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Iva Horn <iva.horn@nextcloud.com>
@i2h3 i2h3 mentioned this pull request Jul 15, 2026
10 tasks
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

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

Digest: sha256:199e88f36c16153d84ee4851ba7c52ed91931a7857faced50e0bd04f21463f35

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.

@i2h3
i2h3 merged commit 9e3c9ca into master Jul 16, 2026
23 checks passed
@github-project-automation github-project-automation Bot moved this from 🏗️ In progress to ☑️ Done in 💻 Desktop Clients team Jul 16, 2026
@i2h3
i2h3 deleted the i2h3/file-provider-upload-size-check branch July 16, 2026 07:45
@mgallien mgallien modified the milestones: 34.0.0, 35.0.0 Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug feature: 📁 file provider macOS File Provider Extension, more general also known as virtual file system. os: 🍎 macOS Apple macOS, formerly also known as OS X

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

3 participants