proof-of-concept done

Author: rufinus

.travis.yml

@@ -21,9 +21,12 @@ matrix:
 
 before_install:
     - sudo apt-get -y -qq update
-#    - sudo apt-get -y -qq install fontconfig libxrender1 xfonts-base xfonts-75dpi
+    - if [[ "$TRAVIS_PHP_VERSION" = "7.0" ]]; then composer require --dev --no-update phpunit/phpunit ~5 ; fi
 
 before_script:
+    - if [[ "$TRAVIS_PHP_VERSION" = "5.5" ]]; then pecl install mailparse-2.1.6 ; fi
+    - if [[ "$TRAVIS_PHP_VERSION" = "5.6" ]]; then pecl install mailparse-2.1.6 ; fi
+    - if [[ "$TRAVIS_PHP_VERSION" = "7.0" ]]; then pecl install mailparse ; fi
     - composer self-update
     - composer install --no-interaction
 

composer.json

@@ -15,8 +15,10 @@
         }
     ],
     "require": {
-        "php": ">=5.5.0",
-        "symfony/yaml": "^3.1"
+        "php":           "^5.5.0 || ^7.0",
+        "ext-mailparse": "*",
+        "symfony/yaml": "^3.1",
+        "php-mime-mail-parser/php-mime-mail-parser": "^2.5"
     },
     "require-dev": {
         "phpunit/phpunit": "~4.8",

src/Detector.php

@@ -9,6 +9,7 @@
  */
 
 namespace Cwd\BounceDetection;
+use PhpMimeMailParser\Parser;
 
 /**
  * Class Detector
@@ -35,23 +36,61 @@ public function __construct($ruleFile = null)
      */
     public function testAll($mail)
     {
-        $result = $this->testRulesAgainst($this->ruleConfig->getBodyRules(), $mail);
+        $result = $this->testRulesAgainst($this->ruleConfig->getDSNRules(), $mail);
 
         if ($result === null) {
-            $result = $this->testRulesAgainst($this->ruleConfig->getDSNRules(), $mail);
+            $result = $this->testRulesAgainst($this->ruleConfig->getBodyRules(), $mail);
         }
 
+        if ($result === null) {
+
+        }
+
+        $result['email'] = $this->findEmail($mail);
+
         return $result;
     }
 
+    /**
+     * @param $mail
+     *
+     * @return string
+     */
+    public function findEmail($mail)
+    {
+        foreach ($this->ruleConfig->getEmailRules() as $rule) {
+            if (preg_match($rule['regexp'], $mail, $match)) {
+                return $match[1];
+            }
+        }
+
+        return null;
+    }
+
+    /**
+     * @param array  $rules
+     * @param string $mail
+     *
+     * @return array
+     */
     public function testRulesAgainst(array $rules, $mail)
     {
+        $parser = new Parser();
+        $parser->setText($mail);
+        $data = $parser->getData();
+
         foreach ($rules as $rule) {
-            if (preg_match($rule['regexp'], $mail, $match)) {
-                return [
+            if (preg_match($rule['regexp'], $data, $match)) {
+                $result = [
                     'rule_cat' => $rule['category'],
                     'rule_no'  => $rule['number'],
                 ];
+
+                if (isset($rule['idx_email'])) {
+                    $result['email'] = $match[$rule['idx_email']];
+                }
+
+                return $result;
             }
         }
     }

src/RuleConfig.php

@@ -48,6 +48,23 @@ public function getDSNRules()
         return (isset($this->rules['dsn'])) ? $this->rules['dsn'] : null;
     }
 
+    /**
+     * @return array
+     */
+    public function getEmailRules()
+    {
+        return (isset($this->rules['email'])) ? $this->rules['email'] : null;
+    }
+
+    /**
+     *
+     * @return string|null
+     */
+    public function getDiagnosticRegExp()
+    {
+        return isset($this->rules['diagnostic']) ? $this->rules['diagnostic'] : null;
+    }
+
     /**
      * @return \ArrayIterator
      */

src/rules.yml

@@ -561,6 +561,14 @@ dsn:
     category: 'delayed'
     number: '0213'
 
+  - regexp: '/Delivery to the following recipient has been delayed/is'
+    category: 'delayed'
+    number: '0214'
+
+  - regexp: '/Delivery to the following recipient failed permanently/is'
+    category: 'unknown'
+    number: '0245'
+
   - regexp: '/(?:alias|account|recipient|address|email|mailbox|user)(.*)invalid/i'
     category: 'unknown'
     number: '0107'
@@ -658,14 +666,9 @@ email:
   - regexp: '/Final-Recipient: rfc822;(.*)/i'
   - regexp: '/quota exceed.*<(\S+@\S+\w)>/is'
 
-action:
-  - regexp: '/Action: (.+)/i'
-
-status:
-  - regexp: '/Status: ([0-9\.]+)/i'
-
-diagnostic:
-  - regexp: '/Diagnostic-Code:((?:[^\n]|\n[\t ])+)(?:\n[^\t ]|$)/is'
+action: '/Action: (.+)/i'
+status: '/Status: ([0-9\.]+)/i'
+diagnostic: '/Diagnostic-Code:((?:[^\n]|\n[\t ])+)(?:\n[^\t ]|$)/is'
 
 
 

tests/BounceTest.php

@@ -21,6 +21,28 @@
  */
 class BounceTest extends \PHPUnit_Framework_TestCase
 {
+
+    protected $falsePositiv = [
+        'tests/fixtures/bounce-email/tt_1234210666.txt',
+        'tests/fixtures/bounce-email/tt_1234211024.txt',
+        'tests/fixtures/bouncehammer/cannot-parse.eml',
+        'tests/fixtures/tmail_bouncer/legit_multipart.eml',
+        'tests/fixtures/tmail_bouncer/yahoo_legit.eml',
+        'tests/fixtures/tmail_bouncer/out_of_office.eml',
+        'tests/fixtures/PHP-Bounce-Handler/46.eml',
+        'tests/fixtures/PHP-Bounce-Handler/29.eml',
+        'tests/fixtures/mail-deliverystatus-bounceparser/spam-with-image.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/aol-senderblock.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/polish-autoreply.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/spam-lots-of-bogus-addresses.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/aol-vacation.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/spam-with-badly-parsed-email.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/bluebottle.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/no-message-collected.msg',
+        'tests/fixtures/mail-deliverystatus-bounceparser/autoreply.msg',
+        'tests/fixtures/node-baunsu/encoded_spam.txt',
+    ];
+
     /**
      * @var Detector
      */
@@ -53,16 +75,9 @@ public function testMail($fileName)
 
         $result = self::$detector->testAll($mail);
 
-        /*
-        if ($result === null) {
-            dump([$fileName => '-- Nothing found --']);
-        } else {
-            dump([$fileName => $result]);
+        if (!in_array($fileName, $this->falsePositiv)) {
+            $this->assertEquals(3, count($result));
         }
-         */
-
-        $this->assertEquals(true, is_array($result));
-
     }
 
     /**