[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