Opened 5 years ago

Closed 5 years ago

#17236 closed Cleanup/optimization (fixed)

Documentation of session behavior during login

Reported by: Byron Ruth Owned by: Christopher Medrela
Component: Documentation Version:
Severity: Normal Keywords:
Cc: krzysiumed@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

Attachments (2)

patch.diff (592 bytes) - added by Christopher Medrela 5 years ago.
17236_v2.diff (1.1 KB) - added by Christopher Medrela 5 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 5 years ago by Paul McMillan

Triage Stage: UnreviewedAccepted

The docs need to point out that when going from an anonymous session to a logged in session, data set during the anonymous session is retained.

The first bullet point in the email is incorrect - the flush code does delete the session, causing a new session id to be created.
https://github.com/django/django/blob/master/django/contrib/sessions/backends/base.py#L240-257

Last edited 5 years ago by Paul McMillan (previous) (diff)

comment:2 Changed 5 years ago by Christopher Medrela

Owner: changed from nobody to Christopher Medrela
Status: newassigned

Changed 5 years ago by Christopher Medrela

Attachment: patch.diff added

comment:3 Changed 5 years ago by Christopher Medrela

Cc: krzysiumed@… added
Has patch: set
Type: UncategorizedCleanup/optimization

comment:4 Changed 5 years ago by Preston Holmes

Patch needs improvement: set

"Note that when logging in session," the grammar is off, not sure what exactly you meant

Also this is a change to a docstring, and Django's docs are not autogenerated, so docs should also be updated in the source files of the sphinx documentation.

Changed 5 years ago by Christopher Medrela

Attachment: 17236_v2.diff added

comment:5 Changed 5 years ago by Christopher Medrela

Cc: krzysiumed@… removed
Patch needs improvement: unset

OK, I amended the patch: the grammar was improved and a note in sphinx docs was added.

comment:6 Changed 5 years ago by Christopher Medrela

Cc: krzysiumed@… added

comment:7 Changed 5 years ago by Jannis Leidel

Triage Stage: AcceptedReady for checkin

comment:8 Changed 5 years ago by Claude Paroz

Resolution: fixed
Status: assignedclosed

In [17704]:

Fixed #17236 -- Clarified that anonymous session data are retained after login. Thanks krzysiumed for the patch.

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