ipsilon.git/ipsilon/util, branch attrs-test

Make the configparser case sensitive.

2015-02-24T17:58:45+00:00

Per the instructions of
https://docs.python.org/2/library/configparser.html#ConfigParser.RawConfigParser.optionxform

Signed-off-by: Patrick Uiterwijk 
Reviewed-by: Simo Sorce

Make available case insensitive mapping matching

2015-02-24T15:58:25+00:00

If ignore_case is True then the incomping attributes are matched
case-insensitively in the policy engine.
The CAse of the incoming attribute is not changed on wildcard
matches. On ther matches attributes will be replaced according
to the mapping tables and the case used will be that of the
mapped attributes.

Signed-off-by: Simo Sorce 
Reviewed-by: Patrick Uiterwijk

Do not crash on failure to load config

2015-02-24T15:58:25+00:00

Just report an error and continue with default values.

Signed-off-by: Simo Sorce 
Reviewed-by: Patrick Uiterwijk

Add config option to load mapping lists

2015-02-24T15:37:45+00:00

This requires careful handling, and should be used sparingly

Signed-off-by: Simo Sorce 
Reviewed-by: Patrick Uiterwijk

Add Policy class to help filter attributes

2015-02-24T15:37:43+00:00

Signed-off-by: Simo Sorce 
Reviewed-by: Patrick Uiterwijk

Add helper to store provider specific data

2015-02-13T19:19:35+00:00

Signed-off-by: Simo Sorce 

Reviewed-by: Rob Crittenden

Add expiration to Idp metadata

2015-01-29T19:06:51+00:00

Also regenerate it frequently, so that any change in configuration can be
automatically reflected in the metadata downloaded my clients over time.

Signed-off-by: Simo Sorce 
Reviewed-by: Patrick Uiterwijk

Fix request multipart logging when only 1 part is present

2015-01-27T21:40:46+00:00

Test to see if the request parameter value is a cherrypy Part
class. This was already being done for the case where the value was a
list, but it was omitted for single values. Logic was combined into new
local function print_param().

Changed the test for the class back to using

    if isinstance(item, cherrypy._cpreqbody.Part):

instead of:

    if getattr(item, "part_class", None):

because using isinstance() clearly indicates what is being done. The
use of getattr() was introduced to prevent a pylint warning concering
use of protected values. The getattr() hack is confusing and proably
not robust if the class implementation changes. The patch now disables
this warning. I cannot explain why cherrypy marks these modules as
protected when clearly one has to utilize them and they are documented
in the cherrypy API doc. Disabling the warning seems the cleanest and
most robust approach.

Signed-off-by: John Dennis 
Reviewed-by: Simo Sorce

Fix int/pep8 errors in latest patches

2015-01-26T22:42:48+00:00

Mea culpa for not checking before pushing

Signed-off-by: Simo Sorce 
Reviewed-by: John Dennis

Add source code context information to debug logs

2015-01-26T19:01:03+00:00

The log.debug() function helpfully adds the name of the function
invoking it but in a complicated software package there are many
functions/methods which share the same name. Thus a debug message
like this:

DEBUG(__init__): xxx

does not give you much context, there are probably hundreds of
__init__ methods. It would help to qualify the method name which it's
class name, that gives a lot more context when reading the
log. Sometimes it's also helpful to know the file and line number.

This patch adds the class name to the function and included the
filename and line number as well. The file path is trimmed to the last
3 components, sufficient to give context but not too verbose. Now the
debug message might look like this instead:

DEBUG(ipsilon/providers/common.py:129 LoadProviders.__init__()): xxx

Also included is a config option 'stacktrace_on_error' which will
include a stacktrace when the log.error function is called. It can be
very useful to see a stacktrace when logging an error, it defaults to
off.

Signed-off-by: John Dennis 
Reviewed-by: Simo Sorce