[Varnish] #961: Parameter parsing bug in esi include.

[Varnish]

#961: Parameter parsing bug in esi include.
-----------------------+----------------------------------------------------
 Reporter:  dharrigan  |        Type:  defect
   Status:  new        |    Priority:  high  
Milestone:             |   Component:  build 
  Version:  3.0.0      |    Severity:  major 
 Keywords:  esi        |  
-----------------------+----------------------------------------------------
 Hello,

 We are using esi includes to replace content in our pages. We seem to have
 come across a bug when the esi include (src) contains parameters:

 i.e., we have three esi tags on one page:

 {{{
 <esi:include onerror="continue" maxwait="15000"
 src="/article/6573/style/1?position=0"></esi:include>

 <esi:include onerror="continue" maxwait="15000"
 src="/article/6573/style/2?position=1&headline=head%20line%202&previewText=preview%20text%202&showPreviewLinks=Y"></esi:include>

 <esi:include onerror="continue" maxwait="15000"
 src="/article/6573/style/3?position=2&headline=headline3&previewText=previewtext3"></esi:include>
 }}}

 When I tail the varnishlog using this command:

 {{{
 varnishlog -i TxUrl
 }}}

 I see this in the log:

 {{{
 13 BackendOpen  b default 127.0.0.1 59363 127.0.0.1 8080
 13 TxURL        b /article/6573/style/1?position=0
 13 BackendReuse b default
 13 BackendClose b default
 13 BackendOpen  b default 127.0.0.1 59365 127.0.0.1 8080
 13 TxURL        b /article/6573/style/2?position=1&e/1?position=0
 13 BackendReuse b default
 13 BackendClose b default
 13 BackendOpen  b default 127.0.0.1 59367 127.0.0.1 8080
 13 TxURL        b /article/6573/style/3?position=2&s=Y
 }}}

 This breaks our functionality of passing parameters back into the system.

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