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

struct fr_dict should be fr_dict_s

Whitespace

and more "shut up static analyzer"

save reason for stopping in the fr_retry_t

more "quiet clang scan"

make clang scan happy

for some reason, it thinks that fr_fault_setup() sets its first
argument to NULL.

compiler warning cleanups

try this instead to make it build without OpenSSL

Revert "add more OpenSSL magic"

This reverts commit 801ae0311868e6e26688db856d432313aa56894e.

add more OpenSSL magic

always build radclient-ng

Move table search functions into table.h, and add new macros to return pointers to values

This lets us create one line search function definitions for custom array types

add radius-bio library

remove unnecessary defines

move bio to separate makefile

Add test for Relay-Agent-Information encoding at the end of a packet

This can be simplified now attribute lists are nested

If attr > 255 then it can't be 82

Don't print error when skipping Message-Type

Simplify and add comment

Correct query