#27328: return `Set-Cookie` if sessionid= None value
-------------------------------------+-------------------------------------
Reporter: Ramin Farajpour | Owner: nobody
Cami |
Type: Bug | Status: new
Component: HTTP handling | Version: 1.10
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Ramin Farajpour Cami):
> Even if we were to change Django to not delete empty sessionid cookies,
that should be a change to the _session_ code (to not call delete_cookie
in that case), not a change to the cookie parsing code. Does that seem
right?
Hi Collin,
you are right,
> and then place it before SessionMiddleware:
{{{
class RemoveEmptyCookiesMiddleware:
def process_request(request):
request.COOKIES = {k: v for k, v in request.COOKIES.items() if k
and v}
}}}
i'm confused , we have here two `for` loop `1- process_request()` `2-
parse_cookie()`, what happen for performance and speed !!!???
so, do you think eligible this method(`RemoveEmptyCookiesMiddleware`)
merged to django master?
--
Ticket URL: <https://code.djangoproject.com/ticket/27328#comment:13>
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 unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/065.d46320c45838455b68ba13404deece02%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
