Opened 23 months ago

Closed 23 months ago

Last modified 9 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 23 months ago by charettes

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

Related to #7836.

comment:2 Changed 23 months ago by russellm

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

comment:3 Changed 23 months ago by nmundar

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

comment:4 Changed 23 months 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 23 months ago by nmundar (previous) (diff)

comment:5 Changed 23 months 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 9 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 9 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 9 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