[tor-commits] [bridgedb/master] Simplify handling of incoming emails without a Subject: header.

isis at torproject.org isis at torproject.org
Fri May 16 18:52:52 UTC 2014


commit 712bacce51622f9fbfb67fd267bad6fd09d1c412
Author: Isis Lovecruft <isis at torproject.org>
Date:   Mon May 5 20:55:43 2014 +0000

    Simplify handling of incoming emails without a Subject: header.
---
 lib/bridgedb/email/server.py |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/lib/bridgedb/email/server.py b/lib/bridgedb/email/server.py
index 97ddcde..838093e 100644
--- a/lib/bridgedb/email/server.py
+++ b/lib/bridgedb/email/server.py
@@ -143,7 +143,7 @@ def createResponseBody(lines, context, toAddress, lang='en'):
                 request=toAddress) for b in bridges)
         return templates.buildMessage(t) % answer
 
-def generateResponse(fromAddress, clientAddress, subject, body,
+def generateResponse(fromAddress, clientAddress, body, subject=None,
                      messageID=None, gpgContext=None):
     """Create a :class:`MailResponse`, which acts like an in-memory
     ``io.StringIO`` file, by creating and writing all headers and the email
@@ -619,7 +619,7 @@ class MailMessage(object):
 
         clientAddr = '@'.join([client.local, client.domain])
         messageID = incoming.getheader("Message-ID", None)
-        subject = incoming.getheader("Subject", None) or "[no subject]"
+        subject = incoming.getheader("Subject", None)
 
         # Look up the locale part in the 'To:' address, if there is one and
         # get the appropriate Translation object:
@@ -629,8 +629,8 @@ class MailMessage(object):
         body = createResponseBody(self.lines, self.context, clientAddr, lang)
         if not body: return d  # The client was already warned.
 
-        response = generateResponse(self.context.fromAddr, clientAddr, subject,
-                                    body, messageID, self.context.gpgContext)
+        response = generateResponse(self.context.fromAddr, clientAddr, body,
+                                    subject, messageID, self.context.gpgContext)
         if not response: return d
 
         logging.info("Sending reply to %s" % client)





More information about the tor-commits mailing list