varnishcache-friends / varnish3to4

Varnish 3 to 4 migration script
BSD 2-Clause "Simplified" License
125 stars 25 forks source link

Error handling could be improved #4

Closed huayra closed 7 years ago

huayra commented 7 years ago

When translating "error" from 3 to 4 is kind of broken. It works fine as long as you do not something like the URL or the Hostname. I.e. in vcl_recv:

but, this example does not work (req.http.host would not work either):

as we should expect:

We understand this as a syntax error that can easily be fixed. Would you welcome such an addition?

fgsch commented 7 years ago

I thought I fixed this. 😞

Feel free to submit a PR if it's still happening.

huayra commented 7 years ago

PR: https://github.com/fgsch/varnish3to4/pull/7