[Varnish] #747: 302 is unconditionally cached by default

[Varnish]

#747: 302 is unconditionally cached by default
----------------------+-----------------------------------------------------
 Reporter:  trombik   |       Owner:  phk  
     Type:  defect    |      Status:  new  
 Priority:  normal    |   Milestone:       
Component:  varnishd  |     Version:  2.1.2
 Severity:  normal    |    Keywords:       
----------------------+-----------------------------------------------------
 rfc2616 states that "A response received with any other status code (e.g.
 status codes 302 and 307) MUST NOT be returned in a reply to a subsequent
 request unless there are cache-control directives or another header(s)
 that explicitly allow it"

 however, bin/varnishd/cache_center.c caches 302 response without such
 headers. an example is:

 {{{
 Connection:Keep-Alive
 Content-Length:17
 Content-Type:text/plain
 Date:Wed, 04 Aug 2010 10:10:08 GMT
 Keep-Alive:timeout=5, max=50
 Location:http://foo.example.org/bar/
 }}}

 the commit log in question simply says "handle 302 for now".

 http://varnish-cache.org/changeset/497

-- 
Ticket URL: <http://www.varnish-cache.org/ticket/747>
Varnish <http://varnish-cache.org/>
The Varnish HTTP Accelerator