We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
This one has been reported & fixed at least twice on the forum, here's one:
https://en.ghost.org/forum/using-ghost/263-bug-stray-n-character-when-auto-detecting-sendmail
The text was updated successfully, but these errors were encountered:
FYI - the fix from the forum was
- resolve(stdout.toString()); + resolve(stdout.toString().slice(0, -1));
Via email, someone else suggested:
diff --git a/core/server/mail.js b/core/server/mail.js index 541e173..16c3b57 100644 --- a/core/server/mail.js +++ b/core/server/mail.js @@ -51,7 +51,7 @@ GhostMailer.prototype.detectSendmail = function () { if (err && !/bin\/sendmail/.test(stdout)) { return reject(); } - resolve(stdout.toString()); + resolve(stdout.toString().replace(/(\n|\r|\r\n)$/, '')); }); }); };
Not sure whether we need one or both.
Sorry, something went wrong.
Fix for sendmail problem
347ded8
closes TryGhost#871 - added solution from email - tested on OSX
3def65e
closes #871 - added solution from email - tested on OSX
sebgie
Successfully merging a pull request may close this issue.
This one has been reported & fixed at least twice on the forum, here's one:
https://en.ghost.org/forum/using-ghost/263-bug-stray-n-character-when-auto-detecting-sendmail
The text was updated successfully, but these errors were encountered: