Correct logic for resetting times

Conf options have CONF_FLAG_REQURED so will exist

Ensure that sqlcounter queries return 0 if there are no matching records

sqlcounter start and end attributes are by default in the control list

Flags need to match what will be on the created DA

After inserting, tmpl_attr_tail_unresolved_add checks the resulting
flags and fails if they differ.

Align default attribute names with sample SQL

And add quoting, needed to parse defaults correctly.

Add reply message attribute to sqlcounter call_env

Move sqlcounter reply_attr to call_env

Less noisy debug

More appropriate variable name

It's no longer a magic paircmp attribute

Use value box cmp / cast to avoid duplicate code

And one more

More doxygen fixes

Doxygen fixes

autofree talloc ctx is now used in all builds

Add function documentation

Correct function docs

start moving radclient-ng to the new bio API

hoist socket name into common function

and get socket name after connect(), too

set the bio type

start moving configuration parameters to fr_bio_fd_config_t

don't open new sockets

Remove repeat talloc_free

If user is in radcheck but not radreply, still merge control items

Ensure test_connection_levels_max() requests are completed/cancelled

Add basic tests of mschap module

Use call_env for libwbclient auth in rlm_mschap

NT Password is not needed for external mschap methods

Include rlm_unbound in packaging

Same as v3.2

Allow for differing versions of ub_event_callback_type

Add tests of rlm_winbind

These are not errors

Use call_env in %winbind.group() xlat

More appropriate debug message types / levels

Remove unused option

Use custom call_env parser to handle automatic domain lookup

Use call_env in winbind mod_authorize

Use new xlat function syntax

xlat_arg_parser for winbind.group is static const

There is no escape function needing access to inst

Use a call_env for rlm_winbind mod_authenticate

Tidy up

Move loading of rlm_files data to call_env

Correct comment

Formatting

Temporary &users list is not talloced, only its members are

Align sample configuration with rlm_files only having one filename

Just use a single filename in rlm_files

If different data is used in different contexts, use multiple instances
of the module

nope.  header length is 4

check for fragment with insufficient room for header

add test and example for eval

add output from fuzzer

hopefully we can get a better error message running the normal
tests.

Remove protocol prefixes from attribute references

Update sqlippool docs

sqlippool should be called in `accounting ...` sections

So that the correct method is used.

Add tests of rlm_sqlippool

Convert remaining sqlippool methods to use call_env

Move the rest of sqlippool alloc to use call_env

Add alloc queries to call_env

Update sqlippool query functions to expect pre-expanded queries and take an rlm_sql_t

Add sql escape uctx alloc / free functions for SQL IP Pool queries

Define custom call_env parser for SQL IP Pool queries

Add box_escape_func to SQL module instance

Make module / xlat instance data available when parsing call_env

And pass it to the custom parsing function.

Add uctx for SQL escape functions

Allows for passing of an exising connection handle

Pass safe_for to xlat_tokenize_input

Add rules for parsing literals in tmpls

Update sqlippool rcodes to match those returned by redis_ippool

More helpful debug messages

Move requested_address to call env

Move pool_name to call env

Add initial call_env to sqlippool

Just for allocated_address_attr to start with

Add tests of if using nested && and ||

Return XLAT_ACTION_DONE on logical false

XLAT_ACTION_FAIL is only when there is a failure, not when the return
from real evaluations is false

add test case to prevent regression

better catch malformed attributes

remove -n and -p

remove coa functionality

in the quest to get bios, we simply delete old code, which will be
eventually replaced with new code

use correct function type to quiet ubsan complaints

we didn't call xlat init, so we don't call xlat free

reference count init/free instead of using boolean

we didn't call xlat init, we shouldn't call xlat free.

normalize names

foo_global_init() and foo_global_free()

for helping with fuzzer issues

I do have loathing for compilers which don't produce error messages

comment out assertion to see if fuzzer will produce better error

make sure to free tag ctx, too

make sure that we call library init / free

free tag ctx, too

no longer any need to call init.

Missed in commit f7778c55206db

Add foreign tests to DHCPv6

clean up autofree

The only remaining errors are:

test.unit.dns, which fails to clean up the DNS dictionaries.
All of the other protocols are fine, so there's some extra magic
here.

test.keywords: fr_dict_free() is asked to remove a dependency
which doesn't exist.  That _should_ be an assertion, but for now
it's a recoverable error which lets other people get work done.

just call common function

add dependency if we load the dictionary

and thus we can call dict->proto->free()

add debug printing for dependencies

don't free dictionaries until we track down reference issues

There are issues with calling proto->free() in the destructor
for the dictionaries

there are issues with foreign references.

set loaded flag before calling proto->init()

to prevent recursion

add assertion

remove unused code

Move to init/free for library, and not for each test case

now that we (mostly) have library-based init/free, we do not need
to do that for every test case.  Either the library is properly
initialized and it works, or the library isn't initialized, and
the tests fail.

Some protocols still need to be converted to use fr_dict_protocol_t

ethernet, tacacs, tftp, vmps

don't add the dependency twice