assert that packets which have replies were actually sent

i.e. once we have a reply, the packet is no longer sent,
it's done.

set work_io_instance=NULL in file, not worker

because many things can cause the file to be deleted...

more cleanups on error, and deletion of detail.work

alloc a worker and run it

add decode & write

which bounces through to proto_detail_work, which does the
actual work.

We should probably have a way to pass the fr_listen_t to proto_detail,
so it can use it's app_io, instead of the one in the proto_detail
instance

free the worker if we're closing the FD

and if the flag tells us to

minor cleanups

add "self" for linking issues.
parent is not "const"
move fr_schedule_t to proto_detail_t, not proto_detail_work_t

and initialize the entries

clean up to allow proto_detail_file to call us

minor cleanups

filename.work is no longer mandatory.  If it isn't defined,
we'll just create it.

If detail.work does exist, watch for it's deletion

more than one NOTE can be returned

open RDWR if tracking, else RDONLY

clear out el->events[] for an FD when the event is deleted

use -, not a-b

don't use t->pending, just rely on heap size

if there are two things showing the same information, it's possible
for them to get out of sync.  So... one way to rule them all.

set t->pending only if there's pending packets

handle 'close' as a separate function

so we can catch it, and inform our parent instance that it
needs to rescan the directory

instance isn't 'const' in close

auto-create "work" configuration

to simplify the users life

note recent changes

/ may occur in dictionary names, too

so we can't put it into the comments in the file

cast to long double

%f is double

cast to double

simplify the translation rules

port relay fixes from 3.0

sort by FD first, not filter.

there are really only 2 filters used, but multiple FDs

don't free children in parallel state destructor

they're already either parented from the request, or linked
to the request via request_data_add.  And will therefore be
freed as necessary.

Plus, if they're detached, we definitely DON'T want to free them
when the parallel section exits

use correct enum value

parallel clones by default

but can be empty via 'parallel empty'

which likely needs a better name

free children on DONE signal

delete the reply after receiving it

connection may be NULL

remove unused variable

dereference correct variable

make ourselves active if we read and there's pending packets

this is likely imperfect, but it's not entirely wrong

double-check idle handling

more cleanups and sanity checks

go over status-server retransmission checks

go over zombie timeout checks

and more cleanups

the basics work.  But (again) the corner cases need to be checked.

more cleanups

start of cleanup and change to better state transitions

Merge pull request #2091 from bmork/unisphere-vsas-for-v4.0.x

add new Juniper/Unisphere/ERX VSAs

remove active_requests

we can calculate it from the queued / sent heaps

travis fix

clean up connection state on failure

always add Event-Timestampt to status packets

and always update it when sending those packets

don't update Acct-Delay-Time for status check packets

enforce limits on retransmissions

and make status check packets use the same retransmission path
as normal packets

use local retry variable

we can send reply to network after max_request_time

do proper state changes on retransmit

add new Juniper/Unisphere/ERX VSAs

Importing new attributes from
https://www.juniper.net/documentation/software/junos/junos161/radius-dictionary/unisphereDictionary_for_JUNOS_v16-1.dct
using the exact attribute names Juniper use, following the precedence set
by the original conversion from dictionary.erx to dictionary.unisphere

Yes, Juniper *do* use four different attributes prefixes here:
 Unisphere, Pcp (sic), Jnpr and ERX(!)

Quite impressive.  And quite a mess.  But at least it's not *our* mess
if we just got with their names...

Signed-off-by: Bjørn Mork <bjorn@mork.no>

Return -1 from fr_event_filter_insert if fr_event_build_evset fails (instead of going into an infinite loop)

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