ipsilon.git, branch non-empty-attrs

Assertion AttributeStatements must be non-empty

2015-03-18T21:14:07+00:00

The saml-core-2.0-os specification section 2.7.3 requires
the AttributeStatement element to be non-empty. Shibboleth verifies
this and rejects assertions that do not comply. We gather attributes
into a local dict first before adding them to the AttributeStatement
so the fix is easy. Test if the dict is empty, move the initialization
of the assertion AttributeStatement inside the test so it's
conditional on whether the dict has members.

Fixes: https://fedorahosted.org/ipsilon/ticket/61
Signed-off-by: John Dennis

Properly handle groups info in SAML provider

2015-03-18T00:38:27+00:00

Also removes internal attributes (any attribute that starts with _

Fixes: https://fedorahosted.org/ipsilon/ticket/71

Signed-off-by: Simo Sorce 
Reviewed-by: Nathan Kinder

Add negative authentication test

2015-03-18T00:37:55+00:00

Signed-off-by: Simo Sorce 
Reviewed-by: Nathan Kinder

Fix error returned from login plugins

2015-03-18T00:37:19+00:00

Some login plugins use form based authentication and let the user retry
on authentication errors. This is fine, however the wrong error code is
returned in this case, 401 should be returned.

Fixes: https://fedorahosted.org/ipsilon/ticket/94

Signed-off-by: Simo Sorce 
Reviewed-by: Nathan Kinder

Make SSSD Info enable the httpd_dbus_sssd boolean.

2015-03-17T14:52:25+00:00

https://fedorahosted.org/ipsilon/ticket/23#comment:13

Signed-off-by: Patrick Uiterwijk 
Reviewed-by: Rob Crittenden

Build dated RPMs by default

2015-03-16T21:47:50+00:00

This stores the build date and git commit in the version.
This way, it's a lot easier to determine when it was last built.

Signed-off-by: Patrick Uiterwijk 
Reviewed-by: Rob Crittenden

Save user attributes on subsequent calls to login.

2015-03-16T21:18:13+00:00

When a login comes in via the remote_login() call no
user attributes are set. These may be later filled in by
a subsequent call to login() after the info plugins are
called but a short-circuit in that function exits if the
user matches the current session.

Add an extra conditional such that if the user matches,
userattributes are passed in and the current user attributes
for this user is empty then save the new data.

https://fedorahosted.org/ipsilon/ticket/86

Signed-off-by: Rob Crittenden 
Reviewed-by: Nathan Kinder

Use the IPA API directly when adding the HTTP principal

2015-03-16T21:13:23+00:00

This is the only way to force in a custom version string
so that the remote IPA server doesn't reject the request
as being newer than the server.

This also removes the need to iterate over all servers
as the IPA connection API does this automatically.

https://fedorahosted.org/ipsilon/ticket/47

Signed-off-by: Rob Crittenden 
Reviewed-by: Nathan Kinder

Fix some pylint warnings in logout test about shadowing variables.

2015-03-16T20:54:15+00:00

Signed-off-by: Rob Crittenden 
Reviewed-by: Nathan Kinder

Add test for multi-SP logout

2015-03-16T17:47:45+00:00

Create an additional SP, log into one, fetch the other and
the client is now logged into both. Log out of the first one
and the client is logged out of both.

https://fedorahosted.org/ipsilon/ticket/58

Signed-off-by: Rob Crittenden 
Reviewed-by: Nathan Kinder