Always set repeatable if we have a repeat function

This means the start function will run as the stack is burned down

Don't check for operation failures in ldap_check_userobj_resume

Print list of attributes we're retrieving

Use correct field in map ctx

Don't do exclusivity check if we're checking the config

Output what group attribute we looked for

Make the query available immediately after resumption

Shouldn't default to returning ok in mod_authorize_resume

Add debugging functions to dump LDAP messages

Add sbuff array concat function

tmp;s failing should unwind the current frame

regexes should return fail if they can't find a group

Add the ability to print instruction priorities to stack_dump

Explicitly check for failure

add rctx into user_modify_resume if we're calling it directly

Use correct check for ldap_initialize success

Duplicate a bunch of code in unlang/function.c *sigh* and allow it to work when the function yields

Assert _before_ allowing talloc_get_type_abort to trip

Fix order of yield and search in  LDAP mod_modify

LDAP_OPT_X_TLS_PROTOCOL_TLS1_3 not available in every libldap...

Fix const

Fix minor issue in rlm_lua

p_result may be NULL

Add variant of functions which do not produce a result

Can't use stack variables for p_result

Only resolve priorities if the rcode is actually set

Frames always have an instruction

More p_result fixes

Allow p_result to be set for rlm_ldap_find_user_async

Switch ldap modifications to using process functions

Add sanity check in unlang_module_yield

Why?  Because if the yield is done _AFTER_ a function has been pushed onto the stack, we get undefined reuslts

Run the ldap.authorize method as module methods, not functions

Audit more uses of unlang_function_push

Results for call_env expansion are written to env_result

Notes on unlang_function_push calls in virtual servers

Nothing seems to use the rcode result from ldap_trunk_query_results, so remove the return codes as they're confusing

Return UNLANG_ACTION_FAIL from RETURN_UNLANG_FAIL

Explicitly store the result of edit get password operations

Explicitly store the result of bind operations in the calling frame in LDAP

Add note on using unlang_interpret_result to make unlang functions transparent

Make the eap_virtual_server_resume function transparent in terms of rcodes

Handshake resume function writes out results to eap_tls_session->state so we don't need to pass through the rcode

Make the logging destination resumption function transparent in terms of rcodes

Make the result pointer for the current frame available

s/unlang_interpret_result/unlang_interpret_rcode/

We discard the return codes from triggers because they're fully async

Add better notes on sigmasks

Check unlang_module_yield return code in ttls.c

Notes on how subrequests are handled in peap.c

peap: Cancel child request's frames

Pass unlang_result_t to modules instead of rlm_rcode_t

Populate this frame's result with the results of the resolutions

Fix scan issues

Use a dummy instruction in frame[0]

Allow the rcode from virtual servers to be redirected

Make discarding function rcodes official

Check we can find slapd

Fix arg order to unlang_subrequest_child_push

Only assert if the child request is runnable

Set priority of the process_reply function high enough to bubble up

Help tracing the crazyness

Yielded makes more sense than resumable

Ensure that if the child hasn't completed, then its parent must be yielded if it's signalled

Don't crash on request timeout

Use section result from rctx in eap_aka_sim state machine

Use unlang_map_yield for LDAP maps

Fix header include in dict.h

Remove useless calls to unlang_function_push and just use unlang_module_yield

Adding new stack frames is expensive, and there's no need to do this as we can just mutate the module's frame

Add better support for async maps

We really don't want to push additional frames if we can help it

No reason to use a function to represent module state when processing accounting queries

Reduce stack dumpage

No need to use functions to resume in the rlm_files module

try-no-sibling test needs to set the rcode back to ok

Now that rcode works correctly, having OR or AND fail when their children fail causes tests to fail

Have xlat push functions take a result_p

Print what's being expanded in stack_dump

Transactions need to operate on the result of the child section, not the current frame result

Rework interpreter to make handling of rcodes sane, and to discard rcodes entirely when a frame is popped

Use common function to print closing section debug

Correct calculation of overall result for parallel keyword

Look in the correct place for transaction results

Fix auto-added headers

Use new rcode location in catch

Not finding a capture group should not be fail, it's just a NULL result

Global timeout is the only thing that produces a timeout rcode outside of a module now

Remove RETURN_MODULE_TRANSPARENT

Don't overwrite frame state in dhcpv6 state machine

Discard rcodes from finally

Don't include ourselves...

Switch to using unlang_result_t for process functions, and unlang functions

Add macros to populate unlang_result_t

Rework state machines so that they operate consistently with new result structure

Fix pre-existing issue in DHCPv4 where the default section rcode for send sections would cause the packet type to switch to Do-Not-Response

Add unlang_result_t to hold both a priority and a rcode

Allow modules to specify the size and type of the rctx to allocate

This mostly saves on boilerplate at the top of every module call.

Pass arguments to push functions in a structure

Makes it easier to add or remove args

We don't return yield anymore

Start simplifying rcode handling

Use proper type for the rcode priority

Minor interpreter cleanups

No customers here...

Make the delay module transparent for rcodes

there is no 6th expansion, so we don't expand it