#18978: Move cleanup management command to contrib.sessions
-------------------------------------+-------------------------------------
Reporter: rasca | Owner: rasca
Type: | Status: new
Cleanup/optimization | Version: master
Component: Core (Management | Resolution:
commands) | Triage Stage: Accepted
Severity: Normal | Needs documentation: 0
Keywords: | Patch needs improvement: 0
Has patch: 1 | UI/UX: 0
Needs tests: 1 |
Easy pickings: 1 |
-------------------------------------+-------------------------------------
Comment (by Elvard):
Hi crodjer,
you've included only
https://code.djangoproject.com/attachment/ticket/18194/sessions-cleanup-
files-1.patch which is not enough as I mentioned before:
> Alright, I've found, that file-based sessions doesn't support deletion
of expired files and patch above doesn't provided it either. It is in
another patch from #18194,
https://code.djangoproject.com/attachment/ticket/18194/files-sessions-
server-expiry-3.patch. However, I can't agree with it's implementation.
See related ticket for more details.
I've already applied that patch to this commit
https://github.com/elvard/django/commit/969822aa8b51af7840d5809b649101b291d26623
As I said, it doesn't work, since file-based sessions doesn't have any
"expiration" mechanism. It's included in the other patch, but I can't
agree with it's implementation, so I opened discussion at #18194.
--
Ticket URL: <https://code.djangoproject.com/ticket/18978#comment:14>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit https://groups.google.com/groups/opt_out.
