From c4aa2a8fc207d464aa23e065b5f2ad0549a58f5e Mon Sep 17 00:00:00 2001
From: Simo Sorce <simo@redhat.com>
Date: Tue, 20 May 2014 15:28:27 -0400
Subject: [PATCH] Fix E256 with stricter pep8 error checker

Signed-off-by: Simo Sorce <simo@redhat.com>
---
 ipsilon/login/common.py          | 2 +-
 ipsilon/providers/saml2/admin.py | 4 ++--
 ipsilon/providers/saml2/auth.py  | 4 ++--
 ipsilon/tools/certs.py           | 2 +-
 ipsilon/util/plugin.py           | 2 +-
 ipsilon/util/user.py             | 2 +-
 6 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/ipsilon/login/common.py b/ipsilon/login/common.py
index f0efebd..a576345 100755
--- a/ipsilon/login/common.py
+++ b/ipsilon/login/common.py
@@ -108,7 +108,7 @@ class LoginManagerBase(PluginObject):
         if self not in plugins['enabled']:
             return
 
-        #remove self from chain
+        # remove self from chain
         root = plugins['root']
         if root.first_login == self:
             root.first_login = self.next_login
diff --git a/ipsilon/providers/saml2/admin.py b/ipsilon/providers/saml2/admin.py
index 4f33e1b..a9fb9e0 100755
--- a/ipsilon/providers/saml2/admin.py
+++ b/ipsilon/providers/saml2/admin.py
@@ -51,8 +51,8 @@ class NewSPAdminPage(Page):
     def POST(self, *args, **kwargs):
 
         if self.user.is_admin:
-            #TODO: allow authenticated user to create SPs on their own
-            #      set the owner in that case
+            # TODO: allow authenticated user to create SPs on their own
+            #       set the owner in that case
             name = None
             meta = None
             if 'content-type' not in cherrypy.request.headers:
diff --git a/ipsilon/providers/saml2/auth.py b/ipsilon/providers/saml2/auth.py
index 65d795d..036ed5e 100755
--- a/ipsilon/providers/saml2/auth.py
+++ b/ipsilon/providers/saml2/auth.py
@@ -174,10 +174,10 @@ class AuthenticateRequest(ProviderPageBase):
 
         nameid = None
         if nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_PERSISTENT:
-            ## TODO map to something else ?
+            # TODO map to something else ?
             nameid = provider.normalize_username(user.name)
         elif nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_TRANSIENT:
-            ## TODO map to something else ?
+            # TODO map to something else ?
             nameid = provider.normalize_username(user.name)
         elif nameidfmt == lasso.SAML2_NAME_IDENTIFIER_FORMAT_KERBEROS:
             nameid = us.get_data('user', 'krb_principal_name')
diff --git a/ipsilon/tools/certs.py b/ipsilon/tools/certs.py
index 3a60488..2ff63f1 100755
--- a/ipsilon/tools/certs.py
+++ b/ipsilon/tools/certs.py
@@ -50,7 +50,7 @@ class Certificate(object):
         with open(self.cert, 'r') as f:
             cert = f.readlines()
 
-        #poor man stripping of BEGIN/END lines
+        # poor man stripping of BEGIN/END lines
         if cert[0] == '-----BEGIN CERTIFICATE-----\n':
             cert = cert[1:]
         if cert[-1] == '-----END CERTIFICATE-----\n':
diff --git a/ipsilon/util/plugin.py b/ipsilon/util/plugin.py
index c57cb29..730ce6c 100755
--- a/ipsilon/util/plugin.py
+++ b/ipsilon/util/plugin.py
@@ -36,7 +36,7 @@ class Plugins(object):
         try:
             if ext.lower() == '.py':
                 mod = imp.load_source(name, file_name)
-            #elif ext.lower() == '.pyc':
+            # elif ext.lower() == '.pyc':
             #    mod = imp.load_compiled(name, file_name)
             else:
                 return
diff --git a/ipsilon/util/user.py b/ipsilon/util/user.py
index ea0b974..f47c072 100755
--- a/ipsilon/util/user.py
+++ b/ipsilon/util/user.py
@@ -94,7 +94,7 @@ class User(object):
 
     @sites.setter
     def sites(self):
-        #TODO: implement setting sites via the user object ?
+        # TODO: implement setting sites via the user object ?
         raise AttributeError
 
 
-- 
2.20.1