comment out checks until we have fully nested VPs

fix "can contain" for structs && key fields

make TACACS+ PAP CHAP. and (maybe) MS-CHAP work

now that the namespaces are parsed correctly for subrequests,
we can make more things work automagically

TACACS+ is TCP, and we don't do dedup checking

if there's no Auth-Type, use TACACS.Packet.Authen-Type

set inst->server_cs from parent, not from finding any server

no need to use "-Reply-" in all of the reply names

allow parsing of namespace

add function to check for "da can contain other da"

really for fr_pair_t validation, but it's still useful to have a
sanity check function in the dictionary code.

json: Fix typo... Don't mis-cast integers

rest: Always emit data

Don't crash when the first component is a HTTP method

just set "*leftover" always

no need to check for conditions, because the previous check
for "smaller than one packet" takes care of what we need.

set leftover to 0 when we have a full packet

don't change tmpl_rules_t parent

that's used for changing dictionaries, and nothing else tracks
or uses it.

use "namespace" instead of "parent".

There are too many things called "parent" already

tacacs: Clear leftover when we decode a complete packet and no data is left in the buffer

This caused a spurious read, which lead to the socket being destroyed and no data being returned to the TACACS client when the initial request was fragmented

tacacs: Comments for mortals

tacacs: Print the error

all TCP sockets are non-blocking

remove last two arguments from tmpl_request_ref_list_from_substr()

the only two public users are the "update" code, and unit_test_attribute

we need to change the internal API to fix some other issues,
so rather than changing the public API, we just clean it up

json_object_afrom_pair_list only operates on leaf values

Replace != with !( == ) in json tests

Don't allow filters on list attr refs

By definition there will only ever be one instance of each list, so
filters are meaningless.

Convert input_list and output_list to tmpl_t

Remove unused tmpl_pair_list_name()

Remove meaninless test

Having tested the value of various attributes in the reply list, it
can't be empty

Correct docs and examples for %(pairs:) and %(concat:)

Tidy and expand pairs keyword test

Remove final instance of tmpl_pair_list_t

Set missing list_def

Pass a ctx to cache_verify

Ensures that unlang_fixup_update() does not replace NUM_UNSPEC with
NUM_ALL

Add tmpl_attr_head_is_list()

Remove tmpl_s.data.attribute.list

The list is now identified by the first attr ref

s/PAIR_LIST_<list>/request_attr_<list>/

and remove PAIR_LIST_<list> macros

Copy filter_type when copying tmpl attr refs

Update tmpl_attr_afrom_list() to work with list_as_attr

Remove list_as_attr from tmpl_rules_t

List tmpls should not be NUM_ALL

There is only ever one instance of each list pair.

Add skip_list option to pair_append_by_tmpl_parent

With all attr tmpls having a list as the first ar, need to be able to
skip that when building into temporary lists rather than into
request->pair_root

Remove code paths for list_as_attr != true

Remove unused tmpl_get_list()

Remove TMPL_TYPE_LIST

Remove un-needed uses of tmpl_is_list()

Logic corrections

Correct references to "all children of a list" in tests

Change &list !* ANY to &list.[*] != ANY in update tests

Define new tmpl_is_list()

Using the attr ref rather than TMPL_TYPE_LIST

Only check for cast if lhs is a leaf da

Add support for list_as_attr to cache map building

Minor changes in test output with list_as_attr = true

update-list-error works with update to edit code

Ensure list ar is updated by tmpl_attr_set_list()

Set attr ref parents when copying tmpl_attr

Allow for list_as_attr when maps are list to list

Allow &list.[*] for selected tmpl parsing

Add tmpl_attr_tail_da_is_leaf() and tmpl_attr_tail_da_is_structural()

Update map_to_list_mod() list to list detection

Will handle any structural to structural copy

Add allow_wildcard option to tmpl_attr_rules_s

Amend dcursor remove callback specification

Allowing the callback to do the actual list removal.

This overcomes issues with tmpl_dcursor operations where the dcursor
refers to the top level list whilst the entries need to be removed from
a child list.

Update callback for dcursor pair removal

To handle removing pairs from child lists

When resolving tmpl_attr which are children of a list attr use default dict

List attributes are in the internal dictionary rather than in a protocol
dictionary.

Reset tmpl evaluation when evaluating children of groups

With list_as_attr, virtual attributes have two entries in the ar list

Add sanity to foreach - skipping structural attributes

Allows for

  foreach &request.[*] { ... }

to process safely

Fix up tmpl_attr_print()

No need for [*] on debug policies

Set the tmpl list while inserting default list ar

Fix up tmpl_attr_tail_is_unresolved()

Update tmpl_attr_afrom_attr_substr() to handle tmpls such as &reply.

&reply refers to the existence of the list, &reply. refers to the
presence of children in the list.

Add tmpl_attr_tail_is_raw()

Use helper function to identify unknown ar

Add ar_is_raw() for checking for raw attr refs

Fix up cond_forbid_groups()

Add tmpl_attr_is_list_attr() to check if an ar refers to a list

Correct delimiter

Correct test for invalid list

Ensure request_attr_ DAs are initialised before they are referenced

s/tmpl_pair_list_t/fr_dict_attr_t const */

Use tmpl_attr_list_from_substr() to parse list names

Replace switch statements over tmpl lists

Remove pair_list_table

Enable visibility of request_attr_ DAs

Add tmpl_list_name() and use it to report list names

Add tmpl_attr_list_from_substr() to parse a list reference

Set PAIR_LIST_UNKNOWN = 0 to catch where the list is not explicitly set

Convert PAIR_LIST_ enums to defines referencing DAs

Allow using clang 15 with -Werror and -Wunreachable-code-generic-assoc (#4884)

Pre C17, there was ambiguity about the type of the controlling expression
in __Generic(). C17 explicitly states that it is subject to rvalue
promotions, which removes qualification from the outermost (or is that
deepest?) type modifier (or the base type if there are no modifiers).

(To clarify with an example, a FOO const * const will become FOO const *
as far as __Generic() is concerned.)

clang 15, perhaps anticipating C17, has -Wunreachabale-code-generic-assoc,
which warns if a __Generic() alternative type is unreachable, e.g.
FOO const * const. This change gets rid of such alternatives in a couple
of macros.

This builds and passes such tests as I can do compiling with gcc, clang 14,
and clang 15.

Pacify clang scan in slab tests

Add reserve callback to slab allocator

Allow for differentiation between alloc callback (just used on first
allocation) and reserve (used each time an element is reserved)

Set talloc type of slab elements

So that doing talloc_get_type_abort() on the entries returned by the
reserve function will work.

Move slab cleanup interval to be initialisation function parameter

Allows it to be sourced from a config option rather than hard coded in
function definition macro

Should be a case insensitive match too

Don't assert if no virtual servers are configured

Print a nice error message instead

tacacs: let's avoid _magic_ values (#4886)

Don't leak memory on invalid packet code (#4887)

remove state tracking

the configuration isn't exposed, and even if it was, the code
doesn't do anything other than track state.  Nothing uses that
state for anything.

If we want to use State to direct EAP packets to the same back-end,
the correct place to put that is likely in the fr_io_track_t
structure.  Which has to be written to in the "encode" or "write"
routine, and then also in the "read" routine.

The network side can then use that field to have a centralized
tracking structure of state -> worker.  Even if this state
is probabilistic, at 64 bits (and a reasonable hash function),
the odds of a packet going wrong are tiny.  If it's an issue,
we could just change the state to a talloc ptr, and use memcmp()
to compare states.

That also lets us put anything into the state, which is flexible.

Simplify use of Python version check

new conflicts with C++

Gcc fixes