Fix issues with uri_escape

Whitespace

Minor typo in doxygen comments

There is no longer a DHCP specific sqlippool module

Add rlm_unbound to stable modules list for packaging

Update SQL counter module howto

Add test of sqlcounter module

sqlcounter is not necessarily time

Use a config option to extend reply attribute value

Rather than doing hidden magic on a specific attribute.  Allows the same
principle to be used for protocols other than RADIUS.

Re-work sqlcounter mod_authorize to push sql xlat on stack

Add instantiation checks that the sql module instance actually exists

Better name for SQL instance (aligned with sqlippool)

SQLite doesn't have GREATEST, use MAX instead

Use pair_append_by_bmpl_parent to allow for nested attributes

Better handling of adding module attributes

Allows either an attribute created by the module or one already in the
dictionary.

If it's a dictionary one it will be checked for the correct type.

This is (usually) creating the attributes, not finding them

And if it fails, that is module failure rather than noop

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