• 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
..
cache Loading commit data...
common Loading commit data...
fuzzers Loading commit data...
hash Loading commit data...
hpack Loading commit data...
http1 Loading commit data...
http2 Loading commit data...
mgt Loading commit data...
proxy Loading commit data...
storage Loading commit data...
waiter Loading commit data...
Makefile.am Loading commit data...
VSC_lck.vsc Loading commit data...
VSC_main.vsc Loading commit data...
VSC_mempool.vsc Loading commit data...
VSC_mgt.vsc Loading commit data...
VSC_sma.vsc Loading commit data...
VSC_smf.vsc Loading commit data...
VSC_smu.vsc Loading commit data...
VSC_vbe.vsc Loading commit data...
builtin.vcl Loading commit data...
flint.lnt Loading commit data...
flint.sh Loading commit data...
vclflint.lnt Loading commit data...
vclflint.sh Loading commit data...