Merge pull request arithmetric#33 from carlosgonser/feature/subject_prefix

added subjectPrefix option

Author: arithmetric

index.js

@@ -9,6 +9,7 @@ console.log("AWS Lambda SES Forwarder // @arithmetric // Version 3.0.0");
 //
 // Expected keys/values:
 // - fromEmail: Forwarded emails will come from this verified address
+// - subjectPrefix: Forwarded emails subject will contain this prefix
 // - emailBucket: S3 bucket name where SES stores emails.
 // - emailKeyPrefix: S3 key name prefix where SES stores email. Include the
 //   trailing slash.
@@ -19,6 +20,7 @@ console.log("AWS Lambda SES Forwarder // @arithmetric // Version 3.0.0");
 //   The key must be lowercase.
 var defaultConfig = {
   fromEmail: "noreply@example.com",
+  subjectPrefix: "",
   emailBucket: "s3-bucket-name",
   emailKeyPrefix: "emailsPrefix/",
   forwardMapping: {
@@ -186,6 +188,15 @@ exports.processMessage = function(data, next) {
       return fromText;
     });
 
+  // Add a prefix to the Subject
+  if (data.config.subjectPrefix) {
+    header = header.replace(
+      /^Subject: (.*)/mg,
+      function(match, subject) {
+        return 'Subject: ' + data.config.subjectPrefix + subject;
+      });
+  }
+
   // Remove the Return-Path header.
   header = header.replace(/^Return-Path: (.*)\r?\n/mg, '');
 

test/assets/message.subjectprefix.txt

@@ -0,0 +1,25 @@
+Received: from example.com (example.com [127.0.0.1])
+ by inbound-smtp.us-west-2.amazonaws.com with SMTP id 81fu1unjk93bm5cb0jlk23fll33spcvf3633l8qg1
+ for info@example.com;
+ Fri, 11 Mar 2016 06:20:55 +0000 (UTC)
+X-SES-Spam-Verdict: PASS
+X-SES-Virus-Verdict: PASS
+Received-SPF: none (spfCheck: 127.0.0.1 is neither permitted nor denied by domain of example.com) client-ip=10.0.0.1; envelope-from=postmaster@example.com; helo=example.com;
+From: Betsy at betsy@example.com <info@example.com>
+To: info@example.com
+Subject: [PREFIX] Test message from Betsy
+Message-Id: <B9ebWRD-000123-3K@example.com>
+Date: Fri, 11 Mar 2016 01:20:54 -0500
+Reply-To: Betsy <betsy@example.com>
+
+This is a test message to info@example.com.
+
+It was sent from betsy@example.com.
+
+These lines should not be affected:
+From: test@example.com
+Reply-To: test@example.com
+Return-Path: test@example.com
+DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/simple;
+	s=gdwg2y3kokkkomn55z2ilkup5wp5hhxx; d=amazonses.com; t=1457977483;
+	h=Date:From:Reply-To:To:Message-ID:Subject:MIME-Version;

test/processMessage.js

@@ -100,5 +100,29 @@ describe('index.js', function() {
         done();
       });
     });
+
+    it('should allow adding a prefix to the Subject in emails', function(done) {
+      var data = {
+        config: {
+          subjectPrefix: "[PREFIX] "
+        },
+        email: {
+          source: "betsy@example.com"
+        },
+        emailData: fs.readFileSync("test/assets/message.txt").toString(),
+        log: console.log,
+        recipients: ["jim@example.com"],
+        originalRecipient: "info@example.com"
+      };
+      var emailDataProcessed = fs.readFileSync(
+        "test/assets/message.subjectprefix.txt").toString();
+      index.processMessage(data, function(err, data) {
+        assert.ok(!err, "processEmail returned successfully");
+        assert.equal(data.emailData,
+          emailDataProcessed,
+          "processEmail updated email data");
+        done();
+      });
+    });
   });
 });