Skip to content

[stable32] fix(files_sharing): Only send password on change - #55983

Merged
nfebe merged 2 commits into
stable32from
backport/55580/stable32
Oct 29, 2025
Merged

[stable32] fix(files_sharing): Only send password on change#55983
nfebe merged 2 commits into
stable32from
backport/55580/stable32

Conversation

@backportbot

@backportbotbackportbotBot commented Oct 24, 2025

Copy link
Copy Markdown

Backport of #55580

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

Todo

  • Review and resolve any conflicts
  • 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.

@nfebe
nfebe marked this pull request as ready for review October 24, 2025 12:16
@nfebe
nfebe requested review from a team as code ownersOctober 24, 2025 12:16
@nfebe
nfebe requested review from leftybournes, salmart-dev, szaimen and yemkareems and removed request for a teamOctober 24, 2025 12:16
@nfebe
nfebe enabled auto-merge October 24, 2025 12:16
@nfebe

Copy link
Copy Markdown
Contributor

/compile

@AndyScherzinger

Copy link
Copy Markdown
Member

/compile rebase

nfebeand others added 2 commits October 29, 2025 19:10
The password param should never be sent if the intention is not
remove it or update it.
This commit adapts the frontend and backend to this rule to avoid weird bugs
especially around updating new shares.
Signed-off-by: nfebe <fenn25.fn@gmail.com>
[skip ci]
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nfebe
nfebe merged commit be74a15 into stable32Oct 29, 2025
206 of 208 checks passed
@nfebe
nfebe deleted the backport/55580/stable32 branch October 29, 2025 19:56
@blizzzblizzz mentioned this pull request Nov 10, 2025
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsfeature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nfebe@AndyScherzinger@nextcloud-command