x/python-ganttclient
Code Issues Proposed changes

PEP fixes

Browse Source
This commit is contained in:
Ryan Lane
2010-11-26 17:59:48 +00:00
parent e0db44d68b
commit 0f02838ef0
1 changed files with 42 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

58
nova/auth/ldapdriver.py
View File

@@ -114,7 +114,7 @@ class LdapDriver(object):
users = [] users = []
for attr in attrs: for attr in attrs:
user = self.__to_user(attr) user = self.__to_user(attr)
if user != None: if user is not None:
users.append(user) users.append(user)
return users return users
@@ -135,24 +135,32 @@ class LdapDriver(object):
if self.__ldap_user_exists(name): if self.__ldap_user_exists(name):
# Retrieve user by name # Retrieve user by name
user = self.__get_ldap_user(name) user = self.__get_ldap_user(name)
if user.has_key('accessKey') and user.has_key('secretKey') and user.has_key('isAdmin'): if user.has_key('accessKey') and user.has_key('secretKey') \
raise exception.Duplicate("LDAP user %s already exists" % name) and user.has_key('isAdmin'):
raise exception.Duplicate("LDAP user %s already exists" \
% name)
else: else:
# Entry could be malformed, test for missing attrs. # Entry could be malformed, test for missing attrs.
# Malformed entries are useless, replace attributes found. # Malformed entries are useless, replace attributes found.
attr = [] attr = []
if user.has_key('secretKey'): if user.has_key('secretKey'):
attr.append((self.ldap.MOD_REPLACE, 'secretKey', [secret_key])) attr.append((self.ldap.MOD_REPLACE, 'secretKey', \
[secret_key]))
else: else:
attr.append((self.ldap.MOD_ADD, 'secretKey', [secret_key])) attr.append((self.ldap.MOD_ADD, 'secretKey', \
[secret_key]))
if user.has_key('accessKey'): if user.has_key('accessKey'):
attr.append((self.ldap.MOD_REPLACE, 'accessKey', [access_key])) attr.append((self.ldap.MOD_REPLACE, 'accessKey', \
[access_key]))
else: else:
attr.append((self.ldap.MOD_ADD, 'accessKey', [access_key])) attr.append((self.ldap.MOD_ADD, 'accessKey', \
[access_key]))
if user.has_key('isAdmin'): if user.has_key('isAdmin'):
attr.append((self.ldap.MOD_REPLACE, 'isAdmin', [str(is_admin).upper()])) attr.append((self.ldap.MOD_REPLACE, 'isAdmin', \
[str(is_admin).upper()]))
else: else:
attr.append((self.ldap.MOD_ADD, 'isAdmin', [str(is_admin).upper()])) attr.append((self.ldap.MOD_ADD, 'isAdmin', \
[str(is_admin).upper()]))
self.conn.modify_s(self.__uid_to_dn(name), attr) self.conn.modify_s(self.__uid_to_dn(name), attr)
return self.get_user(name) return self.get_user(name)
else: else:
@@ -186,7 +194,7 @@ class LdapDriver(object):
if description is None: if description is None:
description = name description = name
members = [] members = []
if member_uids != None: if member_uids is not None:
for member_uid in member_uids: for member_uid in member_uids:
if not self.__user_exists(member_uid): if not self.__user_exists(member_uid):
raise exception.NotFound("Project can't be created " raise exception.NotFound("Project can't be created "
@@ -293,11 +301,14 @@ class LdapDriver(object):
# Retrieve user by name # Retrieve user by name
user = self.__get_ldap_user(uid) user = self.__get_ldap_user(uid)
if user.has_key('secretKey'): if user.has_key('secretKey'):
attr.append((self.ldap.MOD_DELETE, 'secretKey', user['secretKey'])) attr.append((self.ldap.MOD_DELETE, 'secretKey', \
user['secretKey']))
if user.has_key('accessKey'): if user.has_key('accessKey'):
attr.append((self.ldap.MOD_DELETE, 'accessKey', user['accessKey'])) attr.append((self.ldap.MOD_DELETE, 'accessKey', \
user['accessKey']))
if user.has_key('isAdmin'): if user.has_key('isAdmin'):
attr.append((self.ldap.MOD_DELETE, 'isAdmin', user['isAdmin'])) attr.append((self.ldap.MOD_DELETE, 'isAdmin', \
user['isAdmin']))
self.conn.modify_s(self.__uid_to_dn(uid), attr) self.conn.modify_s(self.__uid_to_dn(uid), attr)
else: else:
# Delete entry # Delete entry
@@ -324,15 +335,15 @@ class LdapDriver(object):
def __user_exists(self, uid): def __user_exists(self, uid):
"""Check if user exists""" """Check if user exists"""
return self.get_user(uid) != None return self.get_user(uid) is not None
def __ldap_user_exists(self, uid): def __ldap_user_exists(self, uid):
"""Check if the user exists in ldap""" """Check if the user exists in ldap"""
return self.__get_ldap_user(uid) != None return self.__get_ldap_user(uid) is not None
def __project_exists(self, project_id): def __project_exists(self, project_id):
"""Check if project exists""" """Check if project exists"""
return self.get_project(project_id) != None return self.get_project(project_id) is not None
def __get_ldap_user(self, uid): def __get_ldap_user(self, uid):
"""Retrieve LDAP user entry by id""" """Retrieve LDAP user entry by id"""
@@ -385,12 +396,12 @@ class LdapDriver(object):
def __group_exists(self, dn): def __group_exists(self, dn):
"""Check if group exists""" """Check if group exists"""
return self.__find_object(dn, '(objectclass=groupOfNames)') != None return self.__find_object(dn, '(objectclass=groupOfNames)') is not None
@staticmethod @staticmethod
def __role_to_dn(role, project_id=None): def __role_to_dn(role, project_id=None):
"""Convert role to corresponding dn""" """Convert role to corresponding dn"""
if project_id == None: if project_id is None:
return FLAGS.__getitem__("ldap_%s" % role).value return FLAGS.__getitem__("ldap_%s" % role).value
else: else:
return 'cn=%s,cn=%s,%s' % (role, return 'cn=%s,cn=%s,%s' % (role,
@@ -404,7 +415,7 @@ class LdapDriver(object):
raise exception.Duplicate("Group can't be created because " raise exception.Duplicate("Group can't be created because "
"group %s already exists" % name) "group %s already exists" % name)
members = [] members = []
if member_uids != None: if member_uids is not None:
for member_uid in member_uids: for member_uid in member_uids:
if not self.__user_exists(member_uid): if not self.__user_exists(member_uid):
raise exception.NotFound("Group can't be created " raise exception.NotFound("Group can't be created "
@@ -430,7 +441,7 @@ class LdapDriver(object):
res = self.__find_object(group_dn, res = self.__find_object(group_dn,
'(member=%s)' % self.__uid_to_dn(uid), '(member=%s)' % self.__uid_to_dn(uid),
self.ldap.SCOPE_BASE) self.ldap.SCOPE_BASE)
return res != None return res is not None
def __add_to_group(self, uid, group_dn): def __add_to_group(self, uid, group_dn):
"""Add user to group""" """Add user to group"""
@@ -502,9 +513,10 @@ class LdapDriver(object):
@staticmethod @staticmethod
def __to_user(attr): def __to_user(attr):
"""Convert ldap attributes to User object""" """Convert ldap attributes to User object"""
if attr == None: if attr is None:
return None return None
if (attr.has_key('accessKey') and attr.has_key('secretKey') and attr.has_key('isAdmin')): if (attr.has_key('accessKey') and attr.has_key('secretKey') \
and attr.has_key('isAdmin')):
return { return {
'id': attr['uid'][0], 'id': attr['uid'][0],
'name': attr['cn'][0], 'name': attr['cn'][0],
@@ -516,7 +528,7 @@ class LdapDriver(object):
def __to_project(self, attr): def __to_project(self, attr):
"""Convert ldap attributes to Project object""" """Convert ldap attributes to Project object"""
if attr == None: if attr is None:
return None return None
member_dns = attr.get('member', []) member_dns = attr.get('member', [])
return { return {