[wp-trac] [WordPress Trac] #15706: Allow wildcarded domains in multisite limited email domains

WordPress Trac noreply at wordpress.org
Fri Sep 27 17:23:18 UTC 2013


#15706: Allow wildcarded domains in multisite limited email domains
------------------------------+-----------------------------
 Reporter:  djcp              |       Owner:  westi
     Type:  enhancement       |      Status:  reviewing
 Priority:  normal            |   Milestone:  Future Release
Component:  Multisite         |     Version:
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+-----------------------------
Changes (by nacin):

 * milestone:  3.7 => Future Release


Comment:

 I started to break this down by removing is_email_address_allowed() all
 together, and using is_domain_in_list() directly in the validate signup
 function. I then realized that wildcards were actually useless here,
 because the logic that handles banned domains (and is now in
 is_domain_in_list()) already blocks subdomains when a higher level domain
 is blocked. Limited email domains, however, do not currently have this
 logic. So this is a code problem *and* a UX problem. There is just too
 much weird stuff going on here, need to move this out of 3.7.

 http://irclogs.wordpress.org/chanlog.php?channel=wordpress-
 dev&day=2013-09-27&sort=asc#m697332

--
Ticket URL: <http://core.trac.wordpress.org/ticket/15706#comment:38>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list