Update contributing.md

reject delays are from when we received the packet.

Which should be a bit better behaved if we're proxying, and the
proxy takes forever to respond.  We don't add delays, we just
ensure that rejects are sent no sooner than (say) 1s after
the request was received

more checks for is_root / vendor as per previous commit

find_by_num is only good for top-level attributes.

If you need child, call the the other functions.

be more aggressive about checking for user name/password

when updating maps.  We don't want to catch VSAs, or other
attributes...

more docs

It would be nice if we told people how to patch Net::RADIUS

verify request->username && password, too

fail if we can't start time

simplify the time calculation to remove one branch

added note

Fix conflicting types on FreeBSD

remove TLS initialization.  We don't need it for MD5

Added simple RADIUS server.

Which has hand-coded check / ack routines.

EVFILT_USER / READ / WRITE are in filter, not in flags

more debug

we need room for num_workers messages in the control plane, too

travis

remove fr_worker_control_plane()

fr_worker_channel_create() should be used instead

Include NAKs, which are required for the channels to work

If the worker doesn't NAK a request, the master has no idea
that the request has been dropped.

This NAK could arguably be a control-plane message in the
channel, with a helper function.

don't count RTT for NAKs

which have processing_time = 0

use #ifdef WITH_AUTH_WINBIND

fix condition

Allow authentication retry in winbind

A setup with the following properties:

  * Active Directory backend
  * FreeRadius with eap-inner-proxy
  * Windows client with single sign-on
  * User using different casing in username than in backend

may result in failing connections. It looks like Windows reads the
correct username from the domain server once it has logged in, and uses
that to create the MS-CHAP2-Response attribute. The User-Name attribute
is still the one with the incorrect casing, causing the authentication
to fail.

The introduced config option kicks in after a failed authentication: it
reads the correct username from the backend, tries another
authentication, and uses the found User-Name to calculate
MS-CHAP2-Response if the second authentication works.

rlm_mschap: *inst is const

typo

added <signal.h>

rearrange output

set initial RTT

remove unused macro

more debug stats

use correct order for event status

fix RTT calculation

Added worker test

Which seems to hang often.  <sigh>  But recompilation makes
the hang go away!

show NOOP, too

More debugging

more debugging

count the number of requests we received

simplify debug

simplify debug

add number to request

added master ctx set/get

Minor things missed in previous commits

debug messages

initialize array sizes

break out of the loop when we've found an entry

pass paramers to el->status() in the right order

add worker_channel_create

and assume that decode() is given a REQUEST, instead of
allocating it itself

clarify text

doxygen

added worker debug function

Added debug function

don't look at the other end's ack

look at our view of their ack

typo

<sigh>  clang scan isn't quite smart enough yet

see if this shuts up travis

Exit with an error code reflecting the signal

set CC_HINT(nonnull) on all cursor functions

mark hot functions as hot.

_fr_cursor_init in particular gets flagged by profiling programs as consuming lots of CPU time.

Various doxygen and cursor code fixes

Add new generic cursor functions

s/cursor/pair_cursor/g

It'll be changed back in a future commit... maybe

travis

allocate reply message in worker

add worker context to channel

Because the network signals the worker that a channel is ready,
but the worker needs additional information around that channel

And it's inefficient for the worker to root through it's list
of channels, trying to find a context

set channel to be unused in close

rename function to be more consistent

remove old todo which has since been done

use talloc_pooled_object

Service the control plane on EVFILT_USER signal

which means that the worker should now (mostly) work with new
channels and closing old ones.

add evfilt_user callback, and glue it into the event loop

allocate channel array

docs

yielded requests go to the head of the list

add get control plane AQ

doxygen

add comments explaining functional requirements

more cleanups.

The only place to decode packets is in the main loop.
We have a heap for runnable requests, ordered by (priority, time)
We have one dlist for requests, insert into head, cleanup by tail

added FR_DLIST_INSERT_HEAD

resumable requests have priority over runnable ones.

And don't insert them back into the runnable list.

clean up worker.

Move "to_decode", etc. to struct containing heap + dlist,
so that the "pop" cane be done by priority, but "get oldest"
is done by time order.

Add template unlange_resume() function so that we know what
it's supposed to do.

modify transport APIs a bit based on current design.

Added FR_INSERT_TAIL_PTR

added FR_DLIST_TAIL to get the tail of the list

Fix asserts in ippool_tool

Package client

fix API

clarify names for channel ends

fix return code

travis fixes

move to control-plane signaling for channels

more debug messages.

Should also make num_messages configurable

add debug messages

suppress more signals from the worker, too

Correct logic for master -> worker signals

Which results in the suppression of many more signals

increase size of control-plane queue based on max_outstanding

Expose control-plane push/pop

count resignals

make fr_channel_service_aq() public

remove bad fprintf

use correct ACK

unify public and internal signals

create fr_channel_service_aq()

in preparation for control-plane messages