Context Navigation

← Previous Ticket
Next Ticket →

#51874 closed task (blessed) (fixed)

Update PHPMailer library to 6.2.0

Reported by:	jrf	Owned by:	SergeyBiryukov
Priority:	normal	Milestone:	5.6
Component:	External Libraries	Version:	5.7
Severity:	normal	Keywords:	php8 has-patch commit dev-reviewed
Cc:		Focuses:

Description

PHPMailer 6.2.0 has just been released and includes a small fix for PHP 8 compatibility.

The version included with WP should be updated.

Full details:

Previously: #51502 (6.1.8), #51311 (6.1.7)

Attachments (1)

51874.patch (12.9 KB ) - added by ayeshrajans 6 years ago.: Tests: https://travis-ci.com/github/Ayesh/wordpress-develop/builds/204326555. Similar to the previous updates, we keep three hunks where we force-ignore phpcs checks. This patch updates to the the otherwise identical copy from upstream 6.2.0 tag.

Download all attachments as: .zip

Change History (9)

This ticket was mentioned in Slack in #core-php by hellofromtonya. View the logs.

6 years ago

@ayeshrajans
6 years ago

Attachment 51874.patch added

Tests: https://travis-ci.com/github/Ayesh/wordpress-develop/builds/204326555. Similar to the previous updates, we keep three hunks where we force-ignore phpcs checks. This patch updates to the the otherwise identical copy from upstream 6.2.0 tag.

#2 @jrf
6 years ago

Keywords has-patch commit added
Milestone Awaiting Review → 5.6

I just reviewed @ayesh's patch and looking at the diff, there is basically no risk, only benefit, so I propose we include this in the WP 5.6 release.

Effectively, there are only four functional changes, three which relate to PHP 8 fixes and one other which doesn't carry any risk either.
Everything else are CS changes.

#3 @jrf
6 years ago

Oh and what I nearly forgot to say: thanks @ayeshrajans for the quick turn around on getting a patch ready for this!

#4 @hellofromTonya
6 years ago

Keywords dev-feedback added

Adding the dev-feedback keyword so this ticket shows up in the "Commit Candidates which need Dev Review" section on Next Major Release, Workflow Oriented report.

#5 @SergeyBiryukov
6 years ago

Owner set to SergeyBiryukov
Status new → reviewing

#6 @SergeyBiryukov
6 years ago

Resolution → fixed
Status reviewing → closed

In 49713:

External Libraries: Upgrade PHPMailer to version 6.2.0.

For a full list of changes in this update, see the PHPMailer GitHub:
https://github.com/PHPMailer/PHPMailer/compare/v6.1.8...v6.2.0

Props ayeshrajans, jrf.
Fixes #51874.

#7 @SergeyBiryukov
6 years ago

Keywords dev-reviewed added; dev-feedback removed
Resolution fixed
Status closed → reopened

Reopening for backporting to the 5.6 branch.

Marking as dev-reviewed, as there is already another committer's review in comment:2.

#8 @SergeyBiryukov
6 years ago

Resolution → fixed
Status reopened → closed

In 49714:

External Libraries: Upgrade PHPMailer to version 6.2.0.

For a full list of changes in this update, see the PHPMailer GitHub:
https://github.com/PHPMailer/PHPMailer/compare/v6.1.8...v6.2.0

Props ayeshrajans, jrf.
Reviewed by jrf, SergeyBiryukov.
Merges [49713] to the 5.6 branch.
Fixes #51874.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#51874 closed task (blessed) (fixed)

Update PHPMailer library to 6.2.0

Description

Attachments (1)

Change History (9)

This ticket was mentioned in ​Slack in #core-php by hellofromtonya. ​View the logs. 6 years ago

@ayeshrajans 6 years ago

#2 @jrf 6 years ago

#3 @jrf 6 years ago

#4 @hellofromTonya 6 years ago

#5 @SergeyBiryukov 6 years ago

#6 @SergeyBiryukov 6 years ago

#7 @SergeyBiryukov 6 years ago

#8 @SergeyBiryukov 6 years ago