email-bounce-parser 1.0.15 → 1.0.17

package/README.md

@@ -39,7 +39,7 @@ console.log(result);
 ### Parse a bounce email
 
 `read(body)` parses the bounce email and extracts available information (error code and type, server, original recipient, etc.):
-* `body` must be a string representing the bounce email body (as returned by [mailparser](https://github.com/nodemailer/mailparser), for example)
+* `body` must be a string representing the bounce email **text** body (not HTML body). You can use [mailparser](https://github.com/nodemailer/mailparser) for example.
 
 ```js
 const EmailBounceParse = require("email-bounce-parser");

package/lib/parser.js

@@ -8,6 +8,7 @@ var ERROR_CODES_TYPES = {
   "421"        : "service_not_available",
   "421 4.3.0"  : "temporary_system_problem",
   "421 4.4.5"  : "server_busy",
+  "421 4.7.28" : "ip_blocked_spam",
   "432 4.7.12" : "password_needed",
   "450"        : "action_not_taken_mailbox_unavailable",
   "450 4.2.1"  : "action_not_taken_mailbox_rate_limited",
@@ -64,8 +65,8 @@ var REGEXES = {
     /\(in reply to end of ([A-Z\s]+) command\)$/m, // "in reply to end of DATA command"
     /\(in reply to ([A-Z\s]+) command\)$/m, // "in reply to RCPT TO command"
     /timed out while sending ([A-Z\s]+)$/m, // "timed out while sending (RCPT TO)", "timed out while sending RCPT TO"
-    /error after ([A-Z\s]+)/m, // "LMTP error after RCPT TO"
-    /while sending end of ([a-z\s]+)/m // "while sending end of data"
+    /error after ([A-Z\s]+)/i, // "LMTP error after RCPT TO"
+    /while sending end of ([a-z\s]+)/i // "while sending end of data"
   ],
 
   email_server : [
@@ -84,7 +85,7 @@ var REGEXES = {
     /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,
+    /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?)?) (.+)/,
@@ -94,18 +95,29 @@ var REGEXES = {
   ],
 
   error_lax : [
-    /(^\<.+?\>:.+)/m
+    /^(?:\<.+?\> \(expanded from \<.+?\>\)\:)(.+)/m, // "<john@acme.com> (expanded from <john@digiflex.com>):  Domain name not found"
+    /^(?:\<.+?\>\:)(.+)/m, // "<john@acme.com>: Domain name not found"
+  ],
+
+  error_label_cleanup_recipient : [
+    /^\<.+?\> \(expanded from \<.+?\>\)\:/m, // "<john@acme.com> (expanded from <john@digiflex.com>):  Domain name not found"
+    /^\<.+?\>\:/m, // "<john@acme.com>: Domain name not found"
+  ],
+
+  error_label_cleanup_command : [
+    /\(in reply to (?:end of)?[A-Z\s]+ command\)$/m // "in reply to DATA command" or "in reply to end of DATA command"
   ],
 
   recipient : [
-    /The following addresses had permanent fatal errors (?:.*?)\s\<(.+?)\>/,
-    /The following message to \<(.+?)\> was undeliverable/,
-    /to the following addresses:\s*(.+)/,
+    /the following addresses had permanent fatal errors (?:.*?)\s\<(.+?)\>/i,
+    /the following message to \<(.+?)\> was undeliverable/i,
+    /to the following addresses:\s*(.+)/i,
     /^\<(.+?)\>:/m,
-    /^(.+?)\s?\n?\s*[\[|<]mailto\:(.+?)[\]|>]/m,
-    /^Your message wasn't delivered to (.+?)\s/m,
-    /^Your message to (.+?) couldn\'t/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"
+    /\<(.+?)\> \(expanded from \<.+?\>\)\:/, // "<john@acme.com> (expanded from <john@digiflex.com>): "
+    /\<(.+?)\>:/ // "<john@acme.com>:"
   ],
 
   temporary_safe : /(temporary|temporarily)/i,
@@ -122,7 +134,9 @@ var REGEXES = {
   ],
 
   enhanced_types_server_unknown : [
-    /(does not accept mail)/i
+    /(does not accept mail)/i,
+    /(domain name not found)/i,
+    /(no route to host)/i
   ],
 
   enhanced_types_server_timeout : [
@@ -134,8 +148,8 @@ var REGEXES = {
   ],
 
   error_code : [
-    /(\d{3})[\s\-](\d\.\d\.\d)/,  // "550 5.4.1" or "550-5.4.1"
-    /(\d{3})\-\'(\d\.\d\.\d)/, // "550-'5.4.1"
+    /(\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})/ // "550"
   ]
 };
@@ -276,7 +290,7 @@ class Parser {
 
     if (_match && _match.length > 1) {
       _code  = this.__parseErrorCode(_match[1]);
-      _label = trimString(_match[2]);
+      _label = this.__cleanErrorLabel(_match[2]);
 
       // Check temporary / permanent
       if (_code.basic[0] === "4") {
@@ -306,7 +320,7 @@ class Parser {
       var _match = loopRegexes(this.__regexes.error_lax, str);
 
       if (_match && _match.length > 1) {
-        _label = trimString(_match[1]);
+        _label = this.__cleanErrorLabel(_match[1]);
       }
     }
 
@@ -550,6 +564,50 @@ class Parser {
 
     return _str;
   }
+
+
+  /**
+   * Cleans error label
+   * @private
+   * @param  {string} [label]
+   * @return {string} Unquoted text
+   */
+  __cleanErrorLabel(label="") {
+    var _label = this.__unquoteText(label);
+
+    _label = trimString(_label);
+
+    // Strip recipient (at the beginning) and command (at the end), in case \
+    //   they are still present
+    _label = loopRegexes(this.__regexes.error_label_cleanup_recipient, _label, "replace");
+    _label = loopRegexes(this.__regexes.error_label_cleanup_command, _label, "replace");
+
+    _label = trimString(_label);
+
+    return _label;
+  }
+
+
+  /**
+   * Unquotes text
+   * @private
+   * @param  {string} [str]
+   * @return {string} Unquoted text
+   */
+  __unquoteText(str="") {
+    var _str = str;
+
+    _str = trimString(_str);
+
+    if (
+      (_str.startsWith("'") && _str.endsWith("'")) ||
+      (_str.startsWith("\"") && _str.endsWith("\""))
+    ) {
+      return _str.slice(1, -1);
+    }
+
+    return _str;
+  }
 }
 
 

package/lib/utils.js

@@ -23,6 +23,8 @@ var loopRegexes = function(
   regexes, str, mode = "match", highestPosition = true
 ) {
   var _match;
+  var _minLength = mode === "split" ? 1 : 0;
+  var _maxLength = str.length;
 
   for (var _i = 0; _i < regexes.length; _i++) {
     var _regex = regexes[_i];
@@ -40,15 +42,12 @@ var loopRegexes = function(
       _match = _currentMatch;
 
       // A successfull replace must return a smaller string than the input
-      var _maxLength = str.length;
-
       if (_match.length < _maxLength) {
         break;
       }
     } else if (mode === "split" || mode === "match") {
-      // A successfull split must return an array with at least two items
-      var _minLength = mode === "split" ? 1 : 0;
 
+      // A successfull split must return an array with at least two items
       if ((_currentMatch || []).length > _minLength) {
         if (highestPosition === true) {
           // No previous match?
@@ -80,7 +79,7 @@ var loopRegexes = function(
     }
   }
 
-  return (_match || []);
+  return (mode === "replace") ? (_match || "") : (_match || []);
 }
 
 
@@ -91,7 +90,7 @@ var loopRegexes = function(
  * @return {string} The trimed string
  */
 var trimString = function(str) {
-  return ((str || "").trim() || null)
+  return ((str || "").trim() || "");
 }
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "email-bounce-parser",
-  "version": "1.0.15",
+  "version": "1.0.17",
   "description": "Parses bounce emails and extract errors",
   "author": "Eliott Vincent <eliott@crisp.chat>",
   "main": "lib/index.js",
@@ -30,7 +30,9 @@
     "mail",
     "email",
     "bounce",
-    "bounces"
+    "bounces",
+    "smtp",
+    "rfc"
   ],
   "license": "MIT"
 }