[wp-trac] [WordPress Trac] #16805: default 'from' hostname
WordPress Trac
wp-trac at lists.automattic.com
Wed Mar 9 15:11:39 UTC 2011
#16805: default 'from' hostname
--------------------------+-----------------------------
Reporter: lonnylot | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Mail | Version: 3.1
Severity: normal | Keywords:
--------------------------+-----------------------------
The logic used to create the hostname portion of default e-mail address
when calling wp_mail doesn't make sense.
Lines 384-392 of wp-includes/pluggable.php read:
{{{
if ( !isset( $from_email ) ) {
// Get the site domain and get rid of www.
$sitename = strtolower( $_SERVER['SERVER_NAME'] );
if ( substr( $sitename, 0, 4 ) == 'www.' ) {
$sitename = substr( $sitename, 4 );
}
$from_email = 'wordpress@' . $sitename;
}
}}}
We assume that if www. is in the sitename that it is not part of the
e-mail address. I reviewed a few RFCs and have not found where 'www.' is
prohibited from the hostname portion of an e-mail address. I concede that
john.smith@ www.sitename.com is now common, but it is still a legal name.
Additionally, if the $_SERVER[ 'SERVER_NAME' ] is a sub-domain such as
blogs.sitename.com the e-mail is going to send from john.smith@
blogs.sitename.com.
For consistency I suggest we either remove all nth level domain names
above the first or remove none. This way it will be more obvious when
people need to use the wp_mail_from filter and not just appear to be
random.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16805>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list