Include name of cf username causing emails to be sent in the headers
authorMagnus Hagander <magnus@hagander.net>
Tue, 15 Jul 2014 10:57:17 +0000 (12:57 +0200)
committerMagnus Hagander <magnus@hagander.net>
Tue, 15 Jul 2014 10:57:17 +0000 (12:57 +0200)
pgcommitfest/commitfest/views.py
pgcommitfest/mailqueue/util.py

index 0230c41e907b973981818172005f4591bf7d685c..b5d6af3ecead3d259e1efe7343414629d2bdce15 100644 (file)
@@ -279,6 +279,7 @@ def comment(request, cfid, patchid, what):
                        msg['From'] = "%s %s <%s>" % (request.user.first_name, request.user.last_name, request.user.email)
                        msg['Date'] = formatdate(localtime=True)
                        msg['User-Agent'] = 'pgcommitfest'
+                       msg['X-cfsender'] = request.user.username
                        msg['In-Reply-To'] = '<%s>' % form.respid
                        # We just add the "top" messageid and the one we're responding to.
                        # This along with in-reply-to should indicate clearly enough where
@@ -464,7 +465,7 @@ def send_email(request, cfid):
                        recipients = User.objects.filter(q).distinct()
 
                        for r in recipients:
-                               send_simple_mail(request.user.email, r.email, form.cleaned_data['subject'], form.cleaned_data['body'])
+                               send_simple_mail(request.user.email, r.email, form.cleaned_data['subject'], form.cleaned_data['body'], request.user.username)
                                messages.add_message(request, messages.INFO, "Sent email to %s" % r.email)
                        return HttpResponseRedirect('..')
        else:
index f38272cb342a093f275ee463889f7984db2f3669..60afd5dc4f694eb363cfdf9486740d29dbb2c1c0 100644 (file)
@@ -6,7 +6,7 @@ from email import encoders
 
 from models import QueuedMail
 
-def send_simple_mail(sender, receiver, subject, msgtxt, attachments=None):
+def send_simple_mail(sender, receiver, subject, msgtxt, sending_username, attachments=None):
        # attachment format, each is a tuple of (name, mimetype,contents)
        # content should already be base64 encoded
        msg = MIMEMultipart()
@@ -14,6 +14,8 @@ def send_simple_mail(sender, receiver, subject, msgtxt, attachments=None):
        msg['To'] = receiver
        msg['From'] = sender
        msg['Date'] = formatdate(localtime=True)
+       msg['User-Agent'] = 'pgcommitfest'
+       msg['X-cfsender'] = sending_username
 
        msg.attach(MIMEText(msgtxt, _charset='utf-8'))