Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  nschagen
 Type:  Cleanup/optimization  |   Status:  closed
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:  invalid
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+

Comment (by peterlauri):

 You don't need to share your config, I get what you mean, had to read it
 again :)

 But still it remains unclear why the above LOGGING conf breaks the email
 sending.

 In my test case I use the in memory mail backend, and mails are not being
 sent anymore using the above configuration.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.f99951b6f895fffcaeca759c192c4568%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  nschagen
 Type:  Cleanup/optimization  |   Status:  closed
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:  invalid
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+

Comment (by peterlauri):

 Replying to [comment:9 nschagen]:
 > According to django/conf/global_settings.py, LOGGING is an empty
 dictionary by default. There is also a DEFAULT_LOGGING in
 django/utils/log.py, which is where the default admin emailing behaviour
 is defined. DEFAULT_LOGGING and LOGGING are merged and therefore setting
 your own LOGGING conf does not break the email to admin feature.
 >
 > I have tested this by installing a FileHandler in both DEFAULT_LOGGING
 and LOGGING and the message was written to both files.

 Yes, I have read that. And I was not expecting admin email behaviour to
 break. But the below configuration breaks the admin email functionality, I
 have a test case where DEBUG=False and emails stop being sent. Removing
 the LOGGING completely OR re-adding makes it work again.

 Can you share your LOGGING config?

 {{{
 LOGGING = {
 'version': 1,
 'disable_existing_loggers': False,
 'formatters': {
 'verbose': {
 'format': '%(levelname)s %(asctime)s %(module)s %(process)d
 %(thread)d %(message)s'
 },
 'simple': {
 'format': '%(levelname)s %(message)s'
 },
 },
 'handlers': {
 'console': {
 'class': 'logging.StreamHandler',
 'stream': sys.stdout,
 'formatter': 'simple'
 },
 },
 'loggers': {
 'django': {
 'handlers': ['console'],
 'level': 'INFO',
 'propagate': True,
 },
 }
 }
 }}}

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.3024aac50abc490d19d285a37ada0b57%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  nschagen
 Type:  Cleanup/optimization  |   Status:  closed
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:  invalid
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by nschagen):

 * status:  assigned => closed
 * has_patch:  1 => 0
 * resolution:   => invalid


Comment:

 According to django/conf/global_settings.py, LOGGING is an empty
 dictionary by default. There is also a DEFAULT_LOGGING in
 django/utils/log.py, which is where the default admin emailing behaviour
 is defined. DEFAULT_LOGGING and LOGGING are merged and therefore setting
 your own LOGGING conf does not break the email to admin feature.

 I have tested this by installing a FileHandler in both DEFAULT_LOGGING and
 LOGGING and the message was written to both files.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.21eee053e6239d995bebc24f0b85691a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  nschagen
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by nschagen):

 * has_patch:  0 => 1


Comment:

 https://github.com/django/django/pull/4268

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.466f9102afe1535ef85bb8e5a30c770b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  nschagen
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by nschagen):

 * owner:   => nschagen
 * status:  new => assigned


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.9c02ef312ffc7a50184f950a62d38619%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by gmunumel):

 * owner:  gmunumel =>
 * status:  assigned => new


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.717c3f9985f63453dbcd7936acfa6648%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  gmunumel
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+

Comment (by peterlauri):

 Maybe include a direct link to the AdminEmailHandler as well as complement
 to the reference to the logging configuration?

 
https://docs.djangoproject.com/en/1.7/topics/logging/#django.utils.log.AdminEmailHandler

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.8e83c6215ae3c474e6752b8bbd9d2821%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  gmunumel
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+

Comment (by pnovotnak):

 Does "0001-updated-error-reporting-documentation-as-per-ticket-.patch"
 work?

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.38e8dda8929e46af6359a4b33a87a21f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  gmunumel
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by gmunumel):

 * owner:  nobody => gmunumel
 * status:  new => assigned


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.bbee434b78819c671e152084a93b697d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
--+
 Reporter:  peterlauri|Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.7
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by timgraham):

 * type:  Uncategorized => Cleanup/optimization


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.52d504f0beed294add62c9e2f198b425%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
---+
 Reporter:  peterlauri |Owner:  nobody
 Type:  Uncategorized  |   Status:  new
Component:  Documentation  |  Version:  1.7
 Severity:  Normal |   Resolution:
 Keywords: | Triage Stage:  Accepted
Has patch:  0  |  Needs documentation:  0
  Needs tests:  0  |  Patch needs improvement:  0
Easy pickings:  1  |UI/UX:  0
---+
Changes (by collinanderson):

 * needs_docs:   => 0
 * needs_better_patch:   => 0
 * needs_tests:   => 0
 * stage:  Unreviewed => Accepted


Comment:

 I could see slightly improved wording being helpful, though it would be
 nice to not increase the amount of text too much.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/068.5cdb3e1345cf5d4d6b0c1360a4203b5a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #23784: Error reporting docs needs to state clearly that changing logging conf error emails are not sent out unless configuring it again

[Django]

#23784: Error reporting docs needs to state clearly that changing logging conf
error emails are not sent out unless configuring it again
---+
 Reporter:  peterlauri |  Owner:  nobody
 Type:  Uncategorized  | Status:  new
Component:  Documentation  |Version:  1.7
 Severity:  Normal |   Keywords:
 Triage Stage:  Unreviewed |  Has patch:  0
Easy pickings:  1  |  UI/UX:  0
---+
 Related documentation: https://docs.djangoproject.com/en/1.7/howto/error-
 reporting/

 It states '''To activate this behavior, put the email addresses of the
 recipients in the ADMINS setting.''' and in the following '''See also'''
 it states that this behavior can be customized by changing logging.

 The problem here is that a novice user will not easily catch that they
 break this behavior if they add their own LOGGING into settings.py without
 also adding the AdminEmailHandler handler and corresponding logger.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/053.931fd5b32cc5130962b7520adadf9bf8%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.