Opened 13 years ago

Closed 13 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 13 years ago.
17236_v2.diff (1.1 KB ) - added by Christopher Medrela 13 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 by Paul McMillan, 13 years ago

Triage Stage: UnreviewedAccepted

The docs need to point out that while 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

Version 1, edited 13 years ago by Paul McMillan (previous) (next) (diff)

comment:2 by Christopher Medrela, 13 years ago

Owner: changed from nobody to Christopher Medrela
Status: newassigned

by Christopher Medrela, 13 years ago

Attachment: patch.diff added

comment:3 by Christopher Medrela, 13 years ago

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

comment:4 by Preston Holmes, 13 years ago

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.

by Christopher Medrela, 13 years ago

Attachment: 17236_v2.diff added

comment:5 by Christopher Medrela, 13 years ago

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 by Christopher Medrela, 13 years ago

Cc: krzysiumed@… added

comment:7 by Jannis Leidel, 13 years ago

Triage Stage: AcceptedReady for checkin

comment:8 by Claude Paroz, 13 years ago

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