summaryrefslogtreecommitdiffstats
path: root/ipaclient/install/client.py
diff options
context:
space:
mode:
authorStanislav Laznicka <slaznick@redhat.com>2016-11-03 12:29:45 +0100
committerJan Cholasta <jcholast@redhat.com>2016-11-11 12:13:56 +0100
commitb068d3336ad65748881d0dc74505f41dac9f0f13 (patch)
tree06a66783b32354f216f3816cdbb855ce6dbc3c6d /ipaclient/install/client.py
parent990e1acb1a667b90619e7799bb96e2cd81e97e61 (diff)
downloadfreeipa-b068d3336ad65748881d0dc74505f41dac9f0f13.tar.gz
freeipa-b068d3336ad65748881d0dc74505f41dac9f0f13.tar.xz
freeipa-b068d3336ad65748881d0dc74505f41dac9f0f13.zip
Added file permissions option to IPAChangeConf.newConf()
Also added information about why os.chmod is called sometimes after newConf() calls. https://fedorahosted.org/freeipa/ticket/6392 Reviewed-By: Martin Basti <mbasti@redhat.com>
Diffstat (limited to 'ipaclient/install/client.py')
-rw-r--r--ipaclient/install/client.py2
1 files changed, 2 insertions, 0 deletions
diff --git a/ipaclient/install/client.py b/ipaclient/install/client.py
index 3334396d7..c246402b5 100644
--- a/ipaclient/install/client.py
+++ b/ipaclient/install/client.py
@@ -360,6 +360,7 @@ def configure_ipa_conf(
target_fname = paths.IPA_DEFAULT_CONF
fstore.backup_file(target_fname)
ipaconf.newConf(target_fname, opts)
+ # umask applies when creating a new file but we want 0o644 here
os.chmod(target_fname, 0o644)
@@ -746,6 +747,7 @@ def configure_krb5_conf(
root_logger.debug("%s", krbconf.dump(opts))
krbconf.newConf(filename, opts)
+ # umask applies when creating a new file but we want 0o644 here
os.chmod(filename, 0o644)