Merge ~hloeung/mailman3-core-charm:master into mailman3-core-charm:master

Proposed by Haw Loeung
Status: Merged
Approved by: Paul Collins
Approved revision: 4947f9fc4a35786068a3a92f2e52e24d6e7cc176
Merged at revision: b9bf4daccd60779a3902686ba1a7376e1d1cb73d
Proposed branch: ~hloeung/mailman3-core-charm:master
Merge into: mailman3-core-charm:master
Diff against target: 17 lines (+6/-0)
1 file modified
reactive/mailman3_core.py (+6/-0)
Reviewer Review Type Date Requested Status
Paul Collins lgtm Approve
Review via email: mp+347952@code.launchpad.net

Commit message

Work around missing Message-ID header

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Paul Collins (pjdc) :
review: Approve (lgtm)
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision b9bf4daccd60779a3902686ba1a7376e1d1cb73d

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/reactive/mailman3_core.py b/reactive/mailman3_core.py
2index 3a99872..df82b1e 100644
3--- a/reactive/mailman3_core.py
4+++ b/reactive/mailman3_core.py
5@@ -88,6 +88,12 @@ def configure_postfix():
6 if os.path.exists('/var/lib/mailman3/data/postfix_domains'):
7 subprocess.check_output(['postconf', 'relay_domains=hash:/var/lib/mailman3/data/postfix_domains'])
8 subprocess.check_output(['postmap', '/var/lib/mailman3/data/postfix_domains'])
9+
10+ # HyperKitty expects messages will all have the 'Message-ID' header,
11+ # unfortunately not all clients do so let's work around it (LP: #1776658)
12+ subprocess.check_output(['postconf', 'always_add_missing_headers=yes',
13+ 'local_header_rewrite_clients=static:all'])
14+
15 host.service_reload('postfix')
16 # Open the SMTP port
17 hookenv.open_port(25)

Subscribers

People subscribed via source and target branches