[Varnish] #1113: Backend HTTP status is used when hitting max_restarts after fetch

[Varnish]

#1113: Backend HTTP status is used when hitting max_restarts after fetch
----------------------+-----------------------------------------------------
 Reporter:  scoof     |        Owner:  martin
     Type:  defect    |       Status:  closed
 Priority:  normal    |    Milestone:        
Component:  varnishd  |      Version:  trunk 
 Severity:  normal    |   Resolution:  fixed 
 Keywords:            |  
----------------------+-----------------------------------------------------
Changes (by Poul-Henning Kamp <phk@…>):

  * status:  new => closed
  * resolution:  => fixed


Comment:

 (In [480c0a5d0b7b11e8e6b66f90dd66a2d8b22d235f]) Go over the
 return(restart) code.

 This is vastly inspired by #1113 and scoof+martins testcase+patch.

 Add a new cnt* state "cnt_restart" to do the mandatory restart work,
 and to move the restart limit check out of vcl_recv{}, getting rid
 of a nasty XXX comment.

 Set the explicit 503 when over limit, reset in other cases.

 Expand scoof+martins test-case to cover more vcl methods, which
 were also affected, but not fixed in their patch.

 Fixes   #1113

-- 
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1113#comment:2>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator