• Nils Goroll's avatar
    Loosen assertion on ctx->(req|bo), fix shard and vcl_pipe · 3dc2baed
    Nils Goroll authored
    in VRT_priv_task() we asserted that only one of ctx->req and ctx->bo is
    set when not in vcl_pipe {}, but we also need to extend that assertion
    to when ctx->method == 0 after vcl_pipe as finished because
    VRT_priv_task() could be called from director resolution.
    
    Being at it, I also noticed that our behavior in vcl_pipe {} is
    inconsistent as, from the shard director perspective, it is a backend
    method. So now, vcl_pipe {} is handled like vcl_backend_* {}.
    
    We still need to make up our mind about #3329 / #3330 and depending on
    the outcome we might need to touch some places again which were changed
    in this commit.
    
    Fixes #3361
    3dc2baed
Name
Last commit
Last update
.circleci Loading commit data...
.github Loading commit data...
bin Loading commit data...
doc Loading commit data...
etc Loading commit data...
include Loading commit data...
lib Loading commit data...
m4 Loading commit data...
man Loading commit data...
tools Loading commit data...
.dir-locals.el Loading commit data...
.envrc Loading commit data...
.gitignore Loading commit data...
.lgtm.yml Loading commit data...
.syntastic_c_config Loading commit data...
.travis.yml Loading commit data...
CONTRIBUTING Loading commit data...
ChangeLog Loading commit data...
INSTALL Loading commit data...
LICENSE Loading commit data...
Makefile.am Loading commit data...
README.Packaging Loading commit data...
README.rst Loading commit data...
autogen.des Loading commit data...
autogen.sh Loading commit data...
configure.ac Loading commit data...
flint.lnt Loading commit data...
varnish-legacy.m4 Loading commit data...
varnish.m4 Loading commit data...
varnishapi-uninstalled.pc.in Loading commit data...
varnishapi.pc.in Loading commit data...
vtc.am Loading commit data...