id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
33624	Only disable console logging when ADMINS is set	Alex Dehnert		"The current [https://docs.djangoproject.com/en/4.0/ref/logging/#default-logging-conditions logging defaults] are:
* If DEBUG=True, log to console
* If DEBUG=False, use AdminEmailHandler

In practice, I think this means if you set DEBUG=False and don't touch anything else, errors just vanish -- ADMINS defaults to [], so the emails don't actually go anywhere. The stock settings.py doesn't even mention ADMINS as something you might want to set, so this mistake seems very easy to make. (Also, empirically I made it.)

Perhaps the logging defaults should be:
* If DEBUG=True or ADMINS is empty, log to console
* If DEBUG=False and ADMINS is non-empty, send emails

(Alternatively, ADMINS could be added to the settings.py template, and perhaps a [https://docs.djangoproject.com/en/4.0/topics/checks/ system check] could be added to verify it's populated?)

(I suspect the patch here is easy, but it seems like it requires a ~design decision that it's desirable.)"	Uncategorized	closed	Error reporting	4.0	Normal	invalid			Unreviewed	0	0	0	0	0	0