diff options
author | Aleksander Machniak <alec@alec.pl> | 2013-10-15 18:49:14 +0200 |
---|---|---|
committer | Aleksander Machniak <alec@alec.pl> | 2013-10-15 18:53:09 +0200 |
commit | acbf6eede613a5496cabccbdbfcb2f4e2cbd60d2 (patch) | |
tree | 0c953a71a8222671abf6fd9bf98e8030a459e324 /program/steps | |
parent | 571aa9b04ea6345acc98e1671a8ebcfd7abb4f62 (diff) |
Fix address matching in Return-Path header on identity selection (#1489374)
Conflicts:
CHANGELOG
Diffstat (limited to 'program/steps')
-rw-r--r-- | program/steps/mail/func.inc | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/program/steps/mail/func.inc b/program/steps/mail/func.inc index 8de76c3c9..2938e91e1 100644 --- a/program/steps/mail/func.inc +++ b/program/steps/mail/func.inc @@ -1802,15 +1802,19 @@ function rcmail_identity_select($MESSAGE, $identities = null, $compose_mode = 'r // Try Return-Path if ($from_idx === null && ($return_path = $MESSAGE->headers->others['return-path'])) { + $return_path = array_map('strtolower', (array) $return_path); + foreach ($identities as $idx => $ident) { // Return-Path header contains an email address, but on some mailing list // it can be e.g. <pear-dev-return-55250-local=domain.tld@lists.php.net> // where local@domain.tld is the address we're looking for (#1489241) - $ident1 = $ident['email_ascii']; + $ident1 = strtolower($ident['email_ascii']); $ident2 = str_replace('@', '=', $ident1); + $ident1 = '<' . $ident1 . '>'; + $ident2 = '-' . $ident2 . '@'; - foreach ((array)$return_path as $path) { - if (stripos($path, $ident1) !== false || stripos($path, $ident2)) { + foreach ($return_path as $path) { + if ($path == $ident1 || stripos($path, $ident2)) { $from_idx = $idx; break 2; } |