email-bounce-parser 1.0.17 → 1.0.19

package/README.md

@@ -53,7 +53,7 @@ console.log(result);
 //   email: {
 //     body: "This is the mail system at host mailer.acme.email [...] (in reply to RCPT TO command)",
 //     intro: "This is the mail system at host mailer.acme.email [...] The mail system",
-//     error: "<support@yipee.com>: host smtp.secureserver.net[92.240.81.0] said: [...] (in reply to RCPT TO command)"
+//     error: "<john@yipee.com>: host smtp.secureserver.net[92.240.81.0] said: [...] (in reply to RCPT TO command)"
 //   },
 //
 //   data: {
@@ -62,7 +62,7 @@ console.log(result);
 //         basic: "550",
 //         enhanced: "5.1.1"
 //       },
-//       label: "<support@yipee.com> Recipient not found.",
+//       label: "<john@yipee.com> Recipient not found.",
 //
 //       type: "action_not_taken",
 //
@@ -76,6 +76,8 @@ console.log(result);
 //       }
 //     },
 //
+//     recipient: "john@yipee.com",
+//
 //     server: {
 //       hostname: "smtp.secureserver.net",
 //       ip: "92.240.81.0",

package/lib/parser.js

@@ -82,16 +82,16 @@ var REGEXES = {
   ],
 
   error : [
-    /said\: (\d{3}(?:[\s\-]\d\.\d\.\d\d?)?) (.+) \(in reply to/,
-    /said\: (\d{3}(?:[\s\-]\d\.\d\.\d\d?)?)-(.+)/,
-    /(\d{3}\-\'\d\.\d\.\d\d?) (.+)/,
-    /LMTP error after .+?\:\<.+?\>\: (\d{3}(?:[\s\-]\d\.\d\.\d\d?)?) <.+?\>(.+)$/m,
-    /remote server returned \'(\d{3}(?:[\s\-]\d\.\d\.\d\d?)?) (.+)\'/i,
-    /The response from the remote server was:\s(\d{3}(?:[\s\-]\d\.\d\.\d\d?)?) (.+)/,
-    /refused to talk to me\: (\d{3}(?:[\s\-]\d\.\d\.\d\d?)?) (.+)/,
-    /(?:error|reason)\:\s*?(\d{3}(?:[\s\-]\d\.\d\.\d\d?)?) (.+)/i,
-    /(\d{3}[\s\-]\d\.\d\.\d\d?)/,
-    /(\d{3}\-\'\d\.\d\.\d\d?)/
+    /said\: (\d{3}(?:[\s\-]\d\.\d\.\d{1,3})?) (.+) \(in reply to/,
+    /said\: (\d{3}(?:[\s\-]\d\.\d\.\d{1,3})?)-(.+)/,
+    /(\d{3}\-\'\d\.\d\.\d{1,3}) (.+)/,
+    /LMTP error after .+?\:\<.+?\>\: (\d{3}(?:[\s\-]\d\.\d\.\d{1,3})?) <.+?\>(.+)$/m,
+    /remote server returned \'(\d{3}(?:[\s\-]\d\.\d\.\d{1,3})?) (.+)\'/i,
+    /The response from the remote server was:\s(\d{3}(?:[\s\-]\d\.\d\.\d{1,3})?) (.+)/,
+    /refused to talk to me\: (\d{3}(?:[\s\-]\d\.\d\.\d{1,3})?) (.+)/,
+    /(?:error|reason)\:\s*?(\d{3}(?:[\s\-]\d\.\d\.\d{1,3})?) (.+)/i,
+    /(\d{3}[\s\-]\d\.\d\.\d{1,3})/,
+    /(\d{3}\-\'\d\.\d\.\d{1,3})/
   ],
 
   error_lax : [
@@ -110,12 +110,13 @@ var REGEXES = {
 
   recipient : [
     /the following addresses had permanent fatal errors (?:.*?)\s\<(.+?)\>/i,
+    /there was a temporary problem delivering your message to (.+?). Gmail will retry/i,
     /the following message to \<(.+?)\> was undeliverable/i,
     /to the following addresses:\s*(.+)/i,
-    /^\<(.+?)\>:/m,
-    /^(.+?)\s?\n?\s*[\[|<]mailto\:(.+?)[\]|>]/mi, // "john@acme.com<mailto:john@acme.com>"
     /^your message wasn't delivered to (.+?)\s/mi, // "Your message wasn't delivered to john@acme.com"
     /^your message to (.+?) couldn\'t/mi, // "Your message to john@acme.com couldn't"
+    /^(.+?)\s?(?:\s\(.*?\))?\n?\s*[\[|<]mailto\:(.+?)[\]|>]/mi, // "john@acme.com<mailto:john@acme.com>" or "john@acme.com[mailto:john@acme.com]" or "john@acme.com (john@acme.com)<mailto:john@acme.com>"
+    /^\<(.+?)\>:/m,
     /\<(.+?)\> \(expanded from \<.+?\>\)\:/, // "<john@acme.com> (expanded from <john@digiflex.com>): "
     /\<(.+?)\>:/ // "<john@acme.com>:"
   ],
@@ -135,7 +136,7 @@ var REGEXES = {
 
   enhanced_types_server_unknown : [
     /(does not accept mail)/i,
-    /(domain name not found)/i,
+    /(domain (?:name\s+)?(?:\S+\s)?(?:not found|does not exist))/i, // "domain does not exist" or "domain name does not exist" or "domain acme.com does not exist" or "domain name acme.com does not exist"
     /(no route to host)/i
   ],
 
@@ -147,9 +148,13 @@ var REGEXES = {
     /(out of storage|(?:mailbox (?:is )?full)|(?:user (?:is )?over quota))/i // "user is over quota" (iCloud)
   ],
 
+  enhanced_types_reputation : [
+    /(rate limited due to IP reputation)/i
+  ],
+
   error_code : [
-    /(\d{3})[\s\-](\d\.\d\.\d\d?)/,  // "550 5.4.1" or "550-5.4.1" or "550-5.4.11"
-    /(\d{3})\-\'(\d\.\d\.\d\d?)/, // "550-'5.4.1" or "550-'5.4.11"
+    /(\d{3})[\s\-](\d\.\d\.\d{1,3})/,  // "550 5.4.1" or "550-5.4.1" or "550-5.4.11" or "550-5.4.110"
+    /(\d{3})\-\'(\d\.\d\.\d{1,3})/, // "550-'5.4.1" or "550-'5.4.11" or "550-'5.4.110"
     /(\d{3})/ // "550"
   ]
 };
@@ -299,21 +304,22 @@ class Parser {
         _permanent = true;
       }
 
-      // Make sure the bounce is actually permanent
-      if (_permanent === true) {
-        var _match = this.__regexes.temporary_safe.match(str);
-
-        if (_match && _match.length > 1) {
-          _temporary = true;
-          _permanent = false;
-        }
-      }
-
       if (_code.basic) {
         _type = ERROR_CODES_TYPES[_code.basic] || null
       }
     }
 
+    // In cases where the bounce is detected as permanent, or the \
+    //   temporary / permanent status is still unclear, run additional detection
+    if (_permanent === true || (_temporary === false && _permanent === false)) {
+      var _match = this.__regexes.temporary_safe.match(str);
+
+      if (_match && _match.length > 1) {
+        _temporary = true;
+        _permanent = false;
+      }
+    }
+
     // No label detected? Probably because there's no error code
     if (!_label) {
       // Attempt a lax check
@@ -409,6 +415,12 @@ class Parser {
       _type = "mailbox_full";
     }
 
+    _match = loopRegexes(this.__regexes.enhanced_types_reputation, str);
+
+    if (!_type && _match && _match.length > 1) {
+      _type = "reputation";
+    }
+
     // Detect enhanced-type from enhanced error code
     // Notice: this is done as a last resort, as sometimes enhanced code \
     //   doesn't clearly convey the actual error

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "email-bounce-parser",
-  "version": "1.0.17",
+  "version": "1.0.19",
   "description": "Parses bounce emails and extract errors",
   "author": "Eliott Vincent <eliott@crisp.chat>",
   "main": "lib/index.js",