Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: User changing their email address sends an errant application notification to admins #5272

Open
5 tasks done
asimons04 opened this issue Dec 19, 2024 · 0 comments · May be fixed by #5288
Open
5 tasks done

[Bug]: User changing their email address sends an errant application notification to admins #5272

asimons04 opened this issue Dec 19, 2024 · 0 comments · May be fixed by #5288
Labels
bug Something isn't working

Comments

@asimons04
Copy link
Contributor

asimons04 commented Dec 19, 2024

Requirements

  • Is this a bug report? For questions or discussions use https://lemmy.ml/c/lemmy_support
  • Did you check to see if this issue already exists?
  • Is this only a single bug? Do not put multiple bugs in one issue.
  • Do you agree to follow the rules in our Code of Conduct?
  • Is this a backend issue? Use the lemmy-ui repo for UI / frontend issues.

Summary

On instances with registration applications enabled, when a user changes their email address, an inappropriate new registration application email is sent to admins after the user has clicked the verification link.

Steps to Reproduce

Prerequisites:

  • Instance must be configured to require an application
  • Admin setting "Email admins on receiving new applications" must be enabled
  • Setting to require email verification has no effect on the results (which is fine; changing email should require the new one to be verified before accepting it)
  1. Have any user update their email address
  2. User will receive a confirmation link to verify the new address
  3. User clicks link and successfully verifies the new email address
  4. Admins receive a non-applicable "{user who changed their email} has applied to join {instance}" notification email.

Technical Details

If that same function is called for both initial registration and email change, I would guess the issue is in verify_emai.rs as there doesn't seem to be any check besides whether admins should receive new registration notifications.

Version

BE 0.19.3

Lemmy Instance URL

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant