quantum / esos

An open source, high performance, block-level storage platform.
http://www.esos-project.com/
Other
286 stars 58 forks source link

Consider changing the mail agent from SSMTP to MSMTP. #207

Open 6ax opened 6 years ago

6ax commented 6 years ago

Hi,

Ssmtp has known problems when sending mail from a smartd through ssmtp. Since the syntax for ssmtp is different of the syntax of mail it is needed to create a script to send the email and tell smartd to use it. The ssmtp command is not compatible. It emulates the sendmail command which requires to prepend header lines (e.g. "Subject") before mail text.

Details here: https://sourceforge.net/p/smartmontools/mailman/message/32312279/ http://www.openstora.com/wiki/index.php?title=Fixing_smartmontools https://wiki.archlinux.org/index.php/S.M.A.R.T.

Please consider the possibility of replacing or adding msmtp, and then select the preferred agent (ssmtp or msmtp) in the TUI for compatibility with previous versions of ESOS.

6ax commented 6 years ago

Or include in the ESOS this adapted script for sending via ssmtp. Tnx.

smartd_warning.sh.txt

msmith626 commented 6 years ago

Where does that script go? It replaces an existing script with the same name? I'm open to swapping to a different mailer, but it's a considerable amount of work. Patches / merge requests are welcome.

--Marc

6ax commented 6 years ago

smartd_warning.sh.patch.txt

msmith626 commented 6 years ago

Right... where does this go? It replaces an existing file?

Marc A. Smith Founder, CTO / Parodyne Inc.

// marc.smith@parodyne.com // Mobile: 1-810-955-4735 // Office: 1-877-632-9004

// 4400 S. Saginaw St. Suite 1325 // Flint, MI 48507 USA // www.parodyne.com

On Wed, Jan 17, 2018 at 10:11 AM, 6ax notifications@github.com wrote:

smartd_warning.sh.patch.txt https://github.com/parodyne/esos/files/1639461/smartd_warning.sh.patch.txt

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/parodyne/esos/issues/207#issuecomment-358334343, or mute the thread https://github.com/notifications/unsubscribe-auth/AFkL3mWUYeXLgfjzQklAGRJpWPn7p5cRks5tLg2rgaJpZM4RhNF9 .

6ax commented 6 years ago

Yes, right. Replaces an existing file.

feldsam commented 6 years ago

looks like not, but will be fine have hdd smart monitoring in healthcheck script