[wp-trac] [WordPress Trac] #49277: Implement email sanitize in REST API

WordPress Trac noreply at wordpress.org
Thu Jan 23 16:18:24 UTC 2020


#49277: Implement email sanitize in REST API
-------------------------------------------------+-------------------------
 Reporter:  spacedmonkey                         |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  REST API                             |     Version:  4.4
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch 2nd-opinion has-unit-      |     Focuses:
  tests                                          |
-------------------------------------------------+-------------------------

Comment (by TimothyBlynJacobs):

 Does that line actually get executed? It looks like it prefers regex which
 I think needs to be available to run WP?

 Related #28170, #16867

 > The idea is to just remove character that are not valid in an email.

 Right, what I'm trying to figure out, is if there is a case where the
 sanitization would not allow an email that would have previously been
 allowed by `is_email()`. I understand `filter_var` doesn't do the complex
 checking `is_email` does. But I'm wondering if any of the characters it
 strips would've been allowed by `is_email`.

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


More information about the wp-trac mailing list