Return the correct length for quoted values in fr_value_box_snprint

Remove invalid assert

Tidy up unit_test_attribute more

Don't leak memory in cond_normalise_attribute

Including sql.h causes conflicts on macos Catalina

don't print levels if we're doing syslog

indent VPs we're printing.

Which is only used by radsniff, radnsnmp, and radclient.
All of which need VP indentation for readability.

use "const" and common functions for common work

log functions should take const

add ability to suppress level prefixes for log messages

so that radclient, etc. debug output doesn't get messed up

remove "test.eap" from the "test" target

because pretty much all of the EAP tests are broken.  Once EAP
is fixed, we can re-enable it as part of the test framework

uctx may be NULL

remove debug message

make the cert files depend on the openssl conf files

use predefined TESTBIN instead of local magic

reorder included make files

so that GENERATED_CERT_FILES is available for the tests

replace build with ${BUILD_DIR}.  Fixes #3031

use cached CONF_SECTION

add base / offset to virtual server compilation

so that the CONF_SECTIONs can be cached somewhere

allow unsigned types for time_delta

Merge pull request #3036 from jpereira/v4/tests1

Keep the common test vars in single place

Keep the common test vars in single place

so, let's keep PORT and SECRET declared once.

Print the used port

more reordering

move test rules into src/tests/all.mk

and do minor cleanups

Merge pull request #3035 from jpereira/v4/test-digest

Migrate the tests in src/tests/* to src/tests/digest

use a common name

remove one more $(shell ...)

This is similar to commit 22e25aed

Migrate digest tests to boiler scheme

Not needed those files anymore

Migrate digest tests to boiler scheme

define and use GNU Make functions which replace $(shell find ...)

print warnings only when asked.  Fixes #3032

It will now ignore warnings in normal builds, and only them
when running:

    make check.configure

this change is because we can't control the timestamps on the
`configure` and `configure.ac` files on `git clone` or
`git checkout`.  The `git` program may write one file before
another, and not necessarily in any order we would like.

Since we can't fix that, we just suppress the warnings.

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