-
-
Notifications
You must be signed in to change notification settings - Fork 10.6k
New issue
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
The from address on the confirmation email for updating email in Admin Settings (Support/Newsletter) is incorrect #12802
Comments
Hey @shubhank008, I hear that you are frustrated but I have to be honest. I've read this 3x and I don't quite follow what you are saying. Please can you go back to the issue template we provided and supply a step by step reproduction case demonstrating the problem so that we can try to reproduce it? |
Woops sorry, edited my initial issue above. I am not frustrated as I found a way to fix it (albeit using a solution provided by another user in ghost forums facing same issue and pointing the problem in settings.js). To try and explain it a bit more:
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This is still a problem, using Ghost 4.18 with a custom SMTP server, ghost is not using the from adress specified in config and uses [email protected] instead of [email protected] like said in config. EDIT:
as you see in the logs, I've set the from address to be "[email protected]" but ghost doesn't send it with this address |
Its also in the docs https://ghost.org/docs/config/#from-address Ping @ErisDS |
Quite disappointed has been months since it was reported and in so much details and it was a 2 line code fix (well rather override the config) but the bug its still there. Makes me wonder do not a whole lot ghost users use SMTP or check/test their mails ? |
Sometime it seems they don't see the issues :\ |
Reopening this as a valid bug. There is some weird interplay between the bulk email and SMTP email settings going on. When trying to setup the support and newsletter settings, we send an email to try to validate that the email address being set is usable. Although these settings are for bulk email, the emails we send are sent via SMTP. The logic for what the from address should be needs reviewing. It may be a case that we can simply change this to use the SMTP "from" address, but I will need to check how this behaves on Ghost(Pro) where we make use of the ability to setup mailgun via config rather than the UI. Does someone want to put together a PR for this change and we can collaborate on testing all the cases? |
Our bot has automatically marked this issue as stale because there has not been any activity here in some time. The issue will be closed soon if there are no further updates, however we ask that you do not post comments to keep the issue open if you are not actively working on a PR. We keep the issue list minimal so we can keep focus on the most pressing issues. Closed issues can always be reopened if a new contributor is found. Thank you for understanding 🙂 |
This is still an issue. Supplying |
I encounter the issue today as well. |
This is still a valid bug. As of today this issue is not yet resolved |
Facing this issue. Is there any temporary workaround for now. I am using aws smtp |
You can use the temporary codefix I listed in my original report (the very top post) |
I did that fix and restarted the server and the mail is still being sent from noreply. Is there something i am missing . I have cleared the cache and tried too but it does not work |
Anyone still facing this god forsaken issue, my fix still works except the location/file where you need to make the change has changed in latest ghost (v5) from And don't forget to set the
|
I had this issue too, and I found the solution, but forgot to post it here... Anyway, after having created multiple issues here on Github and on the Ghost Forum, someone from their staff gave the answer on this forum thread. I hope it helps you and others :) |
@marnixhoh I think you are confusing a different issue. @shubhank008 If you have a fix for the original issue, please raise a PR. |
Hi, I'd also like to provide more detail around this issue, caused a problem for me. Bit of background, I've successfully deployed an azure multi-container instance, with persistent storage (Azure File Share). It uses caddy docker image for lets-encrypt and reverse proxy, the mysql-8 docker image for DB and the docker image for ghost v5. My compose file looks like this:
The accompanying local .env file defines these values as the MAILFROM and GHOSTURL. MAILFROM=[email protected] I also have my own postfix box. If a member attempts sign up, the mail from address used by Ghost is: [email protected], which is ignoring the 'mail__from:' set in the environment, further resulting in googlemail rejecting it as spam. I have found though, going into /ghost/#/settings/members?showPortalSettings=true, under support email address, shows that it is indeed [email protected]. Changing this too the desired email and confirming ownership by receipt, will change the address used for member sign-ups, after doing so, I now get this in the postfix logs. Status 250 = Ok is basically smtp speak for, we got and will deliver your mail, sure enough, the google account received the mail. So, why isn't the environment variable defined for 'mail__from' being honoured? Restarting the container group does save the address so at least membership/transactional mails are functional, took a while to work this out mind. Also, why the constraint for mailgun as bulk mail sender? SendGrid for example, offers 100 mails a month for free, using your own domain, whereas mailgun has an associated cost for the same service. Better value proposition for a blogger just starting out. Apologies for the long post, just thought it might help someone struggling with this same issue and to help understand and narrow down what needs fixing in a PR. Great app btw, thanks |
Our bot has automatically marked this issue as stale because there has not been any activity here in some time. The issue will be closed soon if there are no further updates, however we ask that you do not post comments to keep the issue open if you are not actively working on a PR. We keep the issue list minimal so we can keep focus on the most pressing issues. Closed issues can always be reopened if a new contributor is found. Thank you for understanding 🙂 |
Changing setting under |
This comment was marked as abuse.
This comment was marked as abuse.
Please see my responses over on the PR here: #15989 (comment) TL;DR we have a piece of work to change how from addresses work in a holistic way, that is scheduled to be started on immediately. |
@ErisDS Thanks for getting the ball rolling on fixing the various issues with how ghost deals with mail. Will we have appropriate control over setting email from addresses consistent with Mailgun and others? It would set some minds at ease if the plan was actually shared with folks rather than springing the result on us. We all saw how the surprise approach works vis-a-vis the admin settings UI change debacle. |
The issue remains. |
How is it possible this issue to not be fixed for 2 years?! I am very close from moving to a different platform, because of something so simple. |
Actually I solved it (if you really want to use personal email to send newsletter or support), by setting email alias like [email protected] and [email protected] in your SMTP service provider. |
Google and Yahoo are introducing major changes to enforce DMARC in the next 2 months. The core team is currently doing a larger piece of work to support DMARC alignment in email sending, which overlaps-with and will also resolve this issue. These changes will be released in January 2024. |
This comment was marked as off-topic.
This comment was marked as off-topic.
fixes TryGhost#12802 - You can choose any support and newsletter email address in the UI without verification (as long as your SMTP-server / Mailgun can send from it) - All emails will use the mail.from config as the from address as a default: - Staff notification emails no longer use the made up ghost@domain email address - Newsletters no longer default to 'noreply@domain' - Member related emails (signin/signup/comment notifications...) will continue to be send from the chosen support address (Portal settings → Account page), but will now default to the mail.from config instead of noreply@domain if no support address is set.
ref #12802 fixes DMA-27 - You can choose any support and newsletter email address in the UI without verification (as long as your SMTP-server / Mailgun can send from it) - All emails will use the mail.from config as the from address as a default: - Staff notification emails no longer use the made up ghost@domain email address - Newsletters no longer default to 'noreply@domain' - Member related emails (signin/signup/comment notifications...) will continue to be send from the chosen support address (Portal settings → Account page), but will now default to the mail.from config instead of noreply@domain if no support address is set.
🎉 This issue and related issues have been fixed in v5.78.0. Changes in Ghost v5.78.0
What do you need to do
We've updated the documentation about the config, and recommend to check out our post on the forum about DMARC alignment and the new requirements from Google and Yahoo. |
How about create config for mail.support in addition to mail.from, so mail.from is only for sender from address, but the mail.support can be any e-mail address that intended for support e-mail, but this support e-mail will not treated as sender e-mail address when Ghost sending communication to user/member. I think this will makes the configuration more flexible and accomodate several use cases. |
3 years later and this issue is still happening #21423 |
Issue Summary
Currently, using Ghost v4.0, if you try to update the emails in your Admin Settings of Ghost under Members->Email and try to update either support or newsletter, they are set to [email protected]
They should ideally use the "from" address if set in your config's mail part.
To Reproduce
The bigger problem is not this, its the fact that if you set custom SMTP and even "from" address in your config's mail part, and try to update the emails given in your Admin Area above, Ghost ignores whatever is set in your config and still uses [email protected] or [email protected] which creates issue with SMTP config as when Ghost tries to send such mails (confirmation mails), the SMTP server will reject it as the "from" mail is different from the actual userid/smtp emailid.
Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
This is how postfix (SMTP) logs it:
The logic for this part (and how I fixed it) lies here:
core/server/services/members/settings.js
This is how Ghost currently does it
Ideally it should atleast check if a from address exists in your config.mail or use the smtp id instead. This just limits the usage of mail and SMTP in general to a very handful of providers.
I had to fix it by overriding fromEmail to use email itself and then I was able to update the mail in my Admin:
Technical details:
The text was updated successfully, but these errors were encountered: