remove one more $(shell ...)

remove one use of $(shell ...)

Merge pull request #3033 from jpereira/v4/fix2

Fix the radmin test dependency

silence compuler warning.  Fixes #3034

also process CANCEL action in the main signal handler

which simplifies the keyword signal handlers.  All they have to do
is pass the signal to their children.

Fix the radmin test dependency

GNU gmtime_r *still* uses the value of TZ when printing %Z even with gmtime_r which provides the time as UTC‽

Merge pull request #3030 from jpereira/v4/tests3

Some fixes for radmin tests

Fix travis build error

Fix typo in $(TEST).radiusd_kill

We should remove the radiusd.pid

if failed, show the last 100 entries

Be sure to use the latest build of bin/radmin

If informed 'RADIUSD_BIN', don't call 'jlibtool'

Fix to call straight target name 'test.radmin.radiusd_kill'

Instead of trough $(TEST) context.

Just call the 'exit N' due to Make don't expand it correctly

The Make don't expand the '${ret}' inside the macros.

fix compile warnings

sync time periodically.

Which lets us catch time zone changes (e.g. daylight savings)
and clock drift

Merge pull request #3026 from jpereira/v4/tests1

Add capability to 'RADIUSD_SERVICE' share the exactly command.

fix arguments to localtime_r

Add capability to 'RADIUSD_SERVICE' share the exactly command.

* The macro 'RADIUSD_SERVICE' now is sharing the exactly command used to
start the radiusd during the tests.
* Also, we could inform which radiusd we want to use with 'RADIUSD_BIN'

try to parse time zones.

and if we can't, just pretend they're GMT

add function to get time_delta from time zone name

Note that we ONLY allow UTC / GMT and local time zone names.
It is impossible in general to parse time zone names to offsets
from GMT, as there are duplicates in the zone names.

One more

Minor fixups

Add 'exit <num>' command to cause the process to exit, and full help text for all supported commands

Rework unit_test_attribute to use standard logging macros, and add a command table to simplify extending its functionality

Fix errors in table_sorted_value_by_longest_prefix

Dict dump should take const dict

Always print times as UTC

Add mini state machine for running auth sections

Merge pull request #3028 from jpereira/v4/tests2

Fix radmin 'set module'

Fix typo, it should be radiud.log

Add tests for radmin 'set module'

Fix radmin 'set module'

make tests use TEST_BOOTSTRAP function

move test bootstrap code into common function

renaming

* we don't need RADMIN_OUTPUT_DIR, we can just use OUTOUT
* it's better to use ".txt" for text files, even if they are
  input files
* similar ".out" instead of ".ret".  They're still text, but
  we care more to know that they're outputs
* test files don't need numbers.  If the tests are ordered,
  we should put that information into the tests instead of
  in the names
* test files don't need to be in a subdirectory.

Add tests for radmin/control-socket

Add new 'RADIUSD_SERVICE' make target

More informations can be found in src/tests/radiusd.mk

rename unlang functions for clarity

running a request is now just unlang_interpret(request)

functions taking more arguments have longer names

no longer needed

rename function

we're compiling a section, not getting a subsection from a
parent section

reset when going to next, so that it is always done

Remove server/base.h from io/base.h

Because circular dependencies aren't fun

Allow subrequest sessions to be enabled/disabled and for custom unique_ptr and unique_int values to be specified

This is required for modules pushing subrequests as the instructions usually used to identify sessions are dynamically allocated

Don't assign empty string

fix last commit

make '-i filename' work again :(

start of framework for "local" commands

remove debugging message

grammar and spelling in sites-available (#3021)

clients can be proto UDP, TCP, or *

!whitespace is also "not zero and not whitespace"

When parsing zero-terminated strings, stop at zero.

if no proto is specified, look for both udp and tcp

fix assertion

hacks to get parallel children resumable

Use table_len when iterating over tables Closes #3017

Fix typo (#3018)

Fix potential access to dereferenced value (#3016)

Return RLM_MODULE_YIELD from module functions

change check to assertion

add new unlang_subrequest_start()

to start a subrequest

remove "fixup" function

and move repeatable_set() to unlang_module_yield()

Move the store/restore subrequest calls out of rlm_eap

Make asserts less verbose

Be entirely unforgiving about fr_state_store_in_parent and friends being called incorrectly

Ensure the child has its own unique state_ctx, i.e. it's in the same state as when request_data_restore_to_child was called

Fix typo (#3015)

Only free the existing state_ctx if we're sure it's not the parent's

doxygen

try to work around things with unlang_interpret_fixup()

which runs things locally via playing with the stack.

fix return code

add missing declaration

check child state_ctx against parent

The EAP code should arguably call request_detach() or something,
but this is a quick and easy fix

rename "persist" as it's confusing.  Use and check "detachable"

only parent requests are runnable

more debug messages

Don't leak if failed the MS-CHAP password handle

Return correct rcode

Dump packet in radius encoder

Print hex blocks with whitespace

Fix eapmschapv2

Add tracing to mschap_challenge_hash

Fixup rlm_mschap to support EAP-Identity too

use sw->id which is local

use worker->name

instead of function call and thread-local storage

clarify comments and code structure

print "Resuming execution" when we resume

get rid of unlang_group_type

which was always a horrific hack.  Instead, just fix the
compile_action_default() function to sneak a peek at the parent.

Then, add "redundant" as a top-level type / keyword.  which
lets us print out "redundant" in debug mode instead of "group"

rename func / process to interpret

in preparation for adding compile / parse functions to the
unlang_op_t structure

Split out virtual servers and modules used by them

Fix clan scan issues

Fix comparators to return a number within a limited range

Not needed

remove deterministic load balancing, and revert to random

which means we have less magic knowledge in the interpreter, and
less recursive code.  At the cost of being less deterministic

remove "detach" magic from main interpreter loop

detach now "does the right thing", and behaves appropriately.

remove unused parameter to signal()

make UNLANG_TYPE_CALL skip using using resumable frames

and now that nothing uses resumable frames, remove them entirely
from the interpreter

make UNLANG_TYPE_XLAT skip using resumable frames

get rid of unlang_module_resume(), as we no longer need it.

And do some other minor cleanups.