summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Basti <mbasti@redhat.com>2016-01-05 17:58:49 +0100
committerMartin Basti <mbasti@redhat.com>2016-01-08 18:21:26 +0100
commit129d97c10be570c3327445337c534e57a8c12ef6 (patch)
treee8c37536a9fc30587299097114f8b864a5c90c67
parent3d1a8d31343ce0a71d3236e9d8457a06e12c07a9 (diff)
downloadfreeipa-129d97c10be570c3327445337c534e57a8c12ef6.tar.gz
freeipa-129d97c10be570c3327445337c534e57a8c12ef6.tar.xz
freeipa-129d97c10be570c3327445337c534e57a8c12ef6.zip
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>
-rw-r--r--ipapython/sysrestore.py4
-rw-r--r--ipaserver/install/opendnssecinstance.py6
2 files changed, 7 insertions, 3 deletions
diff --git a/ipapython/sysrestore.py b/ipapython/sysrestore.py
index 65ef5da89..c43948db5 100644
--- a/ipapython/sysrestore.py
+++ b/ipapython/sysrestore.py
@@ -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)
diff --git a/ipaserver/install/opendnssecinstance.py b/ipaserver/install/opendnssecinstance.py
index 1c6aaeefa..4d3b65ff9 100644
--- a/ipaserver/install/opendnssecinstance.py
+++ b/ipaserver/install/opendnssecinstance.py
@@ -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: