We don't need the length passed to dict_fixup_clone

Remove double declaration of KDF

Have the tests load their own custom dictionaries

Formatting and minor fixups in unit_test_attribute

Fix obvious, yet hidden, numbering error in dictionary.h3c

Alias Vendor-Discrimination-Str to Ethernet-Interface... they're both octets types

Fix obvious error in Site-Specific numbered attributes

...which was hidden previously by the fact we allowed numbering conflicts

For why not can sentence sense make

WS

internal encoder: ACTUALLY skip name only attributes, instead of just infinitely looping

If the input arguments were invalid... error out, don't just fail silently

eap-aka: Use enumv values from KDF instead of definitions

timeouts are perhaps not quite errors

MRC of 1 is really MRD

i.e. a simple duration, and not a repeated count

Better handling of SQL rcodes in xlats

If RLM_SQL_ALT_QUERY is treated as an error, then redundant SQL xlats
will try the same query again on each database host (with the same
result).

To handle switching to a different query if an INSERT fails due to key
contraints, then the return value of the xlat, which is the number of
affected rows should be checked instead.

No data returned is not a failure

Update MS SQL default schema

When using FreeTDS driver, `char` fields are returned with trailing
spaces, which causes "=" in the [op] field to become "= " - which fails
to parse.
Using `varchar` ensures only the real data length is returned.

Align rlm_sql_freetds NULL behaviour with other drivers

Remove unused function

FreeTDS doesn't return the number of rows from a SELECT

Update rlm_sql_freetds to use trunk connections

Unfortunately FreeTDS doesn't have the Async support that the Sybase
client library does.

Remove needless function

Add basic test of redundant SQL modules

Closed SQL connections can't be used for escaping

The handle gets freed when the connection is closed

Set return type for escape functions to ssize_t

So failures can be reported back

Move tests to be used by all SQL flavours

Set safe_for child xlats inside a redundant xlat

Correct comparison for end of retry period

Revert "Don't load the RADIUS dictionary in the xlat code"

This reverts commit 6cf9cd8db3a700b1ceb056cc14a0d03201966ed8.

Breaks CI tests of foreign protocol decoding

Use mod_accounting in send Accounting-Response

There's very little point in logging accouning responses in detail
files, and the sample config calls "detail" in the send
Accounting-Response section.

Add test of NULL SQL value handling

Align Firebird results handling with that of other database drivers

Handle NULL return values in PostgreSQL

Handle NULL return values in MySQL

Handle NULL fields returned in unixodbc

ODBC uses the data length indicator to report NULL return values.

Protect against NULL return values from SQL queries

Won't happen with the default schemas, but always possible with custom
schemas

Handle NULL returns from SQL when parsing map values

add this, too

this is static

remove call to write_flush

we don't need write_flush

add and use explicit verify_ctx for BIO

so that bio->ctx can always be application data

Don't load the RADIUS dictionary in the xlat code

We're not a RADIUS server... and this appears to be unused

Compiler is correct... if bio is nonnull, this should be do {} while ()

Support resetting the dictionary root

Don't automatically mark up defines as internal, as this breaks the DER encoder

Trunk "request specific" settings go in request, not requests

rename "activate" to "connected"

to match the connection / trunk API

as taken from PR #5428, with edits for v4

add fr_bio_flush() API.

which returns whether or not the BIO was flushed.

Ideally the write callbacks will also be signalled if the write
was blocked, too.

allow RADIUS clients without retries

document more authorization replies

add API to allow signals to run retries

i.e. on DUP signal.  We don't want to have a separate code path,
we just want to run the normal retry mechanism.

remove unlang_module_timeout_add()

and clean up the API for module timeouts / retries

add unlang_module_yield_to_retry()

hoist module timeout into main module state

the yielded ptr is associated with the stack frame, not the request

more cleanups on delete timers

parent the timeout from the frame state

so it is automatically cleaned up when the module exits.

remove unused code

request_ctx is a macro used by the server for other things

cap "next" at "end".

So users of the API don't have to manually check "end", they can
just look at "next"

rename to be clearer

and add comments about what the functions do

ensure that the retry struct is always initialized

before calling the sent() function

add tracking for timers to the radius client bio

Correct comparison of module xlats

Add Firebird mods-config to RHEL packaging

Firebird is not compatible with MS SQL queries

It is a fork of InterBase rather Sybase

Remove vague comment

Report what we're connecting to

Set Firebird connection timeout

Firebird can detect key conflicts

Update rlm_sql_firebird to use trunk connections

Allow for Firebird UPDATE ... RETURNING behaviour

when no records are updated, UPDATE ... RETURNING returns a row with all
fields set to NULL.

Set new allocated results size after allocation

Don't print results in fixed width fields

No other database drivers do this - and "     1" != "1"

SQL driver specific config is driver not dialect related

Firebird doesn't return the row count for SELECT

Formatting tidy ups

Do clearup in finish_query

Rather than when preparing the next query

No need for a mutex

Exclusive use of a connection is handled by pool / trunk connection
code.

Use talloc rather than malloc in rlm_sql_firebird

Don't call query finish when not needed

sqlippool_result_process is already calling query finish

Keep a list of all files we saw during loading the dictionary

This will be used later to record the files attributes were defined in, with buffer lifetimes that are equal to the attribute lifetimes

Field not used

Corrections to new table macros

Minor error message fixes

Typo (CID #1632387)

Add a default for query_timeout

Ensures we have something set for unixodbc where this is used in
calculating the upper bound for polling intervals

Only set rcode if it's currently OK

Don't attempt to close connections which never opened

Update rlm_sql_unixodbc to use trunk connections and async where possible

Async support is driver specific, so some will run synchronous and some
async.

Tested with:
 - Microsoft ODBC Driver 18 for SQL Server
 - FreeTDS driver (Sybase/MS SQL)
 - MariaDB Connector/ODBC(Unicode)
 - MySQL ODBC 9.0 Unicode Driver
 - PostgreSQL Unicode

of which only Microsoft ODBC provided async support.

Remove unused argument

Fix up MS SQL ippool queries

Make field name comparisons case insensitive for SQL maps

Correct group attribute queries

Use macros to reduce boilerplate

Use parse rules when parsing call_env data

Otherwise we don't properly unescape fixed values (e.g. \" in a double quoted string remains \" rather than becoming ")

Remove level of annoying indirection and copying things from the fr_dict_protocol_t to the dict

From comments this seems mostly to avoid cleaning up all the references to default_type_size and subtype_table.

Load the protocol symbol directly using dlsym, as this'll run every time the library is referenced and not just the first time.

Allocate a dictionary attribute extension for protocol specific data if requested to by the fr_dict_protocol_t

Use common talloc_bstr_tolower function