Opened 3 years ago

Closed 3 years ago

Last modified 2 years ago

#20418 closed Bug (fixed)

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

Reported by: ivan@… Owned by: Neven Munđar
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 3 years ago by Simon Charette

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

Related to #7836.

comment:2 Changed 3 years ago by Russell Keith-Magee

Needs documentation: set
Triage Stage: UnreviewedAccepted

comment:3 Changed 3 years ago by Neven Munđar

Owner: changed from nobody to Neven Munđar
Status: newassigned

comment:4 Changed 3 years ago by Neven Munđar

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 3 years ago by Neven Munđar (previous) (diff)

comment:5 Changed 3 years ago by Tim Graham

Resolution: fixed
Status: assignedclosed

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 2 years 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 2 years 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 2 years 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