Opened 3 years ago

Closed 3 years ago

#21714 closed Cleanup/optimization (fixed)

Move LazySettings._configure_logging to django.setup

Reported by: Aymeric Augustin Owned by: Claude Paroz
Component: Core (Other) Version: master
Severity: Normal Keywords: app-loading
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Since Django now has an explicit entrypoint, there's no reason to handle logging in the settings anymore.

Change History (7)

comment:1 Changed 3 years ago by Marc Tamlyn

Triage Stage: UnreviewedAccepted

comment:2 Changed 3 years ago by Claude Paroz

Owner: changed from nobody to Claude Paroz
Status: newassigned

comment:4 Changed 3 years ago by Aymeric Augustin

Could you define configure_logging in another module, for example in django.utils.logging?

django.__init__ should only put the pieces together.

comment:5 Changed 3 years ago by Claude Paroz

PR updated.

comment:6 Changed 3 years ago by Aymeric Augustin

Triage Stage: AcceptedReady for checkin

LGTM

comment:7 Changed 3 years ago by Claude Paroz <claude@…>

Resolution: fixed
Status: assignedclosed

In b8e3373f458aa363d59c61b58be390a5a1156467:

Fixed #21714 -- Moved logging configuration to global setup()

Thanks Aymeric Augustin for the report and the review.

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