use better / new names

remove "todo" which we have now done

Ensure the request is used

minor hacks for control sockets

Call the resume function whenever the interpreter is called

More debug

Remove old asserts and code, add function to check if a request is scheduled

Clean up debug for empty groups

The closing bracket here is emitted already

Don't need to push empty top_frame stack

Set default rcode correctly for the instruction we pushed

We still need an event loop assigned to the request

Actually remove the server_cs field

Use the synchronous interpreter for xlats

Alloc a dummy async section

Remove some references to unlang_interpret

Typo in unlang_call_current

synchronous intepreter - Deal with requests which immediately yield

Allow functions to be top_frames and remove garbage frame in child requests

Fix pairs test

Return should not return _through_ the current module

Fix stupid errors in DHCPv6 state tables and restore code

No modules should call RETURN_MODULE_YIELD

Resume rcode should be set from *p_result

Don't force rcode to OK

Correct action-return test

Sent reply code to OK in the success policy

Don't want server-id for solicit

Include entry for DO_NOT_RESPOND

Allow "Do-Not-Respond"

Add fr_pair_find_by_ancestor

Qualify reference

fill in the correct attributes for Protocol-Error

add support for Protocol-Error

use name from header file

remove unnecessary "reject" field

Start of new DHCPv6 process module

update configuration to match new process module

failure to load a process module is an error

so we'll need one for the "control" virtual server.

Also don't push the config rules manually, dl_module_instance()
has already done that for us.

Not used

remove request->server_cs

Dynamically calculating the value removes lots of annoying boilerplate, and means it's always accurate

Typo

Use internal times for the state tree

Don't has the current server cs, use a unique id provided by the process module.

Remove the majority of call keyword logic

Have everything else use virtual_server_push to push virtual servers onto the stack

Fix Auth-Type Accept/Reject

process_radius - Use server_cs pointer from bootstrap instead of the request

move code to common function and macros

This makes most of the "process protocol" state machines to be
little more than "define data structures, and call generic functions"

let's make this more generic

typo

Notfound should be a reject

Fixup RADIUS enums/macros and radius_process type names

Fix bad logic, and don't free the subrequests

hacky hack

remove unused functions

Packet-Type is always a uint32_t...

this doesn't make a lot of difference, but on some systems aligned
access is better.

Typo

Only say we're entering a section when we have a section to enter

Compile authenticate sections

Set the reply rcode _THEN_ update the state pointer

search for auth_Type in the correct list

Typo

Return an error code when we can't find an unlang_group_t associated with a CS

Integrate packet type array into radius_state

Execute process modules using the unlang stack

Add trace macro

Add macro for checking packet code validity

arg alignment

quiet warnings

Push the process function directly

remove entry_point_set()

and instead just call the virtual server function

shut up compiler

call new the process functions

look at unlang action, instead of rcode

add temp unlang_virtual_server_push()

remove unused variable

create fake module instance for process functions

and start off real modules at number 1, so that the fake module
instances don't interact with real ones

start modules at 1, not 0

so that when we create fake modules for virtual s

first stab at radius processor

Use the standard string escaping functions when printing keys

Fix erroneous \0 byte being added to pool and range IDs

If we don't start with 1 connection the pool isn't mapped and certain commands fail

Include exuberant ctags configuration file

Parent the arg buffers correctly

Check for request

v4: Convert %{expr: } to new xlat api (#4042)

* Convert %{expr: } to new xlat api

* Remove incorrect "UNUSED"

ctx is actually used

Avoid allocating dummy requests in rlm_redis_ippool_tool

That now requires the dictionaries, and we really don't want to add that dependency...

Fix trigger args assert

Comment fix (#4040)

v4: Convert dict xlats to new api (#4039)

* Convert %(attr_by_num: ) to new xlat api

* Convert %(attr_by_oid: ) to new xlat api

* Convert %(vendor: ) to new xlat api

* Convert %(vendor_num: ) to new xlat api

* Convert %(attr: ) to new xlat api

* Convert %(attr_num: ) to new xlat api

v4: Convert %(client: ) to new xlat api (#4038)

* Convert %(client: ) to new xlat api

* Update documentation for %(client: ) xlat

* Update existing uses of %(client: ) xlat in raddb

* Update existing reference to %(client: ) in tests

* Add specific test for %(client: ) xlat

* Tidy client map test

* Fix docs

They were very wrong...

* Fix examples in config file too

Co-authored-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

v4: Define arguments for %(length: ) xlat (#4037)

* Define arguments for %(length: ) xlat

* Update tests that use %(length: ) xlat

* Update docs for %(length: ) xlat

* Make xlat_fmt_aprint() reflect original call syntax

* Make xlat_print reflect original call syntax

v4: Convert %(interpreter: ) to new xlat api (#4036)

* Convert %(interpreter: ) to new xlat api

* Update existing tests to use new %(interpreter: ) syntax

* Update documentation for %(interpreter: ) xlat

* Add test for %(interpreter: ) xlat

* Typo

Co-authored-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

v4: convert %(config :) to new xlat api (#4033)

* Convert %{config: } to new xlat api

* Update documentation for new %(config: ) xlat syntax

* Remove former config xlat escape function

* Add test for %(config: ) xlat

Amend xlat register in unit_test_attribute to new api (#4032)

v4: Convert eap-aka-sim xlats to new api (#4030)

* Convert %{aka_sim_id_method: } to new xlat api

* Update docs for new %(aka_sim_id_method: ) syntax

* Update tests for new %(aka_sim_id_method: ) syntax

* Convert %{aka_sim_id_type: } to new xlat api

* Update docs for new %(aka_sim_id_type: ) syntax

* Update tests for new %(aka_sim_id_type: ) syntax

* Convert %{3gpp_pseudonym_key_index: } xlat to new api

* Update docs for new %(3gpp_pseudonym_key_index: ) syntax

* Convert %(3gpp_pseudonym_decrypt: ) to new xlat api

* Amend documentation for new syntax of %(3gpp_pseudonym_decrypt: ) xlat

* Auto load eap_aka_sim enums for xlats

* Convert %(3gpp_pseudonym_encrypt: ) to new xlat api

* Update documentation to new %(3gpp_pseudonym_encrypt: ) syntax

* Update tests to use new %(3gpp_pseudonym_encrypt: ) syntax

* Move eap_aka_sim tests to eap_sim to match module name so they run.

* Tidy eap_sim test files

Refresh test certificates

v4: Define arguments for json xlats (#4027)

* Define args for json xlats

* Update json tests to match new generic failure message

* Missing arguments will mean no child

* Simplify json encode tests

* Remove input argument errors from json encode test and create error test

* Correct ERROR handling for module tests

* Simplify unlang in json eval tests

* Simplify unlang in json parser tests

* Simplify unlang in json quote tests

* Enable 64bit json eval tests

v4: Define arguments for redis xlats already using new api (#4028)

* Define args for %(redis_node: ) xlat

* Define arguments for %(redis_remap: ) xlat

* Minor formatting

Co-authored-by: Arran Cudbard-Bell <a.cudbardb@freeradius.org>

ICMP xlat should be defined arguments, not a stream (#4026)

Fix issue with delete command that caused entries to remain if the address had never been used

Linter: Perl fixes (#4019)