[wp-trac] [WordPress Trac] #17433: localhost is not accepted as email domain

[WordPress Trac]

#17433: localhost is not accepted as email domain
----------------------------------------+-----------------------------
 Reporter:  sanvila                     |       Owner:
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  Future Release
Component:  Formatting                  |     Version:  3.2
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+-----------------------------
Description changed by SergeyBiryukov:

Old description:

> Hi. Tried to install WordPress on a Debian machine not connected to
> Internet, only for testing purposes, so when the setup procedure asked me
> for an email address, I used mylogin at localhost. The setup procedure,
> however, rejected this as "invalid".
>
> I think the bug is exactly in wp-includes/formatting.php, where it says:
>
>         // Assume the domain will have at least two subs
>         if ( 2 > count( $subs ) ) {
>                 return apply_filters( 'is_email', false, $email,
> 'domain_no_periods' );
>         }
>
> So: Could you please special-case "localhost" in is_email() so that it's
> allowed as email domain?
>
> I guess the probability of someone using @localhost for email "by
> mistake" is extremely low, so this change will unlikely harm the average
> user.
>
> Thanks.

New description:

 Hi. Tried to install WordPress on a Debian machine not connected to
 Internet, only for testing purposes, so when the setup procedure asked me
 for an email address, I used mylogin at localhost. The setup procedure,
 however, rejected this as "invalid".

 I think the bug is exactly in wp-includes/formatting.php, where it says:
 {{{
         // Assume the domain will have at least two subs
         if ( 2 > count( $subs ) ) {
                 return apply_filters( 'is_email', false, $email,
 'domain_no_periods' );
         }
 }}}
 So: Could you please special-case "localhost" in is_email() so that it's
 allowed as email domain?

 I guess the probability of someone using @localhost for email "by mistake"
 is extremely low, so this change will unlikely harm the average user.

 Thanks.

--

--
Ticket URL: <https://core.trac.wordpress.org/ticket/17433#comment:10>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform