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.

Sign up for GitHub

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

Jump to bottom

Handle incorrect emails #130

Closed
sethherr opened this issue Mar 18, 2016 · 4 comments · May be fixed by #366
Closed

Handle incorrect emails #130

sethherr opened this issue Mar 18, 2016 · 4 comments · May be fixed by #366
Labels
Authentication Issues relating to user authentication Ruby Small

Comments

@sethherr
Copy link
Member

SparkPost sends us errors when it can't send because of invalid emails.

Net::SMTPSyntaxError: 501 5.5.2 RCPT TO syntax error

We should probably delete the user account if this happens. We definitely shouldn't retry the email_confirmation_worker 18 times, which is what we do right now 😹
@mftaff
Copy link

mftaff commented Jan 16, 2018

@sethherr is this still an issue? I am new-ish to open source contributions, and would love to give it a shot?

@sethherr
Copy link
Member Author

This is still an issue @mftaff - it would be lovely if you gave it a shot.

@mftaff mftaff mentioned this issue Jan 22, 2018
Open
@sethherr sethherr added the Authentication Issues relating to user authentication label Jul 10, 2019
@parker-erickson
Copy link

parker-erickson commented Dec 9, 2021 •
edited

@sethherr Hey! I see this issue is pretty old, I am also new to open source contributions, and if this is still needed I would love to try solving the problem. Could I give it a go?

@sethherr
Copy link
Member Author

Hey @parker-erickson - thanks for your enthusiasm! Looking at the current errors, it doesn't seem like the sparkpost issue is still a problem.

Thanks for saying something, this seems like it can be closed.

Look for another issue that seems interesting and comment over there?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Authentication Issues relating to user authentication Ruby Small
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Delete user if email isn't confirmable mftaff/bike_index
3 participants
@sethherr @mftaff @parker-erickson