Hi Jacob, I wouldn't be opposed to move email configuration into a dictionary (somewhere between -0 and +0). Although if we plan to do that we should rethink all the existing session variables and other as well I guess and figure out if we should move more settings to dictionaries.
> why shouldn't it makes sense to have different email backends? If you have a staging system you may want to use you local SMTP-relay, while in production you may for instance use AWSs SES <https://docs.aws.amazon.com/ses/latest/dg/Welcome.html> service. This specific example at hand is imo not a good motivator to add support for multiple backends because the settings would imo be different. Take databases as an example: You also don't have staging/production in there but switch the actual values in the default database. > `EMAIL = [...]` I am not sure a list makes sense here and would go for similarity with CACHES & DATABASES since you'd usually identify the backend via a unique name or so. Also DATABASES & CACHES have an OPTIONS dict which is the passed on to the backend, I think we should follow suit here. > Personally, I would prefer SMTP = {...} I do not think SMTP would be a good fit because many services allow HTTP submission, so what we are sending is actually an email and smtp is just a protocol implementation in the backend of AWS SES or so. As for other email backends that do require different options: I do not see an issue when they simply take `EMAIL_AWS_SES_KEY` and document it as such; this doesn't require us to add more flexibility to email backends… So I guess it boils down to the following questions: * Do we want to support multiple (at the same time) email backends, if yes we would move to a settings dict anyways… * If the answer to the above is no, what value does putting it into a single dict give us? In the past I think I have argued for a SECURITY_HEADERS (or similar) dict because it allows us to check the dictionary keys easily for typos; emails probably don't suffer from that problem as badly as security related settings. I hope this can get the discussion going. Cheers, Florian On Sunday, January 30, 2022 at 9:29:27 AM UTC+1 jacob...@gmail.com wrote: > Well, that ticket is 8 years old and in the meantime other email backends > have emerged, requiring different configuration options. > I made this proposal after attempting to fix a 14 year old open ticket > #6989 but this was ultimately postponed, see comment by > Carlton Gibson on > https://github.com/django/django/pull/13728#issuecomment-987762791 > > To summarize the discussion from 7 years ago > > Collin Anderson wrote: > >> I don't see any benefit to moving email settings to a dictionary. It is >> helpful for databases and caches because there can be multiple backends. > > It makes the popular "from local_settings import *" convention harder to >> use. What's wrong with 6 individual settings? If the goal is to allow > > multiple email backends, then let's make that the ticket goal. > > > and Carl Meyer replied: > >> I agree with Collin; unless we are adding new capabilities (i.e. multiple >> configured email backends, which it seems nobody really wants), it's hard > > to find any actual benefit from this change to justify the churn (and the >> additional complexities of working with dictionary settings in >> partial-override scenarios). > > > why shouldn't it makes sense to have different email backends? If you have > a staging system you may want to use you local SMTP-relay, while in > production > you may for instance use AWSs SES > <https://docs.aws.amazon.com/ses/latest/dg/Welcome.html> service. That > service may require additional configuration settings not available in the > local smtp backend. > I can also imagine that in some situations it may make sense to have two > email backends concurrently. We maybe should rethink about that. > > -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/27c2ed9d-92f4-49e3-b770-a74e37cff368n%40googlegroups.com.
