In d7695e48, a legit caching case was turned
into an error case.

Allowing units is much more convenient and keeping them optional
maintains compatibility. On the other hand, changing the defaults
for default_ttl and default_grace has no impact on the generated
RST.

Keeping track of the user input could help since default values
are "washed" like user input, without resorting to a dynamic
default.

Refs #3756

By themselves, structured fields parsing functions don't fail on extra
input. To avoid repeating the same checks twice, they are wrapped into a
function that will help generalize tweak_generic_double() at least once
more.

This is a first step towards even more generalization.

TCP_KEEPALIVE is the TCP_KEEPIDLE's equivalent and uses seconds
 for the idle time.

This session close reason is also used for receiving response
bodies, for example:

--  TTL            RFC 86400 10 0 1652368707 1652368707 1652368707 0 86400 cacheable
--  VCL_call       BACKEND_RESPONSE
--  VCL_return     deliver
--  Timestamp      Process: 1652368707.157422 0.063369 0.000054
--  Filters
--  Storage        file s0
--  Fetch_Body     2 chunked stream
--  FetchError     chunked read err
--  BackendClose   53 self close Failure receiving req.body
--  BereqAcct      1337 0 1337 987 3465216 3466203
--  End

so #ifdef __FreeBSD__, which is OK, since the intent is just to get
VTEST code coverage.

Closes #3795

if only for consistency.

I will keep an eye on vtest.

This reverts commit cea729bb.

Apparently, clang11 on i386 macOS uses `int` instead of `ssize_t` for
`a- b` where bot `a` and `b` is `char`. It leads to an error:
```
vtc_http.c:392:8: error: format specifies type 'ssize_t' (aka 'long') but the argument has type 'int' [-Werror,-Wformat]
                            p - hp->body, t, *p);
                            ^~~~~~~~~~~~
```

Cookie name and value are separated by equal sign, not colon.
Signed-off-by: Dridi Boukelmoune <dridi.boukelmoune@gmail.com>

I adjusted the docs to reflect the removal of spurious trailing
semi-colon from the generated cookie string.

Closes #3754

The separator between cookies in a cookie header is "; " and we should
not generate it in a way that adds a spurious ';' at the end of the
cookie list.

Refs #3754

We no longer pass coverage flags to cc_command.

This puts them closer to the 'NEXT' marker to lower the risk of
overlooking them in the release process.

This tweaks the workflows in circleci so that the same configuration can
be used both in the regular master workflows and the release branches,
removing a brittle release process step of editing there.

Releasing 7.1.0

from 7615e559 - sorry

This reverts commit 8f11c925.

And while at it, make it easier to work with distribution names with
less if statements and more case matching.

Better diff with the --ignore-all-space option.

Removed one paragraph which was really confusing and, I guess, wrong:
IIUC, backend references _always_ need to be taken with VRT_Assign_Backend,
because the referencing vmod can make _no_ assumptions about the assigned
backend.

Also I hope to have clarified use of VRT_StaticDirector().