Update the redis_ippool test cases so they use newer unlang/xlat syntax where appropriate

Use RADIUS attributes everywhere instead of DHCP (we don't allow DHCP outside of DHCP virtual servers now.

and fix other minor things like utility paths...

Typo in redis cluster log message

Fix issue where no ips are added where the initial request triggers a cluster remap and the number of IPs allocated is less than MAX_PIPLINED

don't actually need to set module name

Cleanup log output in rlm_redis_ippool_tool

Fix request leak in unit_test_module on error, by smashing it with a big hammer...

Switch fr_redis_pipline_free to inline function

Formatting

Rename reply_num to reply_cnt

Formatting

Don't leak reply objects the any time the script needs to be loaded

This is a pretty minor leak as the scripts are usually only loaded on startup, but if someone kept trashing the script cache, this could become an issue.

Use proper type for s_ret so we get enum values in the debugger

Remove trigger and dictionary init from rlm_redis_ipool_tool

Fix leak of conf->cs too so we get a clean exit under ASAN

Fix memory leak in fr_redis_get_version

Add additional redis_node and redis_remap xlats so that we can get the redis tests functioning reliably again

The root cause of this problem is that the utility used to setup the cluster no longer assigned key ranges in a deterministic way.

The tests assumed data would be written to one node, and it was instead written to another.

Add additional functions to allow remaps to be triggered, and to lookup the node that should be serviced by a key

Formatting

Fix edge case with fr_vasprintf

%pV:%u would print the boxed value and a pointer to the box.

Deal with REDIS_RCODE_MOVE for directed queries (queries against a specific node)

Before the xlat would just return an empty result and not indicate why

Create value boxes from status messages and errors too

This is actually pretty useful now we have %pV which didn't exist at the time the function was written...

Have fr_redis_reply_free set the pointer containing the reply to NULL

comment out test until we figure out why it fails

Merge pull request #2556 from jpereira/v4/test1

The 'make test' should call 'tests.bin' first

Merge pull request #2555 from jpereira/v4/fix-test

binary unit tests

The 'make test' should call 'tests.bin' first

tests.bin: Add unit_test_attribute test

tests.bin: Add tests for smbencrypt result

Add length parameter to 'ring_buffer_test'

tests.bin: Add radsniff interface list test

tests.bin: Add message's set test

tests.bin: Add control's test

tests.bin: Add queue atomic test

tests.bin: Add tests for all binaries

Fix the '-h' return to be success

Revert "more CC_HINT(nonnull)"

This reverts commit 4fa52e12c96dc43865a7b5f13aca6f13f81c2f19.

<sigh> Stupid compilers don't notice that variables aren't NULL.
This change will have to be done in many small commits

Merge pull request #2554 from jpereira/v4/fix-ring_buffer

ring_buffer_test: Fix fr_cond_assert() call

ring_buffer_test: Fix fr_cond_assert() call

parent node properly, and add a free callback

Merge pull request #2553 from jpereira/v4/fix-rbmonkey

rbmonkey: Fix rbtree_create() call

rbmonkey: Fix rbtree_create() call

use a more consistent error message

radiusd needs a whole lot more things

so it's better to test it later, after the various module
tests, etc.

short names are better

Merge pull request #2552 from jpereira/v4/tests.bin

New 'tests.bin' unit test

Fix missing 'h' for getopt()

Add new 'tests.bin' unit test

Return 'EXIT_SUCCESS' during '-h'

more CC_HINT(nonnull)

Merge pull request #2551 from jpereira/v4/bin-radict

radict: Return 0 during -h

radict: Return 0 during -h

Fix return codes

Fix references to eap.conf

Don't do work if the triggers aren't enabled

Initialise triggers in unit_test_module

Fixup redis tests

Split the initial spawning phase out of the pool init phase

The pool now has additional configuration functions that might need to be called before it starts spawning connections

Typo

Ignore calls to all trigger functions unless trigger_exec_init has been called

stubs for testing binary utilities.

make it work.

we can't use dict_read() to read the protocol dictionaries.
dict_read() loads the file into an existing dictionary, and we
don't want that.

Also don't recurse on directories, because the dictionaries
aren't (yet) smart enough to autoload protocols in subdirectories

load using the new dictionary layout

Merge pull request #2548 from jpereira/v4/fix-dict-conflict

Fix the dictionary conflict

Fix the dictionary conflict

manually pull over changes from v3

Merge pull request #2545 from jpereira/v4/fix-dict

V4/fix dict

Fix fr_dict_internal_afrom_file() reference

Fix ldap/sync dict calls

Fix radmin dict calls

load the dictionaries, which are needed to load the config

Add '-D' command-line option to set the dictionary directory.

Don't load the config when passed '-f'.  That's bad form.  Load it
after all configuration options have been parsed.

Add "read dictionary" code.

Move trigger init until after "read dictionary" code

Merge pull request #2544 from jpereira/v4/fix-radswho

Fix radswho dict calls

Fix radswho dict calls

Fix some cosmetic issues in rlm_redis_ippool_tool

Print correct name, and use correct check

More tests for literals

Fix abort on '\' fmt string.

More xlat tests

move procedure to new file

It's better to have a different file than to ask the admin
to un-comment lots of code.

Add a stored procedure for PostgreSQL rlm_sqlippool.

This matches the functionality of the existing example query which returns the users previous IP, if possible, however it runs approx 400x faster.

we don't need this any more

use parent vendor, not magic vendor

"array" types are allowed in multiple places.

@todo - update the dictionaries with where they are allowed.

typos

Exit earlier in xlat_tokenize_alternation if the alternate is empty

Remove const hack in xlat_tokenize_alternation

More precise error messages

Handle escape sequences in xlat test input/output data

...and add a few more tests.

Previously \t was treated as two characters (\ + t) which is not what we wanted.

remove debug entry

Not a ptr

Use isdigit instead of isnumber

Better empty expression check

Should replace the existing error

Fix another comment

Stray comment

Split out regex, func, attribute parsers from xlat_tokenize_expansion

The logic is almost identical but deals with a few corner cases better

Get exact parse errors passed back from tmpl_afrom_attr_str

Allow syserror to be explicitly freed

Make the tests work without an internet connection/DNS cache

Using the standard fr_strerror functions produces identical errors to passing around a static char **error pointer

Error can now be stacked too, which generally gives the user much better context.

In the vast majority of cases we no longer need to modify the input string

as posted to the list

Fix DHCPv6 bool test

Like EAP-SIM, DHCPv6 represents bools by presence (if true), or absence (if false), of the option.