This has the unfortunate effect of splitting a single switch statement
into two, however the second one being based on an enum should yield a
warning if in the future a new binding is added but not handled.

For example a binding involving the <h> key?

For starters the include table will be used for two things: handle key
presses in interactive mode and generate the "Key bindings" manual
section. The table is so far built to satisfy those two requirements.

It should then be repurposed for a help screen in interactive mode and
possibly refined to accommodate this third use case.

Refs #2990

They have the same effect in practice despite looking different, so the
simpler option is kept. Also considering vi key bindings <j> and <k>
documented in the previous commit, <g> and <G> would definitely have the
same meaning as <HOME> and <END> in this context.

And avoid magic ASCII numbers when we could use character literals
instead.

mgt_cli_askchild() as opposed to directly through mcf_askchild()),
a truncated CLI response is not a failure.

Fixes #3038

Refs 45af9764

I actually did consider this before, but thought we should keep it
because we are currently dicussing to add a CTX to vmod object
destructors. But this piece of code is unrelated and at any rate the
argument can be added back when we need it.

in short: @Dridi is right

VCC generates a call to VPI_vcl_select() before returning to the fsm. We
basically conducted half a rollback in VPI_vcl_slect and a rull rollback
in the fsm.

De-duplicate this dance and assert in the fsm that our request looks
rolled back.

... which is the vcl the topreq started on

It is unclear to me what the original intention was, but at any rate,
the privs are not vcl-specific and the vcl argument was not used.

The motivation for this change is the follow up commit
"cleanup implicit rollback for return(vcl(...))": For this commit, it
would be unclear which vcl to roll back with, but as it actually does
not matter, I think this change is warranted.

As the FSM will fail for this case, we should not run the code in the
first place.

Multiple calls to the synthetic() function result in the accumulation of
all body parts. The ability to set a [be]resp body accidentally resulted
in the same behavior.

In other words writing `set resp.body = "string";` in VCL behaves as if
`set resp.body += "string";` was written instead. This patch introduces
an enum to distinguish between the two desired actions and enables both
syntaxes without touching the behavior of the synthetic() function.

And avoid the indentation of characters in vcc_assign_expr().

Fixes #3079
Closes #3100

At this point we should probably consider renaming the arithmetic table
to something also more general.

Refs #3100

It uses a similar trick as VCC expressions to find where the symbol name
belongs.

Refs #3100

First, gather everything needed, then generate the C code. This avoids a
bit of code duplication and makes the error handling more natural, right
after the attempt at generating a matching expression.

Refs #3100

    Avoid VSB_printf for static strings

    Done with the following semantic patch for Coccinelle:

        @@
        expression vsb, fmt;
        @@

        - VSB_printf(vsb, fmt);
        + VSB_cat(vsb, fmt);

    This patch is available in the Varnish source tree.

    vtc_haproxy: enable mcli with a separate flag from worker mode

    Support for the master cli in worker mode was added with commit 86e65f1
    using the same flag as worker mode (-W). This is a problem with haproxy
    1.8 because it has worker mode but no mcli. This patch adds -S to enable
    mcli for haproxy in a vtc.

and:

    The two patches in attachment fix a bug in the haproxy CLI handling and add
    the support for the haproxy master CLI in VTest. Could you merge them? Thanks
    You can also find in attachment a vtc file which use this new feature.

    Submitted by:   William Lallemand <wlallemand@haproxy.com>

enum gethdr_e and VCL_HTTP have a 1:1 relation except for HDR_OBJ which
does not have a corresponding VCL_HTTP pointer.

As I am about to use a "reverse VRT_selecthttp()" to get from VCL_HTTP
to enum gethdr_e, I would want to guard against accidental introduction
of a regression.

Ref #3145 / 287dc4a6

Ref #3145 / 287dc4a6

Fixes typo in storage backends guide.

There's only one place where a symbol may be assigned the SYM_OBJECT
kind and it is ensured that the JSON descriptor is found and added to
the symbol.

Patch by @ehocdet, commit message edited by @nigoroll:

The root cause of #3131 was misdiagnosed to the extent that, while this
change had prevented it, the root cause was a bug in WS_ReserveSize()
fixed in 505b7bd9

The previous tlv_string() code was correct except for the
fact that error handling should have checked for WS_ReserveSize(ctx->ws,
len+1) <= len (also spotted by @ehocdet).

Someone had mentioned at some point that we would not want to VRT_fail(),
but I think this must have been related to the proxy transport code, not
the proxy vmod.

Ref #3131

Notes:

* for the acceptor, I think it makes sense to keep AN assertion (pun!)
  because varnish is not viable if the session workspace is too small
  to even hold the attributes initialized in the acceptor.

  If this was an issue, we should rather revisit the minimum values for
  the session workspace

* for h1 and h2 session setup, I have used XXXAN() because I am not sure
  how we should best handle allocation failures.

* The relevant bit, for now, is the proxy code which may allocate
  arbitrarily long TLV attributes, so this is the code for which we now
  actually handle errors and test that we do

On the vtc: I added the test to o00005.vtc because there existed a
previous overflow test from 267504b8,
but that only tested for the one case of a WS overflow which was already
handled.

Fixes #3145

This originates from a3d47c25, but
was overlooked in 4e333597:

When there is insufficient space to fulfil the reservation request, we
must not leave the workspace reserved.

Fixes #3131