From a754fe0783ed4d9c327120e01d0aa6e30fde92f8 Mon Sep 17 00:00:00 2001 From: Serghei Mihai Date: Thu, 26 Oct 2017 15:08:08 +0200 Subject: [PATCH] add unsubscription link in mail headers (#19579) --- corbo/utils.py | 1 + tests/test_emailing.py | 1 + 2 files changed, 2 insertions(+) diff --git a/corbo/utils.py b/corbo/utils.py index 3023fa2..d3c3360 100644 --- a/corbo/utils.py +++ b/corbo/utils.py @@ -60,6 +60,7 @@ def send_email(title, content, destinations, category_id): 'identifier': dest}) unsubscribe_link = urlparse.urljoin(settings.SITE_BASE_URL, reverse( 'unsubscribe', kwargs={'unsubscription_token': unsubscribe_token})) + message.set_headers({'List-Unsubscribe': '<%s>' % unsubscribe_link}) message.html = orig_html.replace(UNSUBSCRIBE_LINK_PLACEHOLDER, unsubscribe_link) message.text = orig_text.replace(UNSUBSCRIBE_LINK_PLACEHOLDER, unsubscribe_link) diff --git a/tests/test_emailing.py b/tests/test_emailing.py index 399bfa9..d9fa17a 100644 --- a/tests/test_emailing.py +++ b/tests/test_emailing.py @@ -135,6 +135,7 @@ def test_unsubscription_link(app, categories, announces, custom_mailoutbox): index = i*subscriptions_number+counter signature = urllib.unquote(re.findall('/unsubscribe/(.*)"', mail.outbox[index].html)[0]) unsubscription_link = reverse('unsubscribe', kwargs={'unsubscription_token': signature}) + assert mail.outbox[index]._headers['List-Unsubscribe'] == '' % unsubscription_link assert unsubscription_link in mail.outbox[index].html assert unsubscription_link in mail.outbox[index].text assert unsubscription_link_sentinel != unsubscription_link -- 2.15.0.rc2