diff options
| author | Magnus Hagander | 2018-01-20 13:19:35 +0000 |
|---|---|---|
| committer | Magnus Hagander | 2018-01-20 13:19:35 +0000 |
| commit | 5f292219969839d5b08c3f775ff3810ea63de705 (patch) | |
| tree | 34c2c88c79a920308794a7f19646310bea879a73 | |
| parent | bb7d9b5d83dfd3258a3d1bd573b20f8775688436 (diff) | |
Actually deliver CCed emails to patch authors
Previously we would set the Cc header of the email, but not actually
send a copy to the user. That way if the user was not subscribed to the
hackers email, they'd end up not receiving the email at all, which
somewhat defeats one of the original points of adding the Cc in the
first place.
Spotted by Peter Geoghegan
| -rw-r--r-- | pgcommitfest/commitfest/views.py | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/pgcommitfest/commitfest/views.py b/pgcommitfest/commitfest/views.py index c3007b2..d809340 100644 --- a/pgcommitfest/commitfest/views.py +++ b/pgcommitfest/commitfest/views.py @@ -382,7 +382,13 @@ def comment(request, cfid, patchid, what): msg['References'] = '<%s> <%s>' % (form.thread.messageid, form.respid) msg['Message-ID'] = make_msgid('pgcf') - send_mail(UserWrapper(request.user).email, settings.HACKERS_EMAIL, msg.as_string()) + uw = UserWrapper(request.user) + msgstring = msg.as_string() + send_mail(uw.email, settings.HACKERS_EMAIL, msgstring) + for a in authors: + # Actually send a copy directly to the author. Just setting the Cc field doesn't + # make it deliver the email... + send_mail(uw.email, UserWrapper(a).email, msgstring) PatchHistory(patch=patch, by=request.user, what='Posted %s with messageid %s' % (what, msg['Message-ID'])).save() |