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

EmailBackend cannot be changed via configuration variable #5375

Closed
Moist-Cat opened this issue Jun 20, 2022 · 3 comments · Fixed by #5376
Closed

EmailBackend cannot be changed via configuration variable #5375

Moist-Cat opened this issue Jun 20, 2022 · 3 comments · Fixed by #5376
Milestone

Comments

@Moist-Cat
Copy link
Contributor

Is your feature request related to a problem? Please describe.
During development is easier to make tests sending the emails to stdout, instead of using a real SMTP server so it would be beneficial to be able to change it.

Describe the solution you'd like
Add a configuration variable called EMAIL_BACKEND which defaults to indico.vendor.django_mail.backends.smtp.EmailBackend.
Is really a small change; I already have it working in my fork.

@ThiefMaster
Copy link
Member

Any reason against using maildump as a "real" SMTP server as suggested in the dev setup docs? This is much more convenient to read the emails sent to it than just dumping raw mail content to stdout.

@Moist-Cat
Copy link
Contributor Author

Yeah, but that would add aditional dependiencies and I would have to read even more docs before starting to make changes which have nothing to do with emails. So I thought it was overkill to use yet another tool when I could just dump them in console for now like I would do with Django.

@ThiefMaster
Copy link
Member

It's as simple as pipx install maildump (which installs it in an isolated virtualenv) and starting it :)

Anyway, feel free to send a PR, it's not something I plan to implement but if there's a good PR for it I won't be opposed to merging it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants