Skip to content

[stable32] fix(sessions): only use share node matching the session document - #9523

Merged
mejo- merged 2 commits into
stable32from
backport/9521/stable32
Nov 4, 2026
Merged

mejo- merged 2 commits into
stable32from
backport/9521/stable32

Conversation

@backportbot

@backportbot backportbot Bot commented Nov 3, 2026

Copy link
Copy Markdown

Backport of PR #9521

getFileForSession() returned the node of a single-file share without
checking it against the session's document, so the share token passed
with push, sync and save could belong to a different file. Require the
resolved file id to match the session document id.

Alaready fixed in stable35 and above, where getFileByIdFromShare() does
the same check, as part of #9062.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: ClaudeCode:claude-opus-5-5
@backportbot
backportbot Bot requested review from benjaminfrueh and mejo- November 3, 2026 20:13
@backportbot backportbot Bot added bug Something isn't working 3. to review feature: sharing labels Nov 3, 2026
@mejo- mejo- changed the title [stable32] [stable34] fix(sessions): only use share node matching the session document [stable32] fix(sessions): only use share node matching the session document Nov 3, 2026
@mejo-
mejo- enabled auto-merge November 3, 2026 20:13
@mejo-
mejo- requested a review from max-nextcloud as a code owner November 4, 2026 09:04
Lower versions don't work on ubuntu-24.02 runners.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: ClaudeCode:claude-opus-5-5
@mejo-
mejo- force-pushed the backport/9521/stable32 branch from 22a720f to 7321db9 Compare November 4, 2026 11:51
@mejo-
mejo- disabled auto-merge November 4, 2026 11:51
@mejo-
mejo- merged commit 9ed0615 into stable32 Nov 4, 2026
67 checks passed
@mejo-
mejo- deleted the backport/9521/stable32 branch November 4, 2026 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant