Allow to used mixed case for sysrestore

This commit allows to use mixed case of keys for sysrestore, before this
commit all keys were saved in lowercase what prevents to accesing them.

Original usage of mixed case for sysretore key in opendssecinstance had
to be changed to lowercase to prevent issues on already installed
systems.

https://fedorahosted.org/freeipa/ticket/5574

Reviewed-By: Martin Babinsky <mbabinsk@redhat.com>
This commit is contained in:
Martin Basti 2016-01-05 17:58:49 +01:00
parent 3d1a8d3134
commit 129d97c10b
2 changed files with 7 additions and 3 deletions

View File

@ -71,6 +71,7 @@ class FileStore:
self.files = {}
p = SafeConfigParser()
p.optionxform = str
p.read(self._index)
for section in p.sections():
@ -92,6 +93,7 @@ class FileStore:
return
p = SafeConfigParser()
p.optionxform = str
p.add_section('files')
for (key, value) in self.files.items():
@ -327,6 +329,7 @@ class StateFile:
self.modules = {}
p = SafeConfigParser()
p.optionxform = str
p.read(self._path)
for module in p.sections():
@ -355,6 +358,7 @@ class StateFile:
return
p = SafeConfigParser()
p.optionxform = str
for module in self.modules.keys():
p.add_section(module)

View File

@ -268,11 +268,11 @@ class OpenDNSSECInstance(service.Service):
def __setup_dnssec(self):
# run once only
if self.get_state("KASP_DB_configured") and not self.kasp_db_file:
if self.get_state("kasp_db_configured") and not self.kasp_db_file:
root_logger.debug("Already configured, skipping step")
return
self.backup_state("KASP_DB_configured", True)
self.backup_state("kasp_db_configured", True)
if not self.fstore.has_file(paths.OPENDNSSEC_KASP_DB):
self.fstore.backup_file(paths.OPENDNSSEC_KASP_DB)
@ -377,7 +377,7 @@ class OpenDNSSECInstance(service.Service):
except ValueError as error:
root_logger.debug(error)
self.restore_state("KASP_DB_configured") # just eat state
self.restore_state("kasp_db_configured") # just eat state
# disabled by default, by ldap_enable()
if enabled: