Skip to content

Allow for using "Forgot Password" even when a password isn't set#4768

Open
wes-otf wants to merge 2 commits intomainfrom
fix/forgot-password-not-working
Open

Allow for using "Forgot Password" even when a password isn't set#4768
wes-otf wants to merge 2 commits intomainfrom
fix/forgot-password-not-working

Conversation

@wes-otf
Copy link
Contributor

@wes-otf wes-otf commented Mar 20, 2026

If a user doesn't have a password set in Hypha and they select "Forgot Password", they will get a prompt notifying them that a password reset email has been sent while Django quietly ignores the sending of the email. This overrides the default get_users method in Django's PasswordResetForm to include users without a usable password.

As OTF has seen more outlook users have issues with magic links, this would be a nice alternative (even though maybe an annoying extra step) for the time being until we can properly block the link previews.

This PR also includes some very minor UI tweaks I noticed in testing that make reset prompts similar to the rest of Hypha's headers, help texts & buttons.

Test Steps

  • Ensure that a user without a password can select the "Forgot Password" option during passworded logins to set a password on their account

@wes-otf wes-otf requested a review from frjo March 20, 2026 17:07
@wes-otf wes-otf added Type: Bug Bugs! Things that are broken :-/ Type: Minor Minor change, used in release drafter labels Mar 20, 2026
@frjo
Copy link
Member

frjo commented Mar 20, 2026

@wes-otf Did it not work to block the MS preview stuff with Cloudflare?

In the Cloudflare WAF it should be possible to add a rule that block user agents that contain "MicrosoftPreview".

@frjo
Copy link
Member

frjo commented Mar 20, 2026

Overrides like this is no fun to handle when updating Django in the future.

@frjo
Copy link
Member

frjo commented Mar 20, 2026

If we need to fix this in Hypha I think a solution that adds an extra step to capture MicrosoftPreview is better.

Clicking the onetime link would then go to a form with a single "Continue" button.

It would force all users to do one extra click but it would also capture MicrosoftPreview and any other similar system.

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

Labels

Type: Bug Bugs! Things that are broken :-/ Type: Minor Minor change, used in release drafter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants