Don't allow fr_trunk_connection_requests_requeue to run on draining or draining_to_free

Don't assume anything about the tconn state

Kicks the can down the road for callers on trunk_connection_requests_requeue, but oh well

Use correct enum value

Add asserts around connection state

s/trunk/pool  - Should use the same config option for both pools and trunks (#3297)

s/trunk/pool  - Should use the same config option for both pools and trunks

Singularise

Deal with req_cleanup_delay == 0

Allow fr_dlist_entry_in_list to work with "uninitialised" (but zeroed) list headers

Cleanup event code slightly

Check if timer is in pending list

Enable corruption/type checking for event.c dlsts

update cursor comments

Remove extraneous call to check_for_zombie, have check_for_zombie take tconn instead of h

s/udp_rcode_t/udp_result_t/ otherwise we have rcode->rcode everywhere which looks really dumb

Fix some magic numbers

revert until we find out why its failing

this change appears to yield loops in pair lists?

one last check for is_root

check for is_root here, too

use da->dict, to avoid talloc_parent

Slightly more meaningful goto

Do all state signalling at the same call level

Allow bound requests to survive requeue

remove unnecessary code

just use dlists...

be more accepting of corner cases

remove from ev_to_add on free, if necessary

Shuffle function order

Add public function to allow enqueuing on specific connections

Allow requests to be requeued on an "inactive" connection.

Make type name agree with enum values

Enums before structs

Update the auto-generated .adoc files

errors can be at end of string

remove redunant declaration

set the FD to be readable when we're doing status checks

status server checks should be sent more often

we don't need to wrong times

we don't need 9 digits of accuracy for max_request_time messages

we don't need to show how "cleanup - now" time.  That's the time
we over-ran max_request_time.  The user doesn't know what that
means.

don't free rctx, it will die when the request dies

more cleanups and corner cases

fix checks for zombie period

there are many possible failure conditions, so we have to check
for them all, and handle them all.

don't add multiples of the same attributes

Ignore leaks in getaddrinfo() at /usr/lib/system/libsystem_info.dylib (#3292)

The leak is happening only on OSX >= Catalina + Clang8.

Don't need accessor anymore

make enums public, so callers can use them

fewer calls to fr_time()

cache "last updated" time

Backport dictionary changes from v3.0.x (#3276)

Add Arista VSA 6-9

Update dictionary.wimax attributes (#3283)

It was based on the image "[0105] FIG. 9" in https://patents.google.com/patent/WO2009155120A2/en

Backport from v3.0.x #3282

document revive_interval

add revive_interval

one last one

Fix WiMAX-MSK attribute

sometimes the scanner is smart, too!

cleanups and try to shut up scanner

make status checks work

abstract ID clearing for simplicity

be better about zombies

use treq->rctx

crash less often at exit

check for min before check for no requests

cleanups so it doesn't crash.

move "rcode" to udp_rctx_t
point to rctx from preq, otherwise it's impossible to update
rcode

just use STRINGIFY, and revert previous changes

more checks for zombie connections

point to the correct field

just use "4" instead of more magic.  Fixes #3277

update and document

clean up

better checks for zombie period

don't start if there's been a long delay since we last sent
a packet.

And if we haven't seen any replies, start zombie_period from
when we last sent a packet.

move zombie checks to request_timer

so that they're only run when a request times out

Fix cancellation function to do the right think on requeue

Remove treq and request from the udp_request

Fix parenting of the udp_request.  Should be parented from the treq because its lifetime is bound to the treq, not to the REQUEST *.

It doesn't _really_ matter here, but it does mean we can take advantage of the talloc_pool/freelist in the trunk.

Where it does matter is for trunk protocols which do cancellations, as the REQUEST * and the rctx might go away, but the preq needs to live on so we can cancel any upstream queries.

Expose the tconn's underlying conn

Not needed

Make additional fields in conns and treqs public

and another one...

Fix more NULL issues

Sprinkle nonnull

Remove duplicate dst_ipaddr/dst_port fields

Typo

Print the connection name on failure

Use the provided el in more places

One more to go....

Fixup I/O functions to use the tconn

Yes it makes the error case more complex, oh well.

Get the conn from the tconn

Add argument to test muxer

Pass el to the muxer so that it can set timers.

update format of Proxy-State

4 octets of "my" random number
1 octet of how many of "my" Proxy-State we saw

this lets us know not only the admin configured a proxy loop,
but lets the module be doubly sure that it's a loop

remove unused variable

remove unused variables

move zombie checks to request_fail

move to use udp_connection_t

as udp_handle_t may change over the lifetime of the request

Fix clang scan issues in the trunk code

Now used

Remove unused macro

Add noreturn in acutest.h

Fix incompatible conversion (#3274)

The udp_request_free() returns integer.

Fix typo with 'uctx' (#3275)

fr_event_timer_delete doesn't need to take el

We should probably figure out something for the file descriptors too

Use el where already available

massive changes to switch to trunk API

* basic send / receive works
* configuration works, but needs to be cleanup up and normalized
  * the mods-enabled/radius default config hasn't yet been updated
* zombie checks should work
* status-server should work

* it does not yet track "open to fail" states.  i.e. when it has
  multiple open / active connections that all fail, it should
  refuse to open new connections until one connection responds
  to status-server.

use uint128.h