Skip to content

fix(auth): log invalid token during password reset - #61408

Merged
AndyScherzinger merged 1 commit into
masterfrom
fix/auth/log-password-reset-invalid-token
Aug 6, 2026
Merged

fix(auth): log invalid token during password reset#61408
AndyScherzinger merged 1 commit into
masterfrom
fix/auth/log-password-reset-invalid-token

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

There are six possible cases of invalid token. The UI only handles expired token explicitly, the rest is a generic "invalid token". For debugging we should log the code and exception.

TODO

  • Do

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@ChristophWurst ChristophWurst added this to the Nextcloud 35 milestone Jun 18, 2026
@ChristophWurst ChristophWurst self-assigned this Jun 18, 2026
@ChristophWurst
ChristophWurst requested a review from a team as a code owner June 18, 2026 12:37
@ChristophWurst ChristophWurst added bug 3. to review Waiting for reviews labels Jun 18, 2026
@ChristophWurst
ChristophWurst requested review from ArtificialOwl, artonge, icewind1991 and leftybournes and removed request for a team June 18, 2026 12:37
@ChristophWurst

Copy link
Copy Markdown
Member Author

/backport to stable34

@ChristophWurst

Copy link
Copy Markdown
Member Author

/backport to stable33

@ChristophWurst

Copy link
Copy Markdown
Member Author

/backport to stable32

Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@AndyScherzinger
AndyScherzinger force-pushed the fix/auth/log-password-reset-invalid-token branch from dee23d2 to 4b92060 Compare August 2, 2026 11:49
@AndyScherzinger
AndyScherzinger merged commit 61b8fe3 into master Aug 6, 2026
198 of 200 checks passed
@AndyScherzinger
AndyScherzinger deleted the fix/auth/log-password-reset-invalid-token branch August 6, 2026 13:16
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants