[wp-trac] [WordPress Trac] #38506: Email length can be less than 6 character

WordPress Trac noreply at wordpress.org
Mon Nov 7 04:59:43 UTC 2016


#38506: Email length can be less than 6 character
--------------------------+-----------------------
 Reporter:  mangeshp      |       Owner:  dd32
     Type:  defect (bug)  |      Status:  assigned
 Priority:  normal        |   Milestone:  4.7
Component:  REST API      |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:
--------------------------+-----------------------

Comment (by dd32):

 Sorry, I missed these pings previously.

 I'm all for standardising on `is_email()`. I personally don't see any
 (practical) point in treating an email < 6 char as valid, so it makes
 sense to me for us to add that to the `is_email()` checks.

 I understand the other side of it though, it could potentially be a valid
 email address, we're just refusing to honor that as an valid email address
 for comments - which is why it shouldn't be in `is_email()` but should be
 within the comment handlers.
 At the end of the day though, discussion for keeping it, discarding it,
 adding it to `is_email()` is not worth our time, as it's all theoretical,
 and whomever wants to patch should decide what the most logical option is.

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


More information about the wp-trac mailing list