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

SMTP proxy or secure mail API credentials so instances can send email #24

Closed
benallfree opened this issue Oct 24, 2022 · 4 comments
Closed
Milestone

Comments

@benallfree
Copy link
Collaborator

Can SES make a mail user and can we restrict sending from the pockethost.io domain only?

@benallfree
Copy link
Collaborator Author

SES does not do per-user quotas for IAM users. Instead, we need to implement an smtp proxy https://nodemailer.com/extras/smtp-server/

This is good though, because we can properly enforce:

  • Sending from the instance's subdomain
  • Sending ONLY to email addresses present in the users collection
  • Limiting sending size and frequency
  • Creating upgrade plans (100/day, 1k/day, etc)

@benallfree benallfree added this to the 0.9 milestone Nov 3, 2022
@benallfree
Copy link
Collaborator Author

This one does feel important. Setting up email is a PITA. For the instances to really be viable, people need confirmation emails and pw resets. The only reason we haven't seen this yet is because nobody is in production.

@benallfree benallfree changed the title Secure mail API credentials SMTP proxy or secure mail API credentials so instances can send email Jan 11, 2023
@benallfree
Copy link
Collaborator Author

Another note: might be a good idea to proxy pockethost-central emails through an smtp proxy because if a verification email is sent and causes a hard bounce, SES will block it permanently. We need a way to un-block hard bounces in SES before resending verification emails. At least 10 times or something.

A user had an issue where his email address was not activated and then SES blocked it so he was stuck unable to verify his account.

@benallfree
Copy link
Collaborator Author

moved to roadmap

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

No branches or pull requests

1 participant