remove INSERT_BEFORE, and update API

we only need INSERT as insert after

try to shut up clang scan

add edit lists and tests

so that we can edit lists "in place", and then undo the edits
if anything goes wrong.

functions return fr_pair_t*, not void*

check correct variable, and allow "pos" to be NULL

which lets us insert things at the start of the list.

Tidy up ephemeral module_ctx_t struct creation

Correct fix for rlm_sql_cassandra

Unbound needs the inst name for xlats

Sync up module and instance data name

Fix arguments to cassandra

Use correct mctx fields in rlm_rest

Mark command_encode_dns_labels as nonnull

Use structures to pass arguments to instantiation and detach functions

Keep a pointer to the module instance in the thread instance

Remove mctx->instance

It's annoying having the same data twice.  There's also tons of places this data is cached

use appropriate error functions

which mean we don't need LOG_PREFIX here

Pass the private version of dl_inst down in the mctx

Make TEST_CHECK_*() macros display stringified _got (#4310)

Conditionalize the definition of ALLOC_REQ() (#4309)

shut up compiler

be a bit more forgiving around registering xlats

for module instances like "foo { ... }" versus "foo bar { ... }".
if the xlat is already registered, don't do anything.  If it isn't
registered, then register it

no need to cache CONF_SECTION

no need to print the name

the module isn't going away until after it's data has been freed.

add 'const' here, too

add 'const'

remove rlm_ldap_t from fr_ldap_thread_t

the only use of thread->inst uses thread->inst->config, which is
already cached in thread->config

remove unnecessary pointer indirection, and make config 'const'

no need to call it mod_instantiate()

everything else uses instantiate()

remove unused prototype

Remove redundant comparisons (#4307)

make sure we catch 0 iterations in the $PBKDF2$... format, too

Working trunk tests (#4306)

* Enable working trunk tests

All time function usage is updated to use the new API.

* Change to make test_connection_start_on_enqueue() work

The code added causes the event to handle the backlogged request
to run, the absence of which throws off all the following checks.

don't bother using "rlm_" prefixes everywhere.  They're not needed

use "foo - bar" for names, as that's being used elsewhere, too

and the output looks cleaner.

use inst->name where possible

these are just EAP things, and are common across all EAP modules

remove LOG_PREFIX_ARGS, and just use LOG_PREFIX everywhere

which simplifies things, in preparation for adding log IDs
everywhere, too.

use inst->name instead of dl_module_instance_name_by_data()

use static name with log_prefix and ID

instead of re-converting them every time

unify to inst->name, instead of inst->xlat_name

shut up compiler

return on error

Revert "Remove hot branch in rbtree find_node"

This reverts commit b19a7ac6737e7a0f0aefb20cd94d9441d0e0eb3c.

Remove hot branch in rbtree find_node

Suppress lgtm weak crypto warning

Include dhcpv6 state machine options to control status-code contents

Add idx versions of pair_find functions

Update test now we don't send status by default

Information requests are not directed

Don't add status on success

cache align TCP packets, too

remove unused functions

cache align UDP messages

not TCP message for now, as that requires memmove() on the
"leftover" bytes, which requires more expensive logic

CPU cache line sizes are generally 64 bytes

Revert "Pass dict root not dicts..."

This reverts commit 1503b48d7dfd64c303f7db9b9872f16f54bb7fdb.

Pass dict root not dicts...

Pass in protocol specific dictionaries to the packet debug functions

Invert args

special free function

We apparently don't cache this anymore

singular not plaural

Include our talloc header here

Typo

Aruba dict updates closes #4297

...one more

...and another

Should be signed

Move talloc header

Copy interface id back if it was included

Fix various dumb issues

Fix typos

we don't need a second name here

allow running individual module tests

Need talloc

Dump capabilities to INFO() at various phases in the process lifecycle

add const

move mschap specific rules to top-level, and make top-level more generic

and don't bother re-running 'configure'.  It doesn't work
half of the time.

let's put files into ${localstatedir}, and not /tmp

move db_dir, as nothing uses it

fix xlats for virtual modules

don't even bother for "group".

Register the xlats in the "bootstrap" stage, by walking over the
virtual modules.

switch xlat_redundant() etc. to use the new XLAT API.

use another module for redundant tests

move virtual module to "modules" section

remove all references to "instantiate"

remove special cases for "instantiate", it no longer exists

look up virtual modules using new API

instead of relying on the old "instantiate" section

move virtual_modules back to bootstrap, and track them

expose an API to discover the CONF_SECTION for a virtual module

Revert "Revert "remove the 'instantiate' section, as it's no longer necessary""

This reverts commit 09416b43dfcb6b232bd184fd3a56156a5dd4426a.

allow for multiple instances

add LOG_ID_LIB to files

cleanups, and allow assigning of IDs to files

define LOG_ID_SECTION for the drivers

cleanups

allow for LIB/SECTION IDs to be in different directories

as with rlm_sql and rlm_eap

more notes

Add #include <sys/types.h> to the ssize_t test program (#4303)

That's (at least one place) where ssize_t gets defined, so the
test program can work as intended. (The header's there for Mac
OS X as well.)

Don't print the module prefix

Helps to init the lists

Copy all of the server_id and client_id attributes

Fix assert

Final temporary fix for replacing nested pairs