matthijs/cdist
1
0
Fork 0
forked from ungleich-public/cdist
Code Issues Pull requests Projects Releases Wiki Activity

Remove the umask requirement, set the proper permissions to base_path

Browse source
This commit is contained in:
Arkaitz Jimenez 2013-04-08 17:35:45 +00:00 committed by Nico Schottelius
parent dd3011f447
commit 9dcad37acf
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
cdist/exec/remote.py
View file

@ -60,6 +60,7 @@ class Remote(object):
def create_files_dirs(self): def create_files_dirs(self):
self.rmdir(self.base_path) self.rmdir(self.base_path)
self.mkdir(self.base_path) self.mkdir(self.base_path)
self.run(["/bin/chmod", "700", self.base_path])
self.mkdir(self.conf_path) self.mkdir(self.conf_path)
def rmdir(self, path): def rmdir(self, path):
@ -100,9 +101,6 @@ class Remote(object):
cmd = self._exec.split() cmd = self._exec.split()
cmd.append(self.target_host) cmd.append(self.target_host)
# Always call umask before actual call to ensure proper file permissions
cmd.append("umask 077;")
# FIXME: replace this by -o SendEnv name -o SendEnv name ... to ssh? # FIXME: replace this by -o SendEnv name -o SendEnv name ... to ssh?
# can't pass environment to remote side, so prepend command with # can't pass environment to remote side, so prepend command with
# variable declarations # variable declarations