track active packets in a connection

add explicit STATE to udp_request_t

and other minor fixes:

- use allowed_replies[code], not code2rcode[code]
- ensure that all packets are in the right list
- switch off of state to do packet list enqueue / dequeue

re-initialize fields after re-open

don't decrement num_requests on status_server

more assertions and cleanups

clean up more request stuff, and add more assertions

Table used outside of when WITH_EVENT_DEBUG is defined now

Use kevent_filter_table

ifdef out array for EVENT_DEBUG

%f is double, so an explicit conversion is useful

as a hint to the compiler that we know what we're doing

EVFILT_EXCEPT not defined by libkqueue

add extra logging to event.c

Merge pull request #2089 from alanbuxey/patch-4

update text (typo and grammar)

Merge pull request #2088 from jsynack/oct7updates

Misc updates

Don't close out the connection just signal it to reconnect if we can insert the FD

There are too many corner cases with freeing the connection within fd_active and fd_idle... They don't for example communicate to the caller that the connection is no longer valid.

update text (typo and grammar)

Misc Non-code impacting changes to docs

1. Start to sprinkle https vs. http to encourage a safer web.
Some of this had URLs redone to the wiki as the SSL redirect from freeradius.org
doesn't appear to be consistent. If this is desired then there are more.

2. Misc corrections to links that had become outdated or stale.

3. A few filename renames ( -> .md files)

4. Remove reference to google search box in the mailing list - couldnt find it?

5. Instead of talking about the age of the book - just give the publication date.

6. A few extra bash highlight-isms

Normally I would not batch so much into one commit - but these are all non-code impacting.

Random adjustments and low-hanging fruit

FreeADIUS -> FreeRADIUS :)

Add Foundry-COA-Command (overlaps with existing)

Vendor not used

Move dict xlats into rlm_dict

Remove vendor specification from fr_dict_attr_by_oid as vendors are structural attributes now and it's not required

Revert "Fix a couple of log messages"

This reverts commit eb5bce9fd8648e682ecd3cb761d152f2a90bad30.

Fix a couple of log messages

Populate request packet timestamp

More cleanups in udp_request_free

only call timer delete if we have a timer

don't molest ev

add time values to debug message

set *ev->parent = NULL in the talloc destructor

so that the callers expectations are held true... that we can
just talloc_free() the pointer, and expect everything to be
cleaned up.

print when retransmit timers hit, even if there's no connection

assert that the parent is pointing to us

move timer insert to udp function

more debugging and "finished" flag

enable RADIUS module by default

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