Skip to content

[stable34] Fix error when creating mail shares if custom tokens are enabled#60508

Open
backportbot[bot] wants to merge 2 commits into
stable34from
backport/60378/stable34
Open

[stable34] Fix error when creating mail shares if custom tokens are enabled#60508
backportbot[bot] wants to merge 2 commits into
stable34from
backport/60378/stable34

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 18, 2026

Backport of PR #60378

@backportbot backportbot Bot requested review from a team as code owners May 18, 2026 14:17
@backportbot backportbot Bot requested review from danxuliu, nfebe, sorbaugh and susnux and removed request for a team May 18, 2026 14:17
@backportbot backportbot Bot requested review from Altahrim and artonge and removed request for a team May 18, 2026 14:17
@backportbot backportbot Bot added this to the Nextcloud 34 milestone May 18, 2026
danxuliu and others added 2 commits May 18, 2026 15:13
When a new share is saved it is first created in the backend and then
updated with some additional attributes set in the frontend. If custom
tokens are enabled the attributes to update also include the token.
However, for new shares the token is set by the backend when it is
created, it is not defined by the frontend, so the token returned by the
backend needs to be copied to the share data in the frontend. Otherwise
the update would fail because an empty token is sent.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@backportbot backportbot Bot force-pushed the backport/60378/stable34 branch from c828ff8 to fdb4485 Compare May 18, 2026 15:13
Copy link
Copy Markdown
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

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.

3 participants