Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#30316 closed Cleanup/optimization (fixed)

Link to default logging dictConfig in documentation

Reported by: Andrew Badr Owned by: Ahisahar Pretel
Component: Documentation Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description (last modified by Eunwoo Cho)

When people want to customize Django's logging, my guess is that they'll most often want to start by changing something about the default logging setup. Toward this end, it would be helpful if the Django docs included a dictConfig for the default logging setup. Instead, the docs *describe* the default settings (https://docs.djangoproject.com/en/2.1/topics/logging/#default-logging-configuration), and give dictConfigs for several other possible configurations for logging (https://docs.djangoproject.com/en/2.1/topics/logging/#configuring-logging).

Change History (9)

comment:1 by Andrew Badr, 5 years ago

Easy pickings: set
Needs documentation: set

comment:2 by Carlton Gibson, 5 years ago

Component: UncategorizedDocumentation
Triage Stage: UnreviewedAccepted
Type: UncategorizedCleanup/optimization
Version: 2.2master

I don't think copying the full config would be great. There are already lots of examples in the doc for people to use.

However... getting people to look at the source is always worth it, so a single line link to django/utils/log.py"For Reference the default logging config is defined in..." (See a68c029e224cebe540da7447dbbd27993b4aa793 for source extlink examples.) — would be good.

Last edited 5 years ago by Carlton Gibson (previous) (diff)

comment:3 by Carlton Gibson, 5 years ago

Summary: Include default logging dictConfig in documentationLink to default logging dictConfig in documentation

comment:4 by Ahisahar Pretel, 5 years ago

Owner: changed from nobody to Ahisahar Pretel
Status: newassigned

Hi, this is my first easy picking fix. I will assign the issue to myself and submit the PR this evening if that is ok.

comment:5 by Eunwoo Cho, 5 years ago

Description: modified (diff)

comment:6 by Ahisahar Pretel, 5 years ago

Has patch: set

comment:7 by Nick Pope, 5 years ago

Needs documentation: unset

comment:8 by Mariusz Felisiak <felisiak.mariusz@…>, 5 years ago

Resolution: fixed
Status: assignedclosed

In 717362d8:

Fixed #30316 -- Added source code link to the default logging configuration in logging docs.

comment:9 by Mariusz Felisiak <felisiak.mariusz@…>, 5 years ago

In f6febbc0:

[2.2.x] Fixed #30316 -- Added source code link to the default logging configuration in logging docs.

Backport of 717362d810955b9514e2ccd989f8a1647c9d7a00 from master

Note: See TracTickets for help on using tickets.
Back to Top