summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNarayan Desai <desai@mcs.anl.gov>2007-05-22 16:13:03 +0000
committerNarayan Desai <desai@mcs.anl.gov>2007-05-22 16:13:03 +0000
commitdf43f23d4333a7b15988bb0c7aaa8493df6804b5 (patch)
tree3274924478611cfc0382404ea051ea9b5873a971
parent6616664cc3172cc5d847a495042f21cb3fb24007 (diff)
downloadbcfg2-df43f23d4333a7b15988bb0c7aaa8493df6804b5.tar.gz
bcfg2-df43f23d4333a7b15988bb0c7aaa8493df6804b5.tar.bz2
bcfg2-df43f23d4333a7b15988bb0c7aaa8493df6804b5.zip
internally canonicalize client names (to lower case) (resolves ticket #442)
git-svn-id: https://svn.mcs.anl.gov/repos/bcfg/trunk/bcfg2@3200 ce84e21b-d406-0410-9b95-82705330c041
-rw-r--r--src/lib/Server/Plugins/Metadata.py29
1 files changed, 16 insertions, 13 deletions
diff --git a/src/lib/Server/Plugins/Metadata.py b/src/lib/Server/Plugins/Metadata.py
index 98ae0fb25..8c79fe3ce 100644
--- a/src/lib/Server/Plugins/Metadata.py
+++ b/src/lib/Server/Plugins/Metadata.py
@@ -110,28 +110,31 @@ class Metadata(Bcfg2.Server.Plugin.Plugin):
self.addresses = {}
self.clientdata = xdata
for client in xdata.findall('.//Client'):
+ clname = client.get('name').lower()
if 'address' in client.attrib:
caddr = client.get('address')
if self.addresses.has_key(caddr):
- self.addresses[caddr].append(client.get('name'))
+ self.addresses[caddr].append(clname))
else:
- self.addresses[caddr] = [client.get('name')]
+ self.addresses[caddr] = [clname]
if 'uuid' in client.attrib:
- self.uuid[client.get('uuid')] = client.get('name')
+ self.uuid[client.get('uuid')] = clname
if 'secure' in client.attrib:
- self.secure.append(client.get('name'))
+ self.secure.append(clname)
if client.get('location', 'fixed') == 'floating':
- self.floating.append(client.get('name'))
+ self.floating.append(clname)
if 'password' in client.attrib:
- self.passwords[client.get('name')] = client.get('password')
- for alias in [alias for alias in client.findall('Alias') if 'address' in alias.attrib]:
+ self.passwords[clname] = client.get('password')
+ for alias in [alias for alias in client.findall('Alias')\
+ if 'address' in alias.attrib]:
if self.addresses.has_key(alias.get('address')):
- self.addresses[alias.get('address')].append(client.get('name'))
+ self.addresses[alias.get('address')].append(clname)
else:
- self.addresses[alias.get('address')] = (client.get('name'))
+ self.addresses[alias.get('address')] = [clname]
- self.clients.update({client.get('name'): client.get('profile')})
- [self.aliases.update({alias.get('name'): client.get('name')}) for alias in client.findall('Alias')]
+ self.clients.update({clname: client.get('profile')})
+ [self.aliases.update({alias.get('name'): clname}) \
+ for alias in client.findall('Alias')]
elif dest == 'groups.xml':
self.public = []
self.profiles = []
@@ -244,7 +247,7 @@ class Metadata(Bcfg2.Server.Plugin.Plugin):
def resolve_client(self, addresspair):
'''Lookup address locally or in DNS to get a hostname'''
- print self.session_cache
+ #print self.session_cache
if self.session_cache.has_key(addresspair):
(stamp, uuid) = self.session_cache[addresspair]
if time.time() - stamp < 60:
@@ -256,7 +259,7 @@ class Metadata(Bcfg2.Server.Plugin.Plugin):
raise MetadataConsistencyError
return self.addresses[address][0]
try:
- return socket.gethostbyaddr(address)[0]
+ return socket.gethostbyaddr(address)[0].lower()
except socket.herror:
warning = "address resolution error for %s" % (address)
self.logger.warning(warning)