Actions
Bug #4560
closedapcupsd is missing support for SMTP TLS email and uses old check for SSL setting
Start date:
03/28/2015
Due date:
% Done:
100%
Estimated time:
0.25 h
Plus Target Version:
Affected Version:
2.2
Affected Plus Version:
Affected Architecture:
All
Description
apcupsd_mail.php has the following code for the SMTP SSL setting
if ($config['notifications']['smtp']['ssl'] == "checked") $mail->SMTPSecure = "ssl";
The test does not support the new style of setting in 2.2 and is missing the use of the TLS setting. It should mimic the code in mail_reports.inc
if ((isset($config['notifications']['smtp']['ssl']) && $config['notifications']['smtp']['ssl'] != "unchecked") || $config['notifications']['smtp']['ssl'] == "checked") $mail->SMTPSecure = "ssl"; if ((isset($config['notifications']['smtp']['tls']) && $config['notifications']['smtp']['tls'] != "unchecked") || $config['notifications']['smtp']['tls'] == "checked") $mail->SMTPSecure = "tls";
Updated by Renato Botelho about 9 years ago
- Status changed from New to Feedback
- Target version deleted (
2.3) - % Done changed from 0 to 100
Pull request has been merged
Updated by Chris Buechler almost 9 years ago
- Status changed from Feedback to Resolved
Actions