[wp-trac] [WordPress Trac] #50720: PHPMailer's validator defaults to 'php' after the upgrade

WordPress Trac noreply at wordpress.org
Wed Jul 22 17:47:30 UTC 2020


#50720: PHPMailer's validator defaults to 'php' after the upgrade
--------------------------+------------------------------
 Reporter:  david.binda   |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Mail          |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------------------

Comment (by david.binda):

 Hey @ayeshrajans ! Thanks for the info and questions.

 I've run into the issue described in this ticket with `ace at 204.32.222.14`
 email address which is used in the formatting/IsEmail.php tests.

 The email address is valid accordingly to the WordPress' `is_email()`
 function, and passes the PHPMailer's validation in `pcre8` mode, but fails
 in `php` mode. (please see the 50720.2.diff file with an example)

 > I highly doubt that WordPress can even work without PCRE, so most of
 these conditions will end up being dead code.
 > With WordPress's current server requirements, the only theoretical
 option will be pcre8.

 That's kinda what I was trying to say. Odds that pcre8 was available and
 thus used before the upgrade. That's why I thought it might make sense to
 preserve that setting, if it's available on the server.

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


More information about the wp-trac mailing list