From: Rob Crittenden <rcritten@redhat.com>
Date: Mon, 19 Oct 2015 20:31:15 +0000 (-0400)
Subject: Fix incorrect raise exception syntax
X-Git-Url: http://git.cascardo.info/?p=cascardo%2Fipsilon.git;a=commitdiff_plain;h=07796396a81c73e7a4bf630c7ff2418a956b44b1

Fix incorrect raise exception syntax

This was causing a logout error to throw an exception in
Ipsilon instead, masking the original error.

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

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
---

diff --git a/ipsilon/providers/saml2/logout.py b/ipsilon/providers/saml2/logout.py
index 374e885..75c7869 100644
--- a/ipsilon/providers/saml2/logout.py
+++ b/ipsilon/providers/saml2/logout.py
@@ -199,7 +199,8 @@ class LogoutRequest(ProviderPageBase):
         if response.status_code != 200:
             self.error('SOAP error (%s) (on %s)' %
                        (response.status, logout.msgUrl))
-            raise InvalidRequest('SOAP HTTP error code', response.status_code)
+            raise InvalidRequest('SOAP HTTP error code %s' %
+                                 response.status_code)
 
         if not response.text:
             self.error('Empty SOAP response')