summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimo Sorce <simo@redhat.com>2014-05-20 15:28:27 -0400
committerSimo Sorce <simo@redhat.com>2014-05-20 15:31:08 -0400
commitc4aa2a8fc207d464aa23e065b5f2ad0549a58f5e (patch)
treee334e2228b8f995e2fb87cfef93bced71ce1c800
parentf47a95ba1df58ccf9784c47beeaa0702c469b3e1 (diff)
downloadipsilon.git-c4aa2a8fc207d464aa23e065b5f2ad0549a58f5e.tar.gz
ipsilon.git-c4aa2a8fc207d464aa23e065b5f2ad0549a58f5e.tar.xz
ipsilon.git-c4aa2a8fc207d464aa23e065b5f2ad0549a58f5e.zip
Fix E256 with stricter pep8 error checker
Signed-off-by: Simo Sorce <simo@redhat.com>
-rwxr-xr-xipsilon/login/common.py2
-rwxr-xr-xipsilon/providers/saml2/admin.py4
-rwxr-xr-xipsilon/providers/saml2/auth.py4
-rwxr-xr-xipsilon/tools/certs.py2
-rwxr-xr-xipsilon/util/plugin.py2
-rwxr-xr-xipsilon/util/user.py2
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