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

[WordPress Trac]

#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 ayeshrajans):

 Are there any specific examples that were allowed in PHPMailer 5's `auto`,
 but rejected in PHPMailer 6 `php` validation?

 The `auto` mode appears to be a more forgiving work-around to deal with
 PHP and PCRE version differences. I highly doubt that WordPress can even
 work without PCRE, so most of these conditions will end up being dead
 code.

 WordPress now requires PHP 5.6, so with this patch, the only possible
 options automatically selected would be `pcre8` and `php`.

 I personally think `filter_var` (`php` validation) is the more accurate
 and cleaner approach. I would personally prefer to accept PHPMailer's
 defaults instead of fighting against it.

 Tagging @Synchro of PHPMailer who might provide more insight to this.

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