Reported at #2644146-9: BigPipe Varnish documentation: Surrogate-Control header value's future evolution? by @bugsonly:

fyi,

for current Varnish v4, vcl_fetch() has been replaced by vcl_backend_response(), cref: https://www.varnish-cache.org/docs/4.0/whats-new/upgrading.html#vcl-fetc...

and,

    set beresp.ttl = 0;

throws a VCL compile error

Jan 18 11:18:04 test.loc varnishd[11250]: Error:
Jan 18 11:18:04 test.loc varnishd[11250]: Message from VCC-compiler:
Jan 18 11:18:04 test.loc varnishd[11250]: Expected ID got ';'
Jan 18 11:18:04 test.loc varnishd[11250]: (program line 166), at
Jan 18 11:18:04 test.loc varnishd[11250]: ('input' Line 437 Pos 35)
Jan 18 11:18:04 test.loc varnishd[11250]: set beresp.ttl = 0;
Jan 18 11:18:04 test.loc varnishd[11250]: ----------------------------------#
Jan 18 11:18:04 test.loc varnishd[11250]: Running VCC-compiler failed, exited with 2
Jan 18 11:18:04 test.loc varnishd[11250]: VCL compilation failed

should be, instead,

    set beresp.ttl = 0s;
CommentFileSizeAuthor
#2 2652692-2.patch629 byteswim leers

Comments

Wim Leers created an issue. See original summary.

wim leers’s picture

Status: Active » Needs review
StatusFileSize
new629 bytes
wim leers’s picture

Assigned: Unassigned » fabianx

I'd like a +1 from Fabian.

fabianx’s picture

Status: Needs review » Reviewed & tested by the community

RTBC - looks good to me, but did not test it.

  • Wim Leers committed b00dc84 on 8.x-1.x
    Issue #2652692 by Wim Leers, bugsonly: BigPipe's Varnish documentation...
wim leers’s picture

Status: Reviewed & tested by the community » Fixed

Fixed and committed. Credited @bugsonly in the commit, gave him issue credit on the original issue: #2644146-12: BigPipe Varnish documentation: Surrogate-Control header value's future evolution?.

wim leers’s picture

Category: Bug report » Task

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.