Skip to content

Let console admins reset a forgotten password - #97

Merged
om986 merged 1 commit into
mainfrom
forgot-password
Oct 14, 2026
Merged

om986 merged 1 commit into
mainfrom
forgot-password

Conversation

@om986

@om986 om986 commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The sign-in page can email a one-hour, single-use password reset link when SMTP is configured. The response does not reveal whether the address is an admin.
  • Using the link sets a new password and ends that admin's browser sessions. Two-factor authentication and personal API tokens stay in place.
  • Without SMTP, the link stays hidden. om admin reset-password still sets a password from the machine. Run om migrate for the new reset-token table.

Test plan

  • With SMTP and OM_PUBLIC_URL set, request a reset for a password admin and confirm the email link works once.
  • Request a reset for an unknown address and confirm the page shows the same message and no email is sent.
  • Confirm a reset ends existing browser sessions, rejects the old password, and still requires an authenticator code when two-factor authentication is on.
  • Confirm the forgot-password link is absent when SMTP is not configured.
  • Run om migrate on an existing database and sign in afterward.

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: OpenSourceOM/core/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 3bb69e7e-5abe-4da4-a20d-8c97f9dc9119
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@om986
om986 merged commit 4a9770c into main Oct 14, 2026
7 checks passed
@om986
om986 deleted the forgot-password branch October 14, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant