#12789: ConditionalGetMiddleware behavior improvement.
---------------------------------------------+------------------------------
Reporter: penzoil | Owner: nobody
Status: new | Milestone:
Component: Cache system | Version: SVN
Resolution: | Keywords: ETag,
ConditionalGetMiddleware
Stage: Design decision needed | Has_patch: 1
Needs_docs: 1 | Needs_tests: 1
Needs_better_patch: 1 |
---------------------------------------------+------------------------------
Comment (by forest):
Hi,
The problem is that patch_response_headers sets an ETag header regardless
of the response status_code. This differs from the behavior implemented
by CommonMiddleware, which only sets ETags when 200 <=
response.status_code < 300. Making it consistent with CommonMiddleware
behavior fixes things for me.
I'll attach a new patch.
Thanks,
Forest
--
Ticket URL: <http://code.djangoproject.com/ticket/12789#comment:3>
Django <http://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 this group at
http://groups.google.com/group/django-updates?hl=en.
