Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#25900 closed Bug (fixed)

CommonMiddleware's USE_ETAGS broken

Reported by: Derek J. Curtis Owned by: nobody
Component: Core (Cache system) Version: 1.9
Severity: Normal Keywords: ETags
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


ETag support is broken in CommonMiddleware

This appears to have been broken here:

django.utils.cache.get_conditional_response's etag argument needs to be *unquoted*, however the etag passed by CommonMiddleware's make_response is quoted.

Pull Request with Regression Test:

Change History (4)

comment:1 by Derek J. Curtis, 9 years ago

Type: UncategorizedBug

comment:2 by Tim Graham, 9 years ago

Easy pickings: unset
Triage Stage: UnreviewedReady for checkin

comment:3 by Tim Graham <timograham@…>, 9 years ago

Resolution: fixed
Status: newclosed

In 6be9589e:

Fixed #25900 -- Fixed regression in CommonMiddleware ETag support.

comment:4 by Tim Graham <timograham@…>, 9 years ago

In 364d7d9:

[1.9.x] Fixed #25900 -- Fixed regression in CommonMiddleware ETag support.

Backport of 6be9589eb34f79914666c9d9e1e15bdb7fc44df2 from master

Note: See TracTickets for help on using tickets.
Back to Top