Skip to content
This repository has been archived by the owner on Jul 12, 2023. It is now read-only.

Allow realms to integrate with an SMTP server #787

Closed
whaught opened this issue Oct 8, 2020 · 0 comments · Fixed by #859
Closed

Allow realms to integrate with an SMTP server #787

whaught opened this issue Oct 8, 2020 · 0 comments · Fixed by #859
Labels
kind/enhancement New feature or feature request.

Comments

@whaught
Copy link
Contributor

whaught commented Oct 8, 2020

TL;DR

Currently we use the default firebase email flow for password reset.
If we wish to use email for a 2nd factor, controler/eliminate rate-limiting concerns, and customize invitations, we need to be able to send our own emails.

Design

Proposal

  1. Use the golang SMTP package to send email from a custom template
  2. Include a realm setting for integrating with an email server credential
  3. Create a realm setting that includes a text template for inviting new users.
  4. Replace our hacked SendVerificationEmail with the template and use the server SDK generate link (which has 10x quota).

Alternatives considered
Integrate directly with SendGrid APIs

Additional information
Once we have this capability we can consider using this for TOTP sent via email for 2nd factor auth along with Twilio to replace the SMS based auth.

@icco icco added the kind/enhancement New feature or feature request. label Oct 8, 2020
@whaught whaught changed the title Allow realms to integrate with an SMTP server Allow serverl-level SMTP server override Oct 9, 2020
@whaught whaught changed the title Allow serverl-level SMTP server override Allow server-level SMTP server override Oct 9, 2020
@whaught whaught changed the title Allow server-level SMTP server override Allow realms to integrate with an SMTP server Oct 9, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/enhancement New feature or feature request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants