Users are generated with invalid e-mails following the format user@foo

If you go in and try to edit the generated user during manual testing you have to type in a valid e-mail address to make any changes or delete the user account.

CommentFileSizeAuthor
#9 invalid_emails-1465772-9.patch615 bytesxjkwak
#6 1465772-6-valid_but_false_email_addresses.patch955 bytesxamanu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Status: Active » Closed (works as designed)

I suspect this is intended behavior, because we don't want your server to try to send any e-mails to devel_generated users.

The benefit of this safeguard looks greater than your inconvenience (you seem to be the first person to notice this). I agree that having to enter a valid e-mail address just to delete an account is a pain, but that's a feature of core...

mrf’s picture

mrf CreditAttribution: mrf commented

No need to use a working e-mail, just one that validates. See http://tools.ietf.org/rfc/rfc2606.txt

Seems strange that we would want to create accounts that behave exactly like real user accounts except in this one way. It is impossible to create an account through the UI without a valid email.

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Status: Closed (works as designed) » Active

Well, maybe you get a more favorable answer from moshe...

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Issue tags: +develcontribute

Tagging, let's try to give this a second though in the sprint

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Version: 7.x-1.2 » 8.x-1.x-dev

I suppose we could change user@foo to user@foo.invalid

xamanu’s picture

xamanu CreditAttribution: xamanu commented
Status: Active » Needs review
FileSize
1465772-6-valid_but_false_email_addresses.patch955 bytes

I agree and I like the proposal of using as tld .invalid. Here is a patch for that.

juampynr’s picture

juampynr CreditAttribution: juampynr commented
Status: Needs review » Fixed

Fixed. Thanks!

http://drupalcode.org/project/devel.git/commit/a98e8e9

juampynr’s picture

juampynr CreditAttribution: juampynr commented
Version: 8.x-1.x-dev » 7.x-1.x-dev
Status: Fixed » Needs work

This should be tested against Drupal 7.

xjkwak’s picture

xjkwak CreditAttribution: xjkwak commented
Status: Needs work » Needs review
FileSize
invalid_emails-1465772-9.patch615 bytes

A invalid extension has been added to the mail.

LGLC’s picture

LGLC CreditAttribution: LGLC commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +LONDON_2013_APRIL

Patch applied and tested, and all looks good.

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Reviewed & tested by the community » Fixed

Thanks all, committed!

Automatically closed -- issue fixed for 2 weeks with no activity.