Skip to content
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

Setting a default from address for emails #329

Merged
merged 2 commits into from
Jul 19, 2024
Merged

Conversation

spilth
Copy link
Collaborator

@spilth spilth commented Jul 19, 2024

Problem

While testing out the sendmail switch, sending email now generated the following error:

ArgumentError (SMTP From address may not be blank: nil)

Solution

Set a default from email address for ActionMailer: [email protected]

Type

bug-fix

@spilth spilth requested review from ekate and mnyrop July 19, 2024 16:44
@mnyrop
Copy link
Member

mnyrop commented Jul 19, 2024

@spilth I just sent an email to [email protected] to see who that alias routes to / who controls it. I'm worried it might go to someone no longer at NYU, so we'll want to confirm it's a good one to use before merging. (cc @nmwolf @ekate)

@spilth
Copy link
Collaborator Author

spilth commented Jul 19, 2024

@mnyrop Thank you for checking. I took it for granted that a no-reply email address goes nowhere.

@mnyrop
Copy link
Member

mnyrop commented Jul 19, 2024

@spilth You're right that it shouldn't (it would be unfortunate if someone is getting spammed!) but I also wish we knew who controlled it so we know who can change SMTP settings if needed.

I'm assuming since it's @ geo.nyu.edu and not @ nyu.edu that there is a mail server running on the EC2...? Unless it's somehow aliased to an NYU via google apps account...? Or do you think it's irrelevant? I'm lobbing shots in the dark but am now leaning towards approving & merging the PR now, having you test that it works for what we need, and worrying about that detail later. What would your preference be?

@spilth
Copy link
Collaborator Author

spilth commented Jul 19, 2024

@mnyrop I say we try out the email address you sent me in Slack and we see if it works in staging. If it does, then we'll just need to find the right address. If it doesn't, then we'll need to figure something else out 😅

@spilth spilth merged commit a9d9a8c into main Jul 19, 2024
1 check passed
@spilth spilth deleted the 310-add-default-from-address branch July 19, 2024 19:40
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.

2 participants