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

Fix the copy action to not send the content as a param. (#16127)

Browse source 
Has already been transferred as a tempfile.

This fixes the error in https://github.com/ansible/ansible/issues/16125
but there may be higher level issues that should be fixed as well (other
modules might be able to cause status fields like failed and changed to
return a censored string instead of a bool).  So leaving 16125 open for
now.
This commit is contained in:
Toshio Kuratomi 2016-06-03 12:32:15 -07:00 committed by Toshio Kuratomi
parent 7ddf6dbbd4
commit 6f5d1456bc
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/ansible/plugins/action/copy.py
View file

@ -245,6 +245,8 @@ class ActionModule(ActionBase):
original_basename=source_rel, original_basename=source_rel,
) )
) )
if 'content' in new_module_args:
del new_module_args['content']
module_return = self._execute_module(module_name='copy', module_args=new_module_args, task_vars=task_vars, tmp=tmp, delete_remote_tmp=delete_remote_tmp) module_return = self._execute_module(module_name='copy', module_args=new_module_args, task_vars=task_vars, tmp=tmp, delete_remote_tmp=delete_remote_tmp)
module_executed = True module_executed = True