Skip to content

fix unmatched quote, and allow for from address, etc. config#4

Open
nachbar wants to merge 1 commit intoTalkingQuickly:masterfrom
nachbar:master
Open

fix unmatched quote, and allow for from address, etc. config#4
nachbar wants to merge 1 commit intoTalkingQuickly:masterfrom
nachbar:master

Conversation

@nachbar
Copy link

@nachbar nachbar commented Nov 9, 2015

There was a mismatched single quote as a result of the recent (June) patch, which prevented monit from connecting to my smtp server. monit also complains if the port is quoted.

Also, my smtp server wanted the "from" email address to be set. Thus, this commit addresses those two issues.

Thanks!

@nachbar
Copy link
Author

nachbar commented Nov 9, 2015

I didn't see the other pull request by japaternoster before submitting this one. That pull request also fixes the mismatched quote and removes the quoting from the port.

The quoting of the hostname is permissive, so it can be left or removed at your discretion.

@andreiglingeanu
Copy link

andreiglingeanu commented May 17, 2016

Sorry, I just added one more pull request that fixes this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants