rlm_radius_udp: Move common decode logic into its own function in preparation for adding connection based status checks

rlm_radius_udp: Move allowed replies array

rlm_radius_udp: Produce log messages on write errors

Ooops, don't know how these were removed

radius: More decode error codes

rlm_radius_udp: Move conn_init

trunk: Add last_read_success

trunk: Unused field

trunk: Formatting

trunk: Split FULL and INACTIVE states

trunk: Add back previous functionality to allow requests to be assigned to the backlog if the last event was a connection failure

trunk: Prevent assert when req_cleanup_delay == 0

rlm_radius: Test plan zombie state

rlm_radius: Start of test plan for rlm_radius (incomplete)

rlm_radius_udp: Update comments

rlm_radius: status_check singular not plural

rlm_radius_udp: Handle more error codes on failed write

rlm_radius: Fix sanity check order

rlm_chap: Add tests for rlm_chap

rlm_chap: Remove duplicate message detailing the password source

Add %{chap_password:} for producing values for CHAP-Password

This is mostly for testing rlm_radius

radius: Simplify fr_radius_encode_chap_password and have it accept an input buffer for the password instead of the VP

global: Use assert not error in MEM()

Means we don't have to provide argumets used by FR_LOG_PREFIX whenever MEM() is used

there's no User-Name in a fake packet we just created

add API / wrapper fro fr_pair_group_...() functions

so that we can do bulk conversions of the callers *without*
breaking anything.

i.e. convert the callers to use the new API, which (for now)
is just a macro wrapper around the old API.  Then, once the callers
use the new API format, we can delete the macros, and actually use
the real new API functionality.

fix use of fake / request to be consistent

use vp_group for groups

use vp_group for group access

remove unnecessary whitespace

Merge pull request #3298 from jirutka/cui-pgsql

Fix and improve cui/postgresql

specify name for pkey constraint and use it in query for cui postgresql

use real upsert (ON CONFLICT) instead of workaround in postgresql cui

Clause ON CONFLICT performs better than the RULE workaround and it's
guaranted to be atomic (the workaround is not).

It requires PostgreSQL 9.5 or newer; this version was released in 2016
and the previous versions are already unsupported, so it shouldn't be
a problem for anyone using latest FreeRADIUS versions.

There's a problem with backward compatibility because ON CONFLICT
doesn't work when there's a RULE ON INSERT defined on the table. It
results in the following error in radius.log:

    Error: rlm_sql_postgresql: 0A000: FEATURE NOT SUPPORTED

fix postgresql schema for cui

I'm getting the following error with the current schema:

    rlm_sql_postgresql: 22001: STRING DATA RIGHT TRUNCATION

Moreover, in PostgreSQL the text type is highly preferred above
varchar. Schema for radacct already uses it, so it seems that schema
for cui is just outdated.

I also remove quotes around `now()`, it was probably some archaism.

fix formatting in cui postgresql schema to be consistent

use usec since current second, not usec since epoch

fix crash when no previous regex seen

doxygen

Remove vp_cursor_t from rlm_radius_udp.c

s/pair_cursor/fr_cursor/ in rlm_isc_dhcp.c

More reasonable default

Add more log prefixes.  Switch some statements to debug/info.

We really do want to see rechability messages in the main server log.

Typo

Remove duplicate track delete call

Assert treq is still assigned to a connection on timeout

Pass in tconn to status_check_next

The timer is now parented of h, so this should all be ok...

Signalling request is complete frees everything parented by treq, so probably want to do it after we're done messing around with it

Remove udp_connection_t;

use more standard names "packet" and "reply"

use more standard name "packet"

use more standard name "packet" and "reply"

in preparation for moving to type "group" for lists.

use more standard name "packet"

in preparation for moving to type "group" for lists.

use more standard name "packet" instead of "current"

in preparation for moving to type "group" for lists.

fix crash on invalid attribute

use path count

remove "set fd handlers" hack now that status checks are in the trunk

Past tense as it's after sendmmsg

Not needed, already NULL

Use sendmmsg in rlm_radius_udp.c

Fail requests which can't be requeued

add support for glob.  Fixes #1018

This needs documentation added, too.

move FAILED to timeout function

doxygen (xlat functions)

move status checks to using the trunk API

prioritize status checks

in preparation for making them part of the trunk

Function names and param order

xlat doxygen

Re-order xlat_builtin functions, whitespace

Remove invalid assert

Move closefrom into missing

Add crappy userland sendmmsg implementation

Explicitly free rctx on cancel

Re-order more functions, and explicitly clean up more things

Re-order functions

Allow laziness on the part of the API client, and free any treq children

set treq

Always populate preq and rctx

More asserts

Return the treq to the free list

Simplify populating treq_out

Handle isn't available if the connection failed from the init state

Assert that the treq is a member of a trunk

Always delete the request timeout

More agressive defaults

Cleanup failed requests...

Fix whitespace issues

Not an assert, a useful hint...

Fix assert

Only apply last_above_target delay if we have at least one active connection

Record reconnections (not due to expiry) as failures too

quotes around names...

Only start the response timer if we manage to write out a packet

Agressively reconnect connections on write errors

Remove icky O(n) udata hack

Use 'noop' callback for unset function pointers.  Fixes SEGV when filters are swapped out between corral and service.

Minor cleanup in event.c

clean up status checks

set 'verify=lib' in test dictionary for RADIUS

and change check order for loading protocol libraries.
Try to load them, even if they're not required.  And error out
only if they fail to load AND they're required.

This change makes it a bit more forgiving.

free module_name on error

replicate now takes a different path

Always set te->tt

Not reading from the socket is an error...