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

Verification Email not sent when user created with null email then subsequently changed #749

Closed
EddieWhi opened this issue Jul 17, 2020 · 1 comment
Labels
bug Something isn't working
Milestone

Comments

@EddieWhi
Copy link

Verification Email not sent when user created with null email then subsequently changed

Description

Verification emails are not sent to users when their email address is changed from null to a value even though the tenant is configured to send verification emails on email address change.

Affects versions

Hosted v 1.15.8

Steps to reproduce

Steps to reproduce the behavior:

  1. Configure tenant to verify user email and verify email on change.

  2. Create a user with no email:

 POST api/user
{
	"sendSetPasswordEmail": false,
	"user": {
		"password": "pass",
		"username": "uniqueName"
	}
}
  1. Update the users email:
PATCH api/user/{id}
{
	"user": {
		"email": "me@example.com"
	}
}

The response comes back showing the user as verified (and no email is received):

{
    "user": {
        "active": true,
        ...
        "verified": true
    }
}

If the users email is changed again then the subsequent PATCH triggers the email and sets verified to false.

Expected behavior

When a tenant is configured to verify email on change then the email should be sent whenever it changes, regardless of whether the email was initially set of not.

Screenshots

None

Platform

Hosted

Additional context

Originally reported on forum: https://fusionauth.io/community/forum/topic/195/no-email-verification-if-user-created-without-an-email

@mooreds mooreds added the bug Something isn't working label Jul 20, 2020
@robotdan
Copy link
Member

Thanks for the excellent recreate steps! I've recreated the issue, if the previous email was not defined we were not going through the verification process again.

@robotdan robotdan added this to Backlog in FusionAuth Issues via automation Jul 21, 2020
@robotdan robotdan added this to the 1.18.3 milestone Jul 21, 2020
@robotdan robotdan moved this from Backlog to In progress in FusionAuth Issues Jul 21, 2020
@robotdan robotdan moved this from In progress to Code complete in FusionAuth Issues Jul 21, 2020
@robotdan robotdan moved this from Code complete to Done in FusionAuth Issues Jul 24, 2020
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
FusionAuth Issues
  
Delivered
Development

No branches or pull requests

3 participants