Opened 2 years ago

Closed 2 years ago

Last modified 12 months ago

#20418 closed Bug (fixed)

django.contrib.sessions not needed when using cookie-session-backend

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

Description

The documentation says nothing about that you don't need the app django.contrib.sessions when using cookie-session-backend. The docs says only that you need it in some cases, and that you can remove it if you don't need sessions at all.

Change History (8)

comment:1 Changed 2 years ago by charettes

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

Related to #7836.

comment:2 Changed 2 years ago by russellm

  • Needs documentation set
  • Triage Stage changed from Unreviewed to Accepted

comment:3 Changed 2 years ago by nmundar

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

comment:4 Changed 2 years ago by nmundar

Documentation update is available from this branch: https://github.com/nmundar/django/tree/ticket_20418

Cookies-based backend keeps all the data in the cookie so it does not need to access the database. Because of that there is no need for Session table in the db that gets created when django goes through INSTALLED_APPS settings.

Last edited 2 years ago by nmundar (previous) (diff)

comment:5 Changed 2 years ago by timo

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

Merged in [d9c01da1]

In the future, please following the commit message guidelines ("Fixed #XXXXXX -- Commit description") so that the ticket will automatically be closed when the pull request is merged.
https://docs.djangoproject.com/en/dev/internals/contributing/committing-code/#committing-guidelines

comment:6 Changed 12 months ago by Tim Graham <timograham@…>

In 3a85aae2eacae836e2a92de77615569883b81932:

Revert "Improve cookie based session backend docs." refs #20418

This reverts commit d9c01da1f8ba3d4e0947fd1d619c8f4a9b013360.

This is poor advice as it breaks the test client login (refs #22934).
We can add a note like this back after refs #22986 is resolved.

comment:7 Changed 12 months ago by Tim Graham <timograham@…>

In a3d710a38f1fab5e092cef7a17826674a00e70ea:

[1.7.x] Revert "Improve cookie based session backend docs." refs #20418

This reverts commit d9c01da1f8ba3d4e0947fd1d619c8f4a9b013360.

This is poor advice as it breaks the test client login (refs #22934).
We can add a note like this back after refs #22986 is resolved.

Backport of 3a85aae2ea from master

comment:8 Changed 12 months ago by Tim Graham <timograham@…>

In f3bdb83ba58a696c0696d8a6706d2c44a00da68a:

[1.6.x] Revert "Improve cookie based session backend docs." refs #20418

This reverts commit d9c01da1f8ba3d4e0947fd1d619c8f4a9b013360.

This is poor advice as it breaks the test client login (refs #22934).
We can add a note like this back after refs #22986 is resolved.

Backport of 3a85aae2ea from master

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