Skip to content

Sessions expiry related fixes#453

Closed
crodjer wants to merge 6 commits intodjango:masterfrom
crodjer:sessions-expiry
Closed

Sessions expiry related fixes#453
crodjer wants to merge 6 commits intodjango:masterfrom
crodjer:sessions-expiry

Conversation

@crodjer
Copy link
Copy Markdown

@crodjer crodjer commented Oct 20, 2012

  • Sessions expiry check on server side
  • Use signing framework for decoding/encoding sessions data
  • Bigger session key character set
  • Attempt compatibility fixes for moving to siging based session storage

Related issue: #18194

Use timed signer to check for expiration of session data. This is to
fix ticket django#18194. The sessions based on file backend otherwise do not
expire, as far as the server is concerned.

Signed-off-by: Rohan Jain <[email protected]>

Conflicts:
	django/contrib/sessions/backends/base.py
The existing sessions, which were not signed with the signing
framework is handled with the older decoding method.
Mark the session as modified so that it uses the new encoding method
for storing the data.

Signed-off-by: Rohan Jain <[email protected]>

Conflicts:
	django/contrib/sessions/backends/base.py
Don't enable compatibility with older mechanism by default as it
compromises with the security benefits of introducing signing
framework.

Signed-off-by: Rohan Jain <[email protected]>
@crodjer crodjer closed this Nov 16, 2012
@crodjer
Copy link
Copy Markdown
Author

crodjer commented Dec 31, 2025

Is this relevant now? The changes are from 13+ year ago.

@django django deleted a comment from casey-brooks Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant