[wp-trac] [WordPress Trac] #19614: get_avatar should ignore whitespace in email addresses

WordPress Trac wp-trac at lists.automattic.com
Mon Dec 19 19:34:11 UTC 2011


#19614: get_avatar should ignore whitespace in email addresses
-------------------------+------------------------------
 Reporter:  evansolomon  |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  General      |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------------------

Comment (by johnbillion):

 Seems to me like this is the wrong place to be testing for whitespace in
 an email address. If the email address needs to be valid (in your case
 they do as you're using them to display Gravatars) then you should be
 validating the email addresses when they're entered. What if an email
 address contains other disallowed characters? Why are you only stripping
 whitespace?

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


More information about the wp-trac mailing list