Skip to content

[stable32] fix(apppasswords): Require password also on delete and update - #59607

Merged
nickvergessen merged 2 commits into
stable32from
backport/59602/stable32
Apr 13, 2026
Merged

[stable32] fix(apppasswords): Require password also on delete and update#59607
nickvergessen merged 2 commits into
stable32from
backport/59602/stable32

Conversation

@backportbot

@backportbotbackportbotBot commented Apr 13, 2026•
edited
Loading

Copy link
Copy Markdown

@nickvergessen@backportbot
fix(apppasswords): Require password also on delete and update
03cff6f
Signed-off-by: Joas Schilling <coding@schilljs.com>
@backportbot
backportbotBot requested review from a team as code ownersApril 13, 2026 13:53
@backportbot
backportbotBot requested review from ArtificialOwl, leftybournes, miaulalala, nfebe, nickvergessen, sorbaugh and szaimen and removed request for a teamApril 13, 2026 13:53
@backportbotbackportbotBot added this to the Nextcloud 32.0.9 milestone Apr 13, 2026
szaimen
szaimen approved these changes Apr 13, 2026
View reviewed changes

@szaimenszaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🐘

@nickvergessen

nickvergessen commented Apr 13, 2026

Copy link
Copy Markdown
Member

/compile

@nextcloud-command
chore(assets): Recompile assets
f82e0e8
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nickvergessen
nickvergessen merged commit f4d8f45 into stable32Apr 13, 2026
193 of 195 checks passed
@nickvergessen
nickvergessen deleted the backport/59602/stable32 branch April 13, 2026 15:31
@nextcloud-botnextcloud-bot mentioned this pull request Apr 20, 2026
Merged
@joshtrichards

joshtrichards commented May 12, 2026

Copy link
Copy Markdown
Member

This backport looks incomplete - I think it's missing the PwdConfirmationMode import (and maybe the strict / interceptor stuff). Noticed because a forum user reported not being able to delete tokens on v32.0.9: https://help.nextcloud.com/t/deleting-access-token-not-working/244370

@ernolfernolf mentioned this pull request May 13, 2026
Closed
8 tasks
@nickvergessennickvergessen mentioned this pull request May 13, 2026
Merged
7 tasks
@nickvergessen

nickvergessen commented May 13, 2026

Copy link
Copy Markdown
Member

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessennickvergessennickvergessen approved these changes
@szaimenszaimenszaimen approved these changes
@miaulalalamiaulalalaAwaiting requested review from miaulalala
@nfebenfebeAwaiting requested review from nfebenfebe is a code owner automatically assigned from nextcloud/server-frontend
@sorbaughsorbaughAwaiting requested review from sorbaughsorbaugh is a code owner automatically assigned from nextcloud/server-frontend
@ArtificialOwlArtificialOwlAwaiting requested review from ArtificialOwlArtificialOwl is a code owner automatically assigned from nextcloud/server-backend
@leftybournesleftybournesAwaiting requested review from leftybournesleftybournes is a code owner automatically assigned from nextcloud/server-backend

Assignees

@nickvergessennickvergessen

Labels

3. to reviewWaiting for reviewsbugfeature: authentication

Projects

None yet

Milestone

Nextcloud 32.0.9

Development

Successfully merging this pull request may close these issues.

4 participants

@nickvergessen@joshtrichards@szaimen@nextcloud-command