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:49:14 +0200 |
commit | c20fa4a1ad57a6fa013c7c9ddba72c6c3aafe345 (patch) | |
tree | 5e04feb34741e7c92f46c7fb45c938da762efdcb /program/steps | |
parent | 672a55e6559fd28f81279cc548ce44b00041b477 (diff) |
Fix address matching in Return-Path header on identity selection (#1489374)
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 48afecb60..70441e0d7 100644 --- a/program/steps/mail/func.inc +++ b/program/steps/mail/func.inc @@ -1813,15 +1813,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; } |