Skip to content

[stable-33.0] fix(file-provider): invalidate lock tokens when file paths change. #10186

Merged
mgallien merged 3 commits into
stable-33.0from
backport/10135/stable-33.0
Jun 15, 2026
Merged

[stable-33.0] fix(file-provider): invalidate lock tokens when file paths change. #10186
mgallien merged 3 commits into
stable-33.0from
backport/10135/stable-33.0

Conversation

@backportbot

@backportbot backportbot Bot commented Jun 15, 2026

Copy link
Copy Markdown

Backport of #10135

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

WebDAV lock tokens are bound to the path they were issued for.
When a parent folder moves, child lock tokens become stale and
cause 412/423 upload failures. Clear tokens on rename propagation,
server metadata sync, and upload lock errors.

Assisted-by: Claude:claude-sonnet-4-5
Signed-off-by: Camila Ayres <hello@camilasan.com>
Assisted-by: Claude:claude-sonnet-4-5
Signed-off-by: Camila Ayres <hello@camilasan.com>
@backportbot backportbot Bot requested review from camilasan and i2h3 June 15, 2026 07:24
@backportbot backportbot Bot added this to the 33.0.6 milestone Jun 15, 2026
testParent404ReportsParentAsDeleted wrote metadata then awaited the
enumerator without holding a Realm reference. Under parallel testing the
in-memory store was freed before the enumerator reopened it on another
thread, so the materialized-items query came back empty and the 404 parent
was never reported deleted. Retain the Realm for each test lifetime.

Assisted-by: Claude:claude-opus-4-8
Signed-off-by: Camila Ayres <hello@camilasan.com>
@camilasan camilasan force-pushed the backport/10135/stable-33.0 branch from 69d5b47 to 45547e5 Compare June 15, 2026 07:38
@camilasan camilasan marked this pull request as ready for review June 15, 2026 07:38
@github-actions

Copy link
Copy Markdown
Contributor

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

Digest: sha256:eb7ebf03f3ffa9c60a5e0ad335b0466223d905dcff42927ece559773665d18f9

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 eee30a8 into stable-33.0 Jun 15, 2026
19 of 22 checks passed
@mgallien mgallien deleted the backport/10135/stable-33.0 branch June 15, 2026 08:52
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