#15855: cache_page decorator bypasses any Vary headers set in middleware
-------------------------------------+-------------------------------------
               Reporter:  carljm     |          Owner:  nobody
                   Type:  Bug        |         Status:  new
              Milestone:             |      Component:  Core (Cache system)
                Version:             |       Severity:  Normal
             Resolution:             |       Keywords:
           Triage Stage:  Design     |      Has patch:  1
  decision needed                    |    Needs tests:  0
    Needs documentation:  0          |  Easy pickings:  0
Patch needs improvement:  0          |
                  UI/UX:  0          |
-------------------------------------+-------------------------------------

Comment (by carljm):

 Verified that Luke's fix works on both 1.3.X and trunk. In fact,
 vary_on_cookie isn't even needed, since csrf_protect patches the vary
 header -- the docs can just replace vary_on_cookie with csrf_protect.

 Went a little ways down the path of an automated test to verify that this
 works, but since we have tests for all the component pieces already, the
 test ends up just being a verification that Python does indeed apply
 decorators in the expected order. Which is kind of a stupid thing to test.

 Applying the doc fix to 1.3.X and trunk.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/15855#comment:9>
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 django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to