Use Module-Name and Module-Instance in global triggers

Use correct value for trigger name

Pass trigger_args to trunk_alloc

Add trigger_args to rlm_tacacs

Add trigger_args to rlm_radius

No need for goto error

Add trigger args to rlm_ldap

Add trigger_args to rlm_sql

Add module_trigger_args_build

As a common helper that can be called by modules which use triggers,
especially intended for modules which use trunks.

Add extra internal attributes for triggers

Correct comment

Add trigger_args to trunk

Remove stray &

Allow control of triggers for rlm_redis_* using a conf option

add a completely bizarre, weird, confusing, and ridiculous dictionary

add more helpful error message

No need for documentation build deps on crossbuild checks

Correct talloc parenting of dynamic radclient

create Event-Timestamp if it doesn't exist

account for Acct-Delay-Time, too

typo

Set box to true if touch succeeds

Add test server environment variables to .vscode launcher

Based on values which match test servers set up by CI server setup
scripts.

update key field based on found struct

Ensure ci is populated

Define %file.touch()

Since it's used by ldap_sync tests

These are freed by freeing request

LDAP and SQL triggers are now driven by trunk code

Which use fixed names of `pool.<trigger name>` rather than
`modules.<module name>.<trigger name>`

WS

Correct comment

Add new MIB for trunk state triggers

Don't attempt to run triggers when there's no event list

Which will happen for any triggers fired late in the shutdown

Set trigger_prefix for rlm_detail

So per-instance triggers can just go in a trigger subsection rather than
in file { trigger { ... } }

If trigger_prefix is defined, use the passed in CONF_SECTION

Allows, for example, rlm_detail, which only outputs to files, to avoid
having to put trigger definitions inside overly nested sections.

Add caching of trigger CONF_PAIRs and trigger undef to exfile

Add optional CONF_PAIR ** argument to trigger()

Allows passing in of known CONF_PAIR and return of found pair, so
triggers which are called often don't have to repeatedly hunt for the
pair.

Make connections triggers optional

Cache when trunk triggers are not found

So we don't hunt for them on every state change.

SQL trunks connections change from ACTIVE to FULL and back on every
query due to the limit of 1 query per connection - so it is unlikely
that triggers would be configured for those states, and caching that
they are not configured saves a lot of conf pair finding.

Make exfile based triggers optional

Otherwise, enabling triggers will cause `rlm_linelog` and `rlm_detail`
to perform a lot of hunting for configured triggers as files are openned
and closed.

Return different values depending on why a trigger was not fired

To allow cacheing of un-defined triggers

Use a conf parser function to find module trunk trigger conf sections

Add CONF_SECTION for trunk connection / request triggers

So modules using trunks can have their own trigger sections.

Move trunk triggers boolean to trunk->conf

Splitting between connection and request and add CONF_PARSER rules to
set the options.

If we find the module "trigger" subsection, use it

minor cleanups and notes

add migration flag for key field transition

there's no configuration for it, but the flag is added automatically

it helps to parse unions here, too

move encode_keyed_struct() to its own function

and remove last vestiges of "flat" encoder

move encode_tlv() to child function

run dir is moved to /run (#5636)

Use of MemoryLimit is deprecated (#5638)

move generic encode / decode to encode.c and decode.c

tweak loop, so that there are fewer checks inside of the loop

use generic encoder

more use macros instead of hard-coded field checks

which makes it easier to clean up and rearrange the various
fields in fr_dict_attr_flags_t

use macros in more cases

add new macros for length=uint8/16 and use them in dhcpv4/6

and update the checks on the attr_valid() callback to set the
flags in more circumstances

better checks for parent / child lengths

don't assert if we fail to instantiate the dictionaries

continuation of commit 4644f501cf3

create and use generic decode_value() function

which makes the struct decoder a little simpler.

While we're at it, rework the struct decoder to be clearer.
And decode raw values more often, instead of hoisting the raw
pair to the enclosing struct.

make flags.length 16 bits

because fr_dict_attr_flags_t is 8 octets for alignment, so we still
have a few extra bytes to work with

remove unnecessary code which doesn't do anything any more

rearrange in preparation for encoding unions

Module trigger names need to start "modules."

in order to find the correct section in trigger.conf

Report errors from failed SQL queries

If no rows are affected output zero

Rather than no boxes, which just results in warnings when assigning the
xlat output to an attribute

Copy all instances of Module-Failure-Message to outer request

Some failures produce multiple instances and you want all of them when
logging failures

Update docs from raddb

Typo

rlm_ldap doesn't open new connections for each bind

Add triggers to rlm_crl

Timer list is used

add unions to struct decoder

and add test case for it

a union closes its parent struct

for now, unions are all at the end of a struct

Simplify ifdef use

If the rlm_crl mutex is taken due to CRL fetching, reset the timer event

Avoid mutex issues in rlm_crl

If the current thread is fetching a CRL, rather than wait for the mutex, yield the request until the CRL fetching is complete.

add checks for fixed size unions

don't clear is_known_width field, just force it to be set

some types are always known width.  Others are only sometimes
known width

move child of union to ATTRIBUTE instead of STRUCT

Capture scope_id populated by fr_socket_bind

scope_id is part of the comparison used to look for conflicting
listners.

If it is left at zero, then multiple interface specific listeners can't
be openned using the address 0.0.0.0 as is needed for handling DHCP
broadcast requests.

Don't throw away discovered IPv4 scope_id

Remove unused code

This was only used by `update`

remove forbid_update and rewrite_update migration flags

the framework is temporarily left in place, so that it can be used
if we need to add more flags.  But that should be removed before
a v4 release

remove UPDATE enum

this is no longer a todo

remove "update" handlers

we no longer support update

so we don't need tests for it.

And these tests were not "map" tests, but 'update" tests

remove tests for "update", and tweak docs

do not talloc_free() pre-existing dict on error

safe_characters is irrelevant for MySQL / PostgreSQL

as they use their database specific escaping functions.

SQL-User-Name is nothing to do with escaping now

Add support for "open_query" option to rlm_sql_postgresql

The option is listed (commented out) in the stock queries.conf

Normalize SQLite queries

Normalize PostgreSQL schema and queries

Normalize and correct Oracle schema and queries

Normalize and correct MS SQL schema and queries

fix test

so it actually loads the "good" test dictionary.

add "bad" test dictionaries.  Only one for now.

regenerate from source

clarify error message and fix test