typos and word smithing

be nice to people

create proper nested ctx

No need for an intermediary variable

Always use expanded "if" condition

Makes debug output correct when variables are referenced in conditions.

add missing return

I wish to write an "Ode to C compilers".  It will be composed
moslty of curse words.

it helps to define the group attribute

as paircmp no longer does that for us

convert Unix-Group to xlat

pass in correct value

Why won't the local compiler complain?

remove old paircmp, and update docs to match

use real names for counter start / end

convert horrific %b and %e to real attributes

typo

add and document cache_groups

remove paircmp() callback for SQL-Group

Fix remaining uninitialized scalar values (CID #1503958, #1504020) (#5150)

The latter issue was interesting; the dbuff is set to use ether.addr,
but fr_value_box_ethernet_addr() is passed &ether, which looks like
it will put random garbage in the value box until you notice that
the address is the only member of the type. We'll see whether coverity
considers (fr_ethernet_t * const) fr_dbuff_start(&dbuff) a dangerous
downcast (whatever that means in C) and still complains. I hope not,
because the only reason that comes to mind for it is alignment issues,
which shouldn't happen here.

Change fr_vlog_perror() behavior in what might be an edge case (#5153)

As things stand, if the error stack is non-empty and fmt is null,
the copy of f_rules->first_prefix to sbuff won't make any
difference.

Test LDAP xlat and map using ldapi:// scheme

Use ldap_url_desc2str to canonify LDAP host URI

Gives consistent host URI strings for looking up trunk connections.

Also appears to catch some invalid URIs which ldap_url_parse does not -
e.g. ldap://%2Ftmp%2fldap/...

Add LDAP test using ldapi:// scheme

Set LDAP servers start to 0 in tests

Reduces noise in logs as only the relevant server connection starts

Add LDAP test using SASL for user auth binds

ldap_debug is no longer in module config

Correctly parent value boxes when concatenating

Correctly escape LDAP uri in map

Tidy up

Add regex to allow ldapi:// peercred admin binds in tests

Allow Symas LDAP packages for test server and add ldapi:// listener

Rework LDAP xlat timeout / cancel callbacks

To handle the case where the request has returned but the query has not
yet resumed.

In this case there is no outstanding request to cancel.

Use boolean enum in ldap group membership xlat

To be consistent with SQL group membership xlat

Rename fr_dhcv4_raw_packet_recv() as fr_dhcpv4_raw_packet_recv()

Correct tests where xlat outputs boolean

Pay attention to return value of fr_trunk_request_enqueue

Re-work failure paths to ensure correct cleanups

typo

we don't need to rename the SQL-Group attribute

the register function automatically adds our instance name

respect group_attribute and update it to sql.group

print error and fail if registration fails

remove logintime module

and all references to it, and the attributes it uses

add rule to make adoc file from configuration

move SQL-Group ==... to %{sql.group:...}

no escaping is done, so we don't need func _or_ uctx

Use a dummy escape function to satisfy xlat arg validation

add %{sql.group:name}

to replace SQL-Group == ...

which was misleading because it didn't allow !=, or =~, etc.

move group check to stand-alone function

in preparation for adding %{sql.group:name}

warn on not removing immutable attribute

Fix missing \n (#5155)

Make the configure scripts work better on macOS

cleanups as per unit tests

force these flags, too

don't sort if the list is already flat.

That breaks things due to key fields + child structs

make sorting a bit better

like attributes are now sorted together

also print out offset if the input is long

show which attribute had the issue

Have struct encoder return the PAIR_ENCODE_FATAL_ERROR constant

ldap: Typo

xlat: Fix xlat alternations that use functions

ldap: Print why URI parsing failed

let's make tests pass

+= also means "create if it doesn't exist"

at least for operations where "+=" is meaningful

We probably want to do something similar for union, merge, etc.
if the LHS doesn't exist.

update for new behavior of :=

hoist checks to be more consistent

update behavior of :=

if RHS expansion fails, it still nukes all of the LHS

Reassure coverity (CID #1504052) (#5152)

Non-group structural type attributes may always have a namespace
hash table, but coverity can't figure that out.

force flatten unless migration flags are set

this means we can safely update all of the decoders to create
nested attributes

add flatten migration configuration

it turns out "unflatten" is hard, so we need to find a better way
to deal with things

print out more text around mismatch so we can better find it

add unflatten_after_encode for test API

handle key fields in unflatten

if there's no "next", then we don't need a comma

use correct name

clean up and comments

notes on immutable children of temporary attibutes

allow operations on list which has some immutable values

use vp_ name for immutable field

we can copy immutable values, and the copy is mutable

add undocumented immutable xlat, and test based on it

add immutable flag, and check it most places

the main purpose of immutable flags is to prevent users from
modifying values that the server wants to keep around.

As a result, the main checks for immutable values are in the
various editing routines:

    src/lib/util/calc.c - cannot store results to immutable leaves
    src/lib/util/edit.c - cannot edit immutable values
  or delete immutable leaves from lists
    src/lib/unlang/edit.c - cannot store to immutable leaves

Many of the internal value box / pair APIs will ignore the
immutable flag, and happily over-write values.  This behavior
is likely good enough for now.  The intention is to stop admins
from doing stupid things, and not to prevent the internal code
from doing what makes sense.

There is currently no flag in structural VPs which says "contains
an immutable child".  The edit code therefore has to check each
time by walking the list recursively.  That's fine for now.

There is as yet no code to set the immutable flag, or unit tests.
The goal is to have the decoders set the immutable flag as necessary,
which means that they don't need to save / restore attributes with
special meaning.

hoist the secret flag on fr_pair_value_copy() too

copy tests from keywords/xlat-dhcpv4

migrate more tests to new methods

remove unused definitions

Get rid of remaining unchecked return value (CID #1533664, #1524617)

Tidy up on connection close

Switch from ntohs() to fr_nbo_to_uint16() (CID #1243443)

Takes advantage of coverity thinking fr_nbo_to_foo() taints
the passed pointer, which here is it's only use, while the
returned value is used several times.

Check returns of [sd]buff write functions (CIDs listed below)

CIDs: 1624616, 1524609, 1524611, 1524618, 1524615

Revert "Remove closed connections from the parent's tracking list"

This reverts commit bac1129ed8dab05a976fd97cbcd6df98276104ff.

Remove closed connections from the parent's tracking list

various coverity fixes

allow &leaf += { list }

which applies all of the list to the first found version of "leaf"

move "Alive" to dictionary.compat

it's not defined in any RFC

rlm_sql_mysql: Formatting

rlm_sql_mysql:  Check validity of conn

If conn's magic number is still good, then there's an extremely high chance conn->sock != NULL.  Closes #5144

Improve sql_pair_afrom_row() debug output

It will change the result for:

e.g:

Tue Aug 15 14:24:21 2023: (0) sql - Found row[1]: Acct-Input-Gigawords := 8192

Use canonicalized names for Acct-Status-Type

Alive is Livingston or maybe Cistron?  It's not in RFC 2059 or
RFC 2139.  RFC 2866 has Interim-Update:

https://www.rfc-editor.org/rfc/rfc2866.html#section-5.1

So we change the "accounting" type to Interim-Update, and add
a unit test which verifies that any attempt to read Alive will
result in the server printing Interim-Update

Check returns (CID #1538387)

Humans can figure out that
        FR_DBUFF_REMAINING_RETURN(&foo, bar);
implies that one can safely write bar bytes to the dbuff foo,
but coverity can't. There's another use, in fr_tacacs_encode(),
but the following operations using the dbuff are checked there.

Attempt at a simpler uninit local buffer workaround (CIDs below)

CIDs: 1506690, 1506689, 1504436, 1504041, 1504020, 1503918

This doesn't name the uninitialized local array, but instead
goes via the sbuff/dbuff API to get to the data.

radeapol_uat: Add new script for user tests

We need something similar to scripts/util/raduat, but instead of using radclient. it needs to be implemented in python calling the eapol_test.

the python one needs to also take an eapol_test config as part of the request
as a separate file like:

"my_peap_test", "my_peap_test_conf", "my_peap_test_expected"

"my_peap_test" contains all the attributes we want to send
"my_peap_test_conf" contains the eapol test config
"my_peap_test_expected' contains the attributes we want to see , and whether it'll be an Access-Accept or Access-Reject

the python script needs to implement something similar to raduat, with how the test files are laid out
it needs to take "my_peap_test", "my_peap_test_conf", and figure out how to pass them to eapol_test (maybe using the radict)

i.e. how to get the attributes out of "my_peap_test" and pass them in a format eapol_test wants
passing the config and when eapol_test finishes, checking to see if the attributes it returned match "my_peap_test_expected"

There's no need to load in the suppressions file