Restore previous state

Follow the normal state transition if we have a ->failed callback and it returns FR_CONNECTION_STATE_INIT

remove it from the thread queue, too

remove packet from c->queued on timeout or reply

don't access u->rr if it doesn't exist

print time after the timer has been initialized

access u->timer directly

notes for todo

Don't need this in _conn_failed as _conn_close is always called

remove connection from list / heap

don't use one-character variable names

or prefixes.  or _p where it's not necessary

simplify some logic to make it clearer, and update comments
to describe what the code is doing

formatting

so that it's readable in a terminal

fix documentation

more gluing worker into proto_detail

if "transport != work", then we load the worker, as the various
other modules are intended to use it, too

more preparations for auto-create detail worker

Package proto_detail_work

added proto_detail_file

and started on integrating it.  But it looks like NOTE_EXTEND
for directories isn't supported on OSX.  <sigh>

NOTE_EXTEND can be applied to directories

include type name in message

instance isn't const

add fr_schedule_directory_add()

rename proto_detail_file to proto_detail_work

in preparation for new proto_detail_file

move common "work" header to proto_detail header

ifdef out unused function

add support for unmounted file systems

fix comments

add "track_progress" configuration

minor cleanups

Merge pull request #2084 from herwinw/allow_expired_crl_docs

Added some docs for allow_expired_crl

Merge pull request #2085 from herwinw/docs_2.1.10_and_later

Removed references to version 2.1.10 in the config docs

Removed references to version 2.1.10 in the config docs

That version is years out of date, there's no added value in keeping
references to that version in the docs.

Added some docs for allow_expired_crl

As mentioned on the mailing list today: omission might confuse people

fix assertion for uninitialized variable

GCC definitions for sanitize-address

keep timers around even when removing packets from a connection

so that the timers keep running, and the packets are retransmitted
when the timers hit.

This shouldn't be an issue normally. If a new connection opens,
any pending t->queued are drained, and sent immediately. BUT
there may be some pending packets in the queue which didn't make
it to the connection.  So we have to handle that case, too.

If there is no connection, any timed-out packets are sent only when
a connection is available.  This means that the timers aren't
perfect in that case, but there's no perfect situation here.

add thread to udp_request_t

move retry into udp_request_t

so we can access it when there's no connection

use talloc_ctx, as it doesn't need the actual ID struct

clear timers and remove (now wrong) assert

remove todo as it's done

re-initialize the status check packet on connection failed

and remove any pending writes, events, etc.

Add address-sanitizer 'feature' to version.c

...so we can check if the code was really built with support

doxygen

move retransmission timers to rlm_radius_udp_t

so that they operate across connections

doxygen

typo

set pending flag

reset connection state to INIT on failure

and allow active -> init states for reconnect

move packets back to thread queue, not connection queue

on connection failure.  the thread queue is for packets
without IDs allocated.  The connection queue is for packets
with IDs allocated

stop if we're at end of buffer

transition to FAILED on conneciton failure before reconnect

set last_search = 0

notes on dynamic clients

detail file packets are lower priority than packets from the network

Move most of conn_error to a conn_failed function to reconnections are handled properly

The connection state machine calls conn_failed on signal_reconnect() and when there's issues opening the connection.

Rename CONN_UNUSED to CONN_INIT

Add missing get_type_abort in conn_free

Don't double free status_u

Already freed by talloc_free_children

Use correct argument order for fr_pair_afrom_num

The VPs should always be added (not just at L_DBG_LVL_2)

Name arguments in rdebug* declarations

Callback functions have _<func>

Break long lines

Use correct doxygen

Make it clear the connection is now open

Don't leak memory on reconnection

Obligatory log message pass

Obligatory formatting pass

Add fr_asprintf

Allow RADIUS module calls in accounting/authorize

Specify path to config item

Dup the pool CONF_SECTION when connecting new nodes

CONF_SECTIONs can now only be parsed once so we get pools with no settings otherwise

Typo

Allow for node ID and other elements in cluster slots output

Switch to using WITH_SASL to indicate SASL support

Package proto_detail_file and proto_detail_process

Fix doxygen

Add support for toggling filters on/off without removing the callbacks or ef

Store the map used for function/filter mapping against the ef (less branchy)

Fix formatting in some places

Expose the union containing all the functions

Have fr_event_build_evset write out the active functions

Formatting

doxygen

Revert "add read_pause / read_continue"

This reverts commit 702f9a8b7c7546bdd9d3f49ce700050ff860fade.

Revert "use new read_pause / read_continue routines"

This reverts commit de5dd884c41db425896b83bb04c8bf9af48d812b.

Revert "continue reading only if we've processed all outstanding packets"

This reverts commit 8856d7a291cd5e83711fd1d30cca07225627fd2c.

Pass filter to fr_event_fd_delete and other places

set filter to find

Provide/embed RADIUSD_VERSION_RELEASE

set src/dst ip/port if available

notes on future work

const

debug messages aren't errors

remove unused assignments

fix code setting Done

by looking for \0 instead of \n, and allocating the tracking
entry only if we're processing this entry

size_t casts for off_t

continue reading only if we've processed all outstanding packets

to prevent overloading the system

use new read_pause / read_continue routines

add read_pause / read_continue

which allows the caller to pause reading from a socket,
and then continue reading from it.

With some magic that if there's no read AND write, the socket
is ignored

allow write() to return 0, which means "close the socket"

There may be a better way, but this works for now

add "priority" to detail listener config

and move it out of detail_file config

round up ring buffer size to the nearest power of 2

parse header && decode VPs