Add devcflags and ldflags last so they don't much with autoconf

...apparently not

See if this fixes normal redis tests

Fix cluster reset

Any tests which checks for error output has to touch the receipt file itself if it finds what it expected

Use the same reset script for ippool and normal redis tests

No longer used

Alloc tp_ctx from the autofree ctx

Assert if dl_loader struct isn't initialised and initialist the dl_loader struct in unit_test_attribute Closes #2568

remove output file on error

more use $(OUTPUT)

one last typo?

typos

more $(BUILD_DIR)/tests/keywords to $(OUTPUT)

move to new template

more simplifications

reset more re-used names

move to new template

move to new test template

this file doesn't exist

make sure that the file isn't empty

otherwise Make will re-run the rules on every build

reorganize and clean up

so that the definitions come from templates as much as possible,
instead of re-doing everything

formatting

rlm_lua: Print the version

Fix test target

Fix circular dependency which causes the tests to always run

Manually touch receipt file if the test is meant to error

More test fixups

Fix whitespace

Change rcode to ret

Use EXIT_WITH_FAILURE macros in more utilities

tests: Add '-r' (receipt_file) option in unit tools

tests: Fix missing '&' in src/tests/map/count-error

tests: Add missed 'clean.tests.' target

tests: Fix clean.tests.modules target

ff

misc.c: Add new fr_file_touch/fr_file_unlink

Add missed 'clean.tests' target

Use lua_tonumber for floats and lua_tointeger for integers

Formatting

Fix log level typedef

Add example invocations back

do_lua doesn't access global instance data

Remove key/mutex from rlm_lua

The original code was pre-thread-instances.  This is like vestigial.

Merge pull request #2564 from jpereira/v4/rlm_lua-update

rlm_lua: Update the sample src/modules/rlm_lua/example.lua

Let PgPool detect the stored procedure as needing a write transaction.

- Add an SQL comment, and documentation for the same, which triggers PgPool to not load balance this to a read-only server, and instead send it to a write server. As noted, this does not have any negative effect if PgPool is not in use, so it is useful to have here to avoid problems caused by someone starting to used this procedure when they've got a working PgPool/PostgreSQL installation already.

Signed-off-by: Nathan Ward <nward@searchlight.nz>

Fix the NULL comparison.

Signed-off-by: Nathan Ward <nward@searchlight.nz>

rlm_lua: Update the sample src/modules/rlm_lua/example.lua

use lua_tointeger()

which is more likely what we want.  That returns ptrdiff_t,
which is likely OK

use correct type

stupid doxygen nonsense

jlibtool: special case linking against rlm_ libraries

They don't start with "lib"...

Return 0 if internal dict not loaded in fr_dict_attr_by_name_substr

Both args can be NULL

Typo

Try with just the ippool tests

Use the cast functions in the lua unmarshalling code

Add fr_value_box_bstrndup_shallow

Fix other shallow functions to use a void return type

Use the fancy error marker macro to point to where the field lookups fail

Return with error code if internal dictionaries are not available

Fix flm_lua_get_field

Use CRYPTO_memcmp in more places in the sim library

Remove trigger_exec_free() call.  We don't use triggers here

Update ippool help text

Move set inside of cluster setup loop in case it takes a while for the cluster to come up

Reduce amount of debug output when waiting for the cluster to come up

Use talloc_get_type_abort_const in redis xlat functions

Fix update where no range identifier is specified when the addresses where added

Don't free the reply if we're skipping over the element, redisReplyFree walks over structures recursively freeing all elements

Enable the Redis and Redis IP Pool tests

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