Use shutil.copyfile so as to preserve file ownership in authorized_keys

This commit is contained in:
Stephen Fromm 2012-10-25 11:20:53 -07:00
parent f70fcf6abb
commit 93081eab8e

View file

@ -125,7 +125,13 @@ def writekeys(module, filename, keys):
except IOError, e: except IOError, e:
module.fail_json(msg="Failed to write to file %s: %s" % (tmp_path, str(e))) module.fail_json(msg="Failed to write to file %s: %s" % (tmp_path, str(e)))
f.close() 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): def enforce_state(module, params):
""" """