[wp-trac] [WordPress Trac] #44501: Change "From" email address for GDPR request confirmation email
WordPress Trac
noreply at wordpress.org
Tue Jan 15 07:56:56 UTC 2019
#44501: Change "From" email address for GDPR request confirmation email
-------------------------+------------------------------
Reporter: zaffarn | Owner: garrett-eclipse
Type: enhancement | Status: accepted
Priority: normal | Milestone: Future Release
Component: Privacy | Version: 4.9.6
Severity: normal | Resolution:
Keywords: needs-patch | Focuses:
-------------------------+------------------------------
Changes (by garrett-eclipse):
* keywords: reporter-feedback => needs-patch
* owner: (none) => garrett-eclipse
* status: new => accepted
* milestone: Awaiting Review => Future Release
Comment:
Hi @zaffarn thank you for the feedback. I definitely see the validity of
having the `from_email` filterable for Privacy requests.
I'm sorry this ticket was raised in our team chat and I missed updating it
afterwards.
It was agreed to look into a filter for the reply-to and from email
addresses as we felt filtering them specifically would be very helpful for
organizations with a DPO as you mentioned.
After reviewing some core implementations it appears the best way to
accomplish this would be to introduce the email header as filterable and
pass it along to wp_mail. As currently using the`wp_mail_from` and
`wp_mail_from_name` would affect all email as it's unaware of the email
being for a privacy request.
Two coding examples of this; [[BR]]
Pass Change Email -
https://github.com/WordPress/WordPress/blob/e0c4e5b610bd0453b649f02c514f6eadc059e5be
/wp-includes/user.php#L1974-L2011 [[BR]]
Email Change Email -
https://github.com/WordPress/WordPress/blob/6073c6e209f3877c2d4291f76e3cb3db1c874a99
/wp-includes/functions.php#L6342-L6388
I'll look into this further when I have time.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/44501#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list