WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#7474 closed enhancement (fixed)

Update phpMailer

Reported by: mattyrob Owned by: westi
Milestone: 2.7 Priority: high
Severity: normal Version: 2.6
Component: General Keywords: has-patch tested 2nd-opinion
Focuses: Cc:

Description

Can we update to the most recent stable phpMailer so we can use secure email connections if needed with the SMTPSecure method.

Attachments (1)

php-mailer-patch.diff (198.7 KB) - added by mattyrob 7 years ago.

Download all attachments as: .zip

Change History (8)

@mattyrob7 years ago

comment:1 @ryan7 years ago

Wow, someone is maintaining it again. Sure, we can give it a try. Looks like some extra stuff slipped into the patch though.

comment:2 @westi7 years ago

  • Owner changed from anonymous to westi
  • Status changed from new to assigned

There does seem to be a fair amount of extra stuff slipped in there.

Need to give those changes a good review - don't want to break something which works quite well at the moment.

If you want Ryan I'll take this and give is a good look see before committing.

comment:3 @ryan7 years ago

Have at it.

comment:4 @MattyRob7 years ago

Basically, I grabbed the 3 files for PHP4 and copied them over the existing phpmailer files in the includes folder. I've done this on my main site and it appears to work for me - has done for 48hours now without any missing emails - I'm still getting registrations and comment notifications and emails via Subscribe2 are working fine too.

Just thought you should know :-)

comment:5 @robertaccettura7 years ago

+1 for updating this. Getting TLS support would be a good enhancement. POP-before-SMTP is also worthwhile as it's another popular security scheme.

comment:6 @ryan7 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [8762]) Update phpMailer to 2.0.2. Props mattyrob. fixes #7474

comment:7 @ryan7 years ago

Verified that it included old fixes, especially the security one. Did some light testing.

Note: See TracTickets for help on using tickets.