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

more separation of logic between CONF_SECTION and CONF_PAIR

the rearrangement isn't done yet, but the resulting code is
starting to be more understandable.  Once we separate the
functionality we can start re-doing the parser

added documentation for buildClosures function (#2974)

Make rlm_python mostly functional with Python 3

Only pending issue is the config not being imported correctly into the python environment

Fix help text

assign to '=' on error

Of course, errors are errors.

Revert "dict can be NULL.  Others can't.  Fixes #2971"

This reverts commit d4a76e2bcd93b8a155efb834d994925fdb6cb5e6.

dict can be NULL.  Others can't.  Fixes #2971

Fix operator assignment

Default to internal dictionary if we can't find one in a parent

s/this/parent/ in cf_file.c because 'this' is a reserved name in lldb

Start moving things to using the correct logging macro

This is probably going to be a mostly manual process :(

Hmm?

Quiet clang scan

Strip -Wall when returned from python-config

Fix integer widths in rlm_python.c

Helps to regenerate the configure script

lineno isn't a pointer...

Yet more flag stripping *sigh*

make cf_expand_variables() take lineno instead of *lineno

which was done for who knows what historical reasons

make fr_cond_tokenize() take CONF_SECTION we're parsing

which ends up being easier than the alternative.  That way
instead of passing a bunch of things around, we can just pass
a CONF_SECTION

ensure that we always pass a CONF_SECTION

More filtering of ldflags

More python-config stupidity

-Werror args probably don't contain spaces, but they do contain hyphens

Deal with -Werror where it has args

make process_map() call cf_get_token()

which regularizes the parsing of the various things

we only need 4 buffers

rearrange the user of the buffers, and document what they're for.

simplify process_include()

so that it no longer uses the old APIs

allow unlang expansions in name1 for conf section/pairs

which can occasionally be useful.  And, is preparation for
removing the rest of the use of gettoken()

use 'this' section, not the top-level one

expand variables in cf_get_token

errors are errors

s/Ssize_t/Size_t/ in rlm_python

Use PyLong_FromSsize_t where appropriate

Switch numeric interchange to PyLong, but use PyNumber to check for compatibility with Python v2 and v3

Just use python-config to configure the python module

Python 3 compatibility fixes

Revert "We don't need to clear repeatable anymore it's done by frame_cleanup"

This reverts commit bbc8a8bfe272095cc186d1313b98438d19c65d7b.

fix nested %{...}

allow bare %{...} in conditions

which for now are parsed and printed as double-quoted strings.

But it makes some things much easier

decrement before checks

do less memcpy

make cf_expand_variables() take a length of input

which makies other things simpler, such as the conditional
tokenizer

cleanups and create cf_get_token()

which wraps tmpl_preparse(), and eventually cf_expand_variables()
so that the config file parser *doesn't* use gettoken().

Instead, it should use the new APIs which provide for better
error reporting and more functionality

Add test for return after policy

Policy sets return_point on that stack frame, so we need to make sure it's cleared when the next instruction is executed

We don't need to clear repeatable anymore it's done by frame_cleanup

Clear break_point and return_point when moving to the next instruction or popping things

We really do only need to set top_frame when pushing new things onto the stack

Print unwind at the stack level

Set dependencies correctly for return keyword tests

less code == better code

typo

Fix missing free() in $INCLUDE

fix list for proper nesting

text was already in the partial include file

typo

use ":" instead of "::" everywhere

should be fixed-width

this file was moved

document changed behavior

use standard macro

unescape bare %{...} too

doesn't seem to make any difference, even though it is sometimes set

make some functions static

they are old, and now only used in one place

use tmpl_preparse(), which now does most necessary things

tmpl_preparse() is now capable of parsing %{...}

Create an initial, empty CRL in DER encoding #2964

be a bit more flexible about 'map'

map { ...} is OK in sigtran as a config section

allow , and ; after words

less code is better code

simplify the parser.

it's clearer to just check for characters directly, instead of
calling functions and then checking for token values

move "expand variables" into add_pair() function

move add_pair() to separate function

and start cleaning up the parser to be simpler and more understandable

remove WITH_CONF_WRITE

unused, untested, likely not to work.  If we re-do the parser,
we can make this functionality part of the requirements.

Until then, this code is making refactoring mode difficult

ue correct CA password when not the default "whatever"

without passin and passout, the CA passphrase is taken from ca.cnf where it is "whatever". In passwords.mk it is also "whatever" so that is usually okay.
However when changing the password in passwords.mk, the CA is still generated with "whatever" but subsequent cert generation commands use the custom password -> failure.