Set Reply-To for data breach emails.

mysociety · Aug 9, 2023 · 4e86f74 · 4e86f74
1 parent 0d2fa6c
commit 4e86f74
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/lib/data_breach.rb b/lib/data_breach.rb
@@ -53,6 +53,7 @@ def data_breach(report, logged_in_user)
       from = MailHandler.address_from_name_and_email(
         'WhatDoTheyKnow.com data breach report', blackhole_email
       )
+      set_reply_to_headers(nil, 'Reply-To' => report.contact_email)
 
       # Set a header so we can filter in the mailbox
       headers['X-WDTK-Contact'] = 'wdtk-data-breach-report'

diff --git a/spec/integration/report_a_data_breach_spec.rb b/spec/integration/report_a_data_breach_spec.rb
@@ -37,6 +37,7 @@
     expect(last_email.from).to eq(['do-not-reply-to-this-address@localhost'])
     expect(last_email.to).to eq(['postmaster@localhost'])
     expect(last_email.subject).to eq('New data breach report')
+    expect(last_email.header["Reply-To"].value).to eq('test@example.com')
     expect(last_email.body).to include('URL: https://example.com')
     expect(last_email.body).to include('Special category or criminal offence data: Yes')
     expect(last_email.body).to include('DPO email: test@example.com')