Code

#19697 closed New feature (fixed)

Add an overview of good practices for production settings

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

Description

Quoting Claude:

add a section in the docs where we explain what settings should be typically added in deployed projects, but not suitable in public repositories (DEBUG=False, DATABASES, STATIC_ROOT, SECRET_KEY, ADMINS, SERVER_EMAIL, etc.) and link to this section in the template.

Attachments (0)

Change History (8)

comment:1 Changed 15 months ago by aaugustin

  • Triage Stage changed from Unreviewed to Accepted

This was discussed in the context of the new project template. A link to that document should be added below the comment that reads: "# Quick-start development settings - unsuitable for production".

comment:2 Changed 14 months ago by carljm

Lukasz Langa mentioned in IRC today that it would be good to document that most production sites probably want to use the caching template loader. This isn't exactly something that is "not suitable in public repositories", but it fits in a slightly broader document about "things you probably want to do when you go to production." Using the caching loader in development is painful because you don't get template updates until a reload, but almost all sites probably should be using it in production.

comment:3 Changed 14 months ago by carljm

Also, ALLOWED_HOSTS definitely belongs here; see #19875

comment:4 Changed 14 months ago by aaugustin

This is an excellent start, thank you!

A few suggestions:

  • group settings by category: 1) for security 2) for performance 3) for error reporting -- add more if necessary
  • look for other production-related settings in the settings list
Version 0, edited 14 months ago by aaugustin (next)

comment:5 Changed 14 months ago by aaugustin

Pull request re-sent together here: https://github.com/django/django/pull/839

comment:6 Changed 13 months ago by aaugustin

  • Owner changed from nobody to aaugustin
  • Status changed from new to assigned

comment:7 Changed 13 months ago by aaugustin

  • Has patch set

comment:8 Changed 13 months ago by Aymeric Augustin <aymeric.augustin@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

In 912b5d2a6bc78067d6a7e130f10514c51bd1a58f:

Fixed #19697 -- Added a deployment checklist.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.