Amend comments for rest module "pool" to reflect code behaviour

Handle changes in libcurl options

Use FR_CURL_REQUEST_SET_OPTION where request is available

So errors are RERROR rather than ERROR

Don't set CURLOPT_PROTOCOLS twice

Always use error4

Try pausing the child processes until the parent signals on macOS

Cleaner version of waitid hack on macOS

Diagnose macos CI failures

Should be waitid not waitpid...

Fix tmpl_dcursor_test warnings and make it possible to easily step into the tmpl tokenizer on failing tests

mark the parent redundant xlat as taking "void"

so that we can free it's args, and they don't be expanded.

The arguments will still be used for the parser, so it's not
all lost.

change !~ to !(... =~ ...)

Try bumping CI to macos-12

Param is now used

rest: Search for pool in the right place

Formatting

Don't warn about unreachable generic types when the lvalue is case (unreachable-code-generic-assoc)

What a completely useless and utterly stupid warning

Fixes for running with a non system version of clangd on macos

Remove accidentally comitted tests

This reverts commit d18cd6caf55d5c210211fd14b6bf817d972f4614.

Formatting

json: Set the default output mode correctly

curl: Similar signal fix for smtp and imap

curl: Deal with socket closures in a more libcurl friendly way

event: Call the read event on EV_EOF if we're reading and have pending data

Correct bounds check for max_request_time

rlm_rest: Always install a signal handler for unlang_module_yield calls

rlm_rest: Formatting

Fix lua_alloc_cmd (#4813)

(0)      redis_ippool - ERROR: (0) error   : ERR Error running script (call to f_555f9a10a670669b1725edfc4b558c4c5eb3f9ad): @user_script:14: user_script:14: attempt to compare number with string

We don't need to do "%{Attr-Name[#]}" either

Just bare xlats are fine now

change "%{Attr-Name}" to &Attr-Name

It's really not clear where this obsession with quoting came from,
but it's unnecessary.

Time delta check only needs lt and gt comparisons

Rename fr_equality_op to reflect its actual meaning (#4823)

The array contains true values for comparison operators, not
just equals and not equals. A rename to fr_comparison_op
makes a positive logic version of the distinction between
comparisons (which will coompare any number of values)
and other binary operators (which only do one value) clearer.

Scheduled fuzzing: Update src/tests/fuzzer-corpus/radius.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tacacs.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dns.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv6.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/dhcpv4.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/util.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/tftp.tar

Scheduled fuzzing: Update src/tests/fuzzer-corpus/vmps.tar

add RPEDEBUG wehn failing assigment and update tests

clean up and add new docs

more != to !(==)

global search and replace

perl -p -i -e 's/\(([^ ]+) != ([^ ]+)\)/(!($1 == $2))/g'

make redundant xlats work

via hackity hack,  The real solution is not to copy the input
(unexpanded) arguments.  Instead, the xlat_redundant() call
should simply copy it's expanded arguments, and call each
function in turn.

i.e. xlat_redundant() has no need to call xlat push or xlat
yield, as it already has all of the arguments it needs.

That also stops the inputs from being epanded repeatedly.

more a!=b --> !(a==b)

perl -p -i -e 's/\(([^ ]+) != ("[^"]+")\)/(!($1 == $2))/g'
             's/\("([^"]+)" != ([^ ]+)\)/(!($1 == $2))/g'
     's/\("([^"]+)" != ("[^"]+")\)/(!($1 == $2))/g' ...

lets initialize fields, m'kay?

open files passed on command line

allow -f to specify all file names

Ensure output directory exists for extracted pcap

shut up clang scan

add CoA filter and reply file to rc_file_pair_t

use dlists for filenames

that way they get processed in the order that they appear in
on the command line, and not in some random order by sorting
the filename.

This isn't stricyly needed here, but will be needed for receiving
CoA packets.

Add -A Foo-Bar

so we can use an arbitrary attribute to match CoA packets to
outgoing requests

first pass at allowing radclient to read CoA packets

simplify code.

xlat's have not been supported for a while.  So there's no reason
to manually root through the list looking for things.

use dlist functions instead of hand-rolled things

add === and !==

After other languages which have the same thing.

This is less useful for FreeRADIUS, as all "variables" are strongly
typed.  But it could be used for non-quoted xlats, and it could
be used in the future for "void" types

tests for expr with non-existent attributes

Ensure gdb knows where to load relative path library files

Temporarily revert to sleep in place of triggers

until issues with exec are resolved.

CI: Correct port mapping for 389ds container

Handle failure to copy args (CIS #1517104)

Don't need to build everything to get pkg_version

A better way...

Fix local RPM build scripts

change  (foo != bar) to !(foo == bar)

See previous commit for long explanation, and Perl script which
made the changes

fix failing tests

perl -p -i -e 's/\(([^ ]+) != ([^ ]+)\)/(!($1 == $2))/'  $(make test.keywords.help | sed 's,test\.keywords\.,src/tests/keywords/,g')

i.e. change

if (foo != bar) {
test_fail
}

to

if (!(foo == bar)) {
test_fail
}

The first test is "foo does not exist OR foo != bar"
the second test is "not ( foo exists AND foo == bar )"
which are very different.

After making that change, these tests failed.  So we've updated
the checks to be correct, which now make the tests succeed.

if-paircmp uses the old-style conditions

don't use "update"

upcast string to octets

everything else was upcast to octets anyways

radict: Add new options (#4819)

-c print out in CSV format
-H print out the Headers

remove unnecessary edit

probably left over from some earlier cleanup and rewrite

just use real attributes in the control list

Use trigger to detect when server is ready in ldap_sync tests

More reliable than just a sleep 1

Add trigger calls to ldap_sync when syncs start and stop

Take a copy of trigger args so async triggers have access to them

Simplify - cursor not needed just to prepend one pair

Ephemeral xlats do not rely on xlat thread instance data

Instantiate ephemeral xlat for trigger

Fallback to internal dictionary if request->dict not set

Currently all trigger args are in the internal dictionary
and request->dict is not set.

docs

Reduce debug noise for triggers not configured

update docs to match code

update local dictionaries for local variables.

Right now this means that local variables are really just local
attribute definitions.  And we can then use those attributes
anywhere.  And, they're not cleaned up when the scope exits.

So... don't use them for now.

Allow dictionaries to depend on each other

and add API to see if the dictionaries are compatible.
Because we can no longer depend on comparing pointers

preliminary code to interpret UNLANG_TYPE_VARIABLE

does nothing right now, but at least doesn't crash if you try to
use local variables

Remove redundant internal dictionary copy of LDAP-Sync attrs

Report which sync is being started / abandoned

Add UBNT dictionary (#4818)

Correctly parent value boxes from their pairs

preliminary support for compiling local variables

However, it doesn't quite work as yet.  The tmpl_rules_t have
parents, and this a parent dict to check.  But the pass2 resolution
functions just get passed the local dictionary, and not the entire
tmpl_rules_t.

As a result, the attributes are looked up in either the local
dictionary. or in the protocol one.  But there's no fallback
from one to the other.

use consistent names

t_rules is tmpl_rules_t
at_rules is tmpl_attr_rules_t

typo

add fr_dict_protocol_alloc() for use with local dictionaries

update parser to allow local variable definitions

and don't allow unlang statements inside of list assignments like

&request += {
...
}

This was previously parsed, and then would result in some weird
load-time error.  Forbidding it here makes the errors a bit clearer.

always initialize output buffer.  Fixes #4816

Add max_outstanding option to limit number of processing ldap_sync packets

Add fr_network_listen_outstanding to get the number of outstanding packets

Version number fixes for RPM packaging