cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge pull request #1453 from sfromm/issue1211

Browse source 
Use shutil.copyfile so as to preserve file ownership in authorized_keys
This commit is contained in:
Michael DeHaan 2012-10-25 18:22:56 -07:00
commit 68bf6b1343
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
authorized_key
View file

@ -125,7 +125,13 @@ def writekeys(module, filename, keys):
except IOError, e:
module.fail_json(msg="Failed to write to file %s: %s" % (tmp_path, str(e)))
f.close()
shutil.move(tmp_path, filename)
try:
shutil.copyfile(tmp_path, filename)
os.unlink(tmp_path)
except IOError, e:
module.fail_json(msg="Failed to copy temp file to %s: %s" % (filename, str(e)))
except OSError, e:
module.fail_json(msg="Failed to remove temp file: %s" % str(e))
def enforce_state(module, params):
"""