Till Brehm
2015-09-18 d8faa4e7e3fb74e554f6c9339f6fe4e2883c3bf1
refs
author Till Brehm <tbrehm@ispconfig.org>
Friday, September 18, 2015 13:43 -0400
committer Till Brehm <tbrehm@ispconfig.org>
Friday, September 18, 2015 13:43 -0400
commitd8faa4e7e3fb74e554f6c9339f6fe4e2883c3bf1
tree f61a6192f71a99cdd5a82717e3252830758317d7 tree | zip | gz
parent 5cc181a90f7d27f4ec578a8a892b4dd0ca44adf4 view | diff
Fix by dirkd: Since file() doesn't remove "\n" from the lines read the condition $lines[$l] == '' never evaluated to true. So $mailHeaders would always contain every line from the template while $mailBody contained none of them.

Together with the bugfix of PHP#68776 [1] disallowing multiple "\n"'s & preceding "\n" in mail()'s $additionalHeaders arg this caused mails not being sent because $mailHeaders would include at least one "\n\n" which was meant to separate the mail headers from the body.

[1] https://bugs.php.net/bug.php?id=68776
1 files modified
2 ■■■ changed files
server/lib/classes/monitor_tools.inc.php 2 ●●● diff | view | raw | blame | history