Skip to content

[stable33] fix: Send password to owner when password sending fails, regardless of enforcement - #64423

Merged
GhassenKefi merged 2 commits into
stable33from
backport/50517/stable33
Sep 19, 2026
Merged

GhassenKefi merged 2 commits into
stable33from
backport/50517/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented Sep 16, 2026

Copy link
Copy Markdown

Backport of PR #50517

…f enforcement

Previously, the `sendPasswordToOwner` method would only trigger if password enforcement was enabled and sending the password to the recipient failed. This behavior was inconsistent with the intended functionality, as the owner should receive the password whenever sending to the recipient fails, regardless of whether password enforcement is enabled.

This change ensures that the owner is notified of the password whenever sending it to the recipient fails, improving consistency and user experience.

Signed-off-by: nfebe <fenn25.fn@gmail.com>

WIP test(share_by_mail): Send password to owner if `send_by_mail` is disabled

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@backportbot
backportbot Bot requested a review from a team as a code owner September 16, 2026 15:58
@backportbot
backportbot Bot requested review from Altahrim, CarlSchwan, artonge, leftybournes, nfebe, salmart-dev and susnux and removed request for a team September 16, 2026 15:58
@backportbot backportbot Bot added this to the Nextcloud 33.0.10 milestone Sep 16, 2026
@susnux
susnux requested review from GhassenKefi and removed request for nfebe September 16, 2026 15:59
@susnux susnux assigned GhassenKefi and unassigned nfebe Sep 16, 2026
Signed-off-by: Ghassen kefi <ghassen.kefi.dev@gmail.com>
@GhassenKefi
GhassenKefi force-pushed the backport/50517/stable33 branch from 0216a38 to 3845049 Compare September 19, 2026 15:14
@GhassenKefi
GhassenKefi merged commit d1e3815 into stable33 Sep 19, 2026
173 of 175 checks passed
@GhassenKefi
GhassenKefi deleted the backport/50517/stable33 branch September 19, 2026 17:20
@GhassenKefi GhassenKefi removed their assignment Sep 19, 2026
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.

4 participants