Not sure if this was reported but we noticed an issue with 7.4.3 in Ubuntu with mailer field in the config - before it executes, it makes the characters to lower case. This is not good practice in case you have capital letter switches, for example: msmtp -C ... is converted to msmtp -c ... and totaly breaks the command.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Thanks for the report. The code was patched so that mailer is no longer lower-cased. The lower-casing of configuration values deserves a fuller review, but for now it should fix your issue.
The 7.4.3 release is no longer supported, and thus I recommend you move to the current 7.5.3. If you want to patch your older logwatch.pl script, you can replace the sub CleanVars code in your file with the newest version of sub CleanVars.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I'm glad this is fixed in new versions, let's hope our repo will eventually catch-up (in like 2025 :D ).
Due to maintenance reasons, I had to make a workaround by making lowercase wrapper script for my "mailer" msmtp. This should work for both older and newer versions and once we get to >7.5.3 I don't need to fix it back. :)
Thank you for your support and good work.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Just to clarify (perhaps for others): the patch was just pushed through based on your report, so will not be available under the "Files" tab until the next tagged release. If you download the current git snapshot (under the "Git" tab), the patch will be there. So your comment of > 7.5.3 is correct.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Not sure if this was reported but we noticed an issue with 7.4.3 in Ubuntu with mailer field in the config - before it executes, it makes the characters to lower case. This is not good practice in case you have capital letter switches, for example: msmtp -C ... is converted to msmtp -c ... and totaly breaks the command.
Thanks for the report. The code was patched so that mailer is no longer lower-cased. The lower-casing of configuration values deserves a fuller review, but for now it should fix your issue.
The 7.4.3 release is no longer supported, and thus I recommend you move to the current 7.5.3. If you want to patch your older
logwatch.pl script, you can replace thesub CleanVarscode in your file with the newest version ofsub CleanVars.I'm glad this is fixed in new versions, let's hope our repo will eventually catch-up (in like 2025 :D ).
Due to maintenance reasons, I had to make a workaround by making lowercase wrapper script for my "mailer" msmtp. This should work for both older and newer versions and once we get to >7.5.3 I don't need to fix it back. :)
Thank you for your support and good work.
Just to clarify (perhaps for others): the patch was just pushed through based on your report, so will not be available under the "Files" tab until the next tagged release. If you download the current git snapshot (under the "Git" tab), the patch will be there. So your comment of > 7.5.3 is correct.