Don't keep wide_path or wide_name around

Prefer whatever version python-config is, and then python3.

minor simplifications

"filter" is now like "update", too

un-set special flag when we're done with update / map

it helps to stat() the file

make cf_file_include iterative instead of recursive

check max stack only if we're loading the file

rename for clarity

use stack->bufsize instead of talloc_array_length

check max stack depth too

move map/update flags to stack frame

which incidentally also fixes issues with $INCLUDE in the middle
of an "update" section.  That may be a bad idea, but there's no
real reason why it should be impossible

merge cf_file_include() with cf_section_read()

so that it's less code, and we don't have two mutually recusive
functions.

move filename strdup and assignment outside of cf_file_include

so that we can look at removing the function entirely, and moving
to an iterative approach

convert cf reader to use file stack

right now it's the naive "use stack struct instead of passing
variables to a function".  It still recurses.  This is pass one.

Pass two is fixing the reader so that it's one function which
just pushes things into the stack *instead* of recursing.

Revert "CID #145370 - Re-enable commented out code"

This reverts commit 1ceefc9925eb9570a67384ee4935e7f34cc5d3be.

Which breaks control and tls-cache virtual servers

remove unused variable and definitions

various jenkinsfile fixes (#2996)

set User here, too

force user/group to be radiusd

add newly created VPs to the state, not old VPs

remove chown.

It's not needed according to the Systemd documentation

---
RuntimeDirectory=foo/bar baz
the service manager creates /run/foo (if it does not exist), /run/foo/bar, and /run/baz. The directories /run/foo/bar and /run/baz except /run/foo are owned by the user and group specified in User= and Group=, and removed when the service is stopped.
---

don't increment variable if we're not going to use it

i.e. leave things alone on error instead of changing them, and
then returning an error

Don't deref actions when passing to NUM_ELEMENTS

CID #1453698 - Don't know what this is about... maybe this will help?

CID #1453690 - Bounds check on result

CID #1453691 - Check return value of cass_result_column_name

CID #1453692 - Check stack overrun *AFTER* incrementing ctx->stack_depth

CID #1453693 - Bounds check on result

CID #1453694 - Head must be initialised‽ How the hell did clang scan miss this.

CID #1453695 - Fix TOCTOU on directory permissions

CID #1453696 - Check return codes when checking timer events

CID #1453697 - Array length vs index check incorrect

CID #1453699  - Bad offset in bytesex swap

CID #145370 - Re-enable commented out code

rlm_python: Don't leak if got problems to parse the tuple (#2992)

rlm_python: Only set the tuple if both parameters are valid (#2991)

Invert test pass/fail calls

define variables outside of 'ifdef openssl'

create 'if' with correct name2

Add log output to some python tests

Reorganise tests, and add explicit test for shared storage

Produce errors if we fail marshalling

rlm_python: Fix the mod_populate_vptuple() to return correct 'octets' value (#2989)

Fix radiusd.config in rlm_python

Misunderstanding about how the inittab works, just creates a singleton, which still need to be imported individually into each interpreter.  It does not create a module that's *automatically* imported into each interpreter.

rlm_python: PyObject functions should return 'Py_RETURN_NONE' instead of 'NULL' (#2988)

Skip auth_cext_compat for python3

Add version number in rlm_python

Fixup the dependency code to cache the default sections for version numbers and features

rlm_python: Fix the comments (#2987)

Fix some potentially invalid uses of PyDECREF

Ensure the thread state is the same as the one we set, after calling python land

Typo in comment

Don't need this twice

Log information about the certificate chain at higher debug levels, when making OCSP stapling requests

Fix comment headers

use git commit info to generate release string for packages (#2985)

return empty string when no changes are detected (#2984)

actually fix it <sigh>

fix doxygen

one more "shut the heck up"

shut up stupid analyzer

rlm_python: show also the stacktrace (#2979)

It shows the "<file>:<line_num> <func()>

rlm_python: the radiusd.py is deprecated (#2982)

Due to currently be loaded as a built-in function.

rlm_python: Fix the log() call (#2981)

Start splitting LDAP tutorial content into sections

Formatting fixes

rlm_python: Update example.py to be Python3 friendly (#2980)

Of course, keep to be supported by Python 2.x also.

Rcode and formatting fixes

Fix processing instantiate function return codes

We don't care if detach fails

rlm_python: The do_python_single() returns rlm_code_t (#2978)

* rlm_python: The do_python_single() returns rlm_code_t

Never return values less than 0.

* rlm_python: Only call 'instantiate' if it was defined

* rlm_python: Just use the correct rlm_rcode_t instead of int

* rlm_python: If the execution failed, just print the errors

It's helpful to see errors in runtime.

Use the correct osname

detect changes to package manifests and pass no-cache flag to docker if found (#2959)

shut up clang scan

rlm_python: Load the libpython using correct extension (#2977)

don't "goto exit" where exit is "return NULL"

actually add CC_HINT(nonnull)

add CC_HINT(nonnull) to shut up static analyzer

allow '@', too

fix canonicalize error function

so that it prints the correct thing

remove extraneous comments

more fixes for parser

allow more characters
if we don't use regex, treat '/' as a normal string

remove unused variable

nope, don't un-escape things ever

remove last vestige of getstring() etc.

allow cf_get_token() to do escaping or not

it generally escapes names, etc.  But not values

better checks for BARE_WORD strings

which may have ${..} in the middle of them.  The old parser
would just accept anything that wasn't whitespace.  Which would
cause other side-effects.

The updated parser only allows a limited set of characters.  Which
works for all test cases, but which might disallow some corner
cases not tested for.

Fix calling doc test target in travis.yml

Aways Py_DecRef(inst->module)

Fix stupid timestamp issue

rlm_python: Fix marked unused complains (#2976)

crossbuild: sh needs container up

Not this one...

Fix tests target names to be les lisfy

Minor fix

rlm_python: Add hack to bypass bug with 'pythoh2.7-config' in the OSX (#2975)

As discussed with Arran. the 'python2.7-config' has a weird bug on the
OSX that if you have CFLAGS="..." declared with whatever content you get
the below behavior.

$ python2.7-config --cflags | grep -o "\-arch [^ ]*"
$ CFLAGS='whatever' python2.7-config --cflags | grep -o "\-arch [^ ]*"
-arch x86_64
-arch i386
$

It results to break the build.

Catch more g and w flags

be more forgiving about break and return

when there are things after them

make conditional parser check for regexes, too

if tmpl_preparse() is told to not require a regex, then it should
treat '/' as any other character