"Something went wrong, perhaps this email is already registered, try the forgot password link"

AI-generated summary

Users are unable to register new accounts on the forum, encountering the error: ‘Something went wrong, perhaps this email is already registered, try the forgot password link.’ The issue stems from a recent regression in PR #24098, where performAccountCreation() incorrectly passes model.-prefixed attributes to User.createAccount(), causing the backend to receive empty email data and return a 400 error. Arkshine confirmed the bug and traced it to the JavaScript code. cvx acknowledged the issue and submitted a fix in PR #24169. n1bff confirmed the fix worked after upgrading. Arkshine credited Jarek for the fix and thanked them. No relevant logs were found, though unrelated Net::SMTPServerBusy errors appeared in logs.

Users cannot register new accounts in my forum.

After filling out the signup form and clicking “Create your account”, the modal dialog gets a red header bar with the message

Something went wrong, perhaps this email is already registered, try the forgot password link

This happens with any mail address. I have no blocked email domains.

The browser’s network log shows a request to https://example.com/u receiving a 400 error and the response {"errors":["param is missing or the value is empty: email"]}

https://example.com/logs/ shows nothing remotely related I think, maybe some Job exception: Net::SMTPServerBusy errors from 3 hours ago.

5 Likes

I can reproduce as well.

There is a regression in this PR from yesterday.

https://github.com/discourse/discourse/pull/24098


The issue is here:

In performAccountCreation(), attrs for name, email, and username contain properties name with model. and is passed to User.createAccount(attrs).

However, createAccount is not expecting that, resulting in empty data.

6 Likes

Sorry about that! Will be fixed shortly: FIX: Account creation regression by cvx · Pull Request #24169 · discourse/discourse · GitHub

8 Likes

Thanks, upgrading fixed it.

3 Likes

Just for reference, Jarek fixed it heren thanks! :+1:

https://github.com/discourse/discourse/pull/24169

1 Like

This topic was automatically closed after 23 hours. New replies are no longer allowed.