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.

Change length to operate on incoming boxes instead of using an attribute ref

The general rule is that refs are only used where information from the VALUE_PAIR is required

comment out illegal dictionaries

these vendors haven't shipped product for a decade.  People
wanting terrible attributes can deal with it themselves

butchery to make the tests pass.

Fix the unit-test dictionary so that it doesn't barf.

Fix all.mk so that it doesn't depend on a file that doesn't exist

Move most of the "Unit-*" tests to radius_unit.txt

remove dict.txt and command.txt from running during the tests.

For dict.txt, the framework doesn't provide for dictionary
definitions within a BEGIN-PROTOCOL framework.

For command.txt, the command parser no longer allows ":" in command
names, and there's no point in tracking it down right now.

load the radius dictionary, too

don't depend on a file which doesn't exist

t means that the unit tests haven't been running for months.

some fixes to make it slightly better

actually showing encoded data would be useful

Merge pull request #2535 from jpereira/v4/fix5

rlm_isc_dhcp: Fix initialization

rlm_isc_dhcp: Fix initialization

typo

parse `server-identifier IPADDR`

and do some minor cleanups

one more fr_skip_spaces

Doxygen

Drop h and H from randstr, people should use %{hex:%{randstr:b}} instead

- They were only there because the xlat evaluation code wasn't binary safe.  Now it is, there's no reason to keep them.
- Make generating large amounts of random binary data more efficient
- Increase repetition limit to 1024 and warn if we override what the user specified

Unconst errstr

Don't leak errstr

Allow fmt arguments in RDMARKER and REMARKER

Fix function documentation

Don't print anything if we're outside a module

Move skip spaces macro

Use fr_skip_spaces macro

Add fr_value_box_list_len_min

Print concat error

Merge pull request #2530 from jpereira/v4/util1

Add fr_skip_spaces() helper to strip whitespace

Merge pull request #2531 from jpereira/v4/value-length

Add fr_value_box_list_length() function

Various xlat cleanups

Add fr_value_box_list_length() function

Add fr_skip_spaces() helper to strip whitespace

Merge pull request #2529 from jpereira/v4/xlat-async-tag

xlat: move '%{tag: ...}' to async

Merge branch 'master' into v4/xlat-async-tag

Merge pull request #2528 from jpereira/v4/xlat-async-length

xlat: move '%{length: ...} to async

Merge branch 'master' into v4/xlat-async-length

Merge pull request #2523 from jpereira/v4/xlat-async-strlen

xlat: move '%{strlen: ...} to async

Merge branch 'master' into v4/xlat-async-strlen

Merge pull request #2522 from jpereira/v4/xlat-async-module

xlat: move '%{module: ...} to async

xlat: move '%{tag: ...}' to async

fixup! xlat: move '%{length: ...}' to async

xlat: move '%{length: ...}' to async

add support for `next-address IPADDR`

add parsers for `server-name STRING` and `filename STRING`

typo

xlat: move '%{module: ...} to async

remove extraneous printf

make hosts global

xlat: move '%{strlen: ...} to async

remove duplicate sql config

Fix libcurl's apparently random line termination and concatenation in debug output

Doxygen?

Add length to rest_debug_log

Minor doc typos closes #2524

Add debug functions for libcurl

Integers passed to curl_set_option should be long

Should be debug level 2

Formatting

Typo

Document PostgreSQL states override

Rename 'done' rcode

Shouldn't be a title for $-INCLUDE

Build error

Allow user configurable sqlstate return codes for PostgreSQL

Formatting

Default to sending the application name in v4

Function to mark pair as parsed, for when we're manually processing pairs

Minor fixes in trie function

- Data to insert should be marked as const
- Callback should take keylen as size_t

Notes on setting up PostgreSQL server for macOS

add "pedantic" flag

add "inst" to parsing state

check for first 3 characters, to avoid strncmp() cost

typo

encode "bool" types, too

run through the formatting script.

and touch up a few attributes

Merge pull request #2514 from jpereira/v4/isc-dictionary2

Add ISC dictionaries for rlm_isc_dhcp. ref #2501

Merge pull request #2519 from jpereira/v4/fix1

rlm_isc_dhcp: Add missing talloc_free()

rlm_isc_dhcp: Add missing talloc_free()

allow and parse 'array of'

and parse arrays attributes properly, too

create a new VP for each array VPs

add in the new option that the user defined

start of adding option definitions

cleanups and fixes

differentiate between options we deliberately ignore, and ones
we don't implement.

start of parsing for multiple different option statements

rlm_isc_dhcp: Add ISC dictionary. ref #2501

don't print offset

we don't track which token came from which line, so it's best
to not mislead the admin.

We can fix this by tracking line numbers and start of lines
in the input buffer, which is a bit more work...

start of a parser for quoted strings, too

use a macro for less cut & paste

note that the macro is lowercase, for less shouting.
It also lets us more clearly distinguish between the "not done"
commands, and the commands which do something

add more commands.  Manual merge of #2515

look at first character for speed

xlat_debug_attr should not produce an error if we're not at DEBUG level 2

I think these were previously ignored in bare xlats which is why this wasn't previously a problem

Change more messages to DEBUG2

Print the result of policy evaluations

Switch most calls to RDEBUG to REDBUG2

Frontend modules left using RDEBUG, module return codes changed to RDEBUG, this should hopefully produce more consistent debug output, and make radiusd -x  more useful

Use fr_value_box_to_network in DHCPv4 encoder

remove the options from the top-level table.

The options are implicit in `option NAME VALUE`

the NAME is looked up in the DHCPv4 dictionary

ignore boolean options with "ignore"

formatting

make notes about matching `host` only if `fixed-address` matches

move to binary search for commands