1
|
|
|
<?php |
2
|
|
|
/* |
3
|
|
|
* This file is part of Bounce-Detection. |
4
|
|
|
* |
5
|
|
|
* (c)2016 cwd.at GmbH <[email protected]> |
6
|
|
|
* |
7
|
|
|
* For the full copyright and license information, please view the LICENSE |
8
|
|
|
* file that was distributed with this source code. |
9
|
|
|
*/ |
10
|
|
|
|
11
|
|
|
namespace Cwd\BounceDetection; |
12
|
|
|
use PhpMimeMailParser\Parser; |
13
|
|
|
|
14
|
|
|
/** |
15
|
|
|
* Class Detector |
16
|
|
|
* |
17
|
|
|
* @package Cwd\BounceDetection |
18
|
|
|
* @author Ludwig Ruderstaller <[email protected]> |
19
|
|
|
*/ |
20
|
|
|
class Detector |
21
|
|
|
{ |
22
|
|
|
/** |
23
|
|
|
* @var RuleConfig|null |
24
|
|
|
*/ |
25
|
|
|
protected $ruleConfig = null; |
26
|
|
|
|
27
|
|
|
public function __construct($ruleFile = null) |
28
|
|
|
{ |
29
|
|
|
$this->ruleConfig = RuleFactory::getRules($ruleFile); |
30
|
|
|
} |
31
|
|
|
|
32
|
|
|
/** |
33
|
|
|
* @param string $mail |
34
|
|
|
* |
35
|
|
|
* @return array |
36
|
|
|
*/ |
37
|
247 |
|
public function testAll($mail) |
38
|
|
|
{ |
39
|
247 |
|
$result = $this->testRulesAgainst($this->ruleConfig->getDSNRules(), $mail); |
40
|
|
|
|
41
|
247 |
|
if ($result === null) { |
42
|
18 |
|
$result = $this->testRulesAgainst($this->ruleConfig->getBodyRules(), $mail); |
43
|
18 |
|
} |
44
|
|
|
|
45
|
247 |
|
if ($result === null) { |
|
|
|
|
46
|
|
|
// @todo |
47
|
18 |
|
} |
48
|
|
|
|
49
|
247 |
|
$result['email'] = $this->findEmail($mail); |
50
|
|
|
|
51
|
247 |
|
return $result; |
52
|
|
|
} |
53
|
|
|
|
54
|
|
|
/** |
55
|
|
|
* @param $mail |
56
|
|
|
* |
57
|
|
|
* @return string |
58
|
|
|
*/ |
59
|
247 |
|
public function findEmail($mail) |
60
|
|
|
{ |
61
|
247 |
|
foreach ($this->ruleConfig->getEmailRules() as $rule) { |
62
|
247 |
|
if (preg_match($rule['regexp'], $mail, $match)) { |
63
|
134 |
|
return $match[1]; |
64
|
|
|
} |
65
|
216 |
|
} |
66
|
|
|
|
67
|
113 |
|
return null; |
68
|
|
|
} |
69
|
|
|
|
70
|
|
|
/** |
71
|
|
|
* @param array $rules |
72
|
|
|
* @param string $mail |
73
|
|
|
* |
74
|
|
|
* @return array |
75
|
|
|
*/ |
76
|
247 |
|
public function testRulesAgainst(array $rules, $mail) |
77
|
|
|
{ |
78
|
247 |
|
$parser = new Parser(); |
79
|
247 |
|
$parser->setText($mail); |
80
|
247 |
|
$data = $parser->getData(); |
81
|
|
|
|
82
|
247 |
|
foreach ($rules as $rule) { |
83
|
247 |
|
if (preg_match($rule['regexp'], $data, $match)) { |
84
|
|
|
$result = [ |
85
|
229 |
|
'rule_cat' => $rule['category'], |
86
|
229 |
|
'rule_no' => $rule['number'], |
87
|
229 |
|
]; |
88
|
|
|
|
89
|
229 |
|
if (isset($rule['idx_email'])) { |
90
|
|
|
$result['email'] = $match[$rule['idx_email']]; |
91
|
|
|
} |
92
|
|
|
|
93
|
229 |
|
return $result; |
94
|
|
|
} |
95
|
242 |
|
} |
96
|
18 |
|
} |
97
|
|
|
} |
98
|
|
|
|
This check looks for the bodies of
if
statements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete.These
if
bodies can be removed. If you have an empty if but statements in theelse
branch, consider inverting the condition.could be turned into
This is much more concise to read.