From 8a19d3139142b738e270d69fafe1f06a3c142528 Mon Sep 17 00:00:00 2001 From: Robert Wehner Date: Mon, 24 Mar 2014 09:06:25 -0600 Subject: [PATCH] Update 'validate=' parameter documentation to make clear that the command passed via this paramter is not passed via shell and so all shell features will not work. --- library/files/copy | 1 + library/files/lineinfile | 3 ++- library/files/template | 3 ++- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/library/files/copy b/library/files/copy index dbf9c71b4f6..bbf277837a8 100644 --- a/library/files/copy +++ b/library/files/copy @@ -73,6 +73,7 @@ options: description: - The validation command to run before copying into place. The path to the file to validate is passed in via '%s' which must be present as in the visudo example below. + The command is passed securely so shell features like expansion and pipes won't work. required: false default: "" version_added: "1.2" diff --git a/library/files/lineinfile b/library/files/lineinfile index bdc7b51ed90..bad0cf093ce 100644 --- a/library/files/lineinfile +++ b/library/files/lineinfile @@ -110,7 +110,8 @@ options: validate: required: false description: - - validation to run before copying into place + - validation to run before copying into place. The command is passed + securely so shell features like expansion and pipes won't work. required: false default: None version_added: "1.4" diff --git a/library/files/template b/library/files/template index 39f92c72a72..b4a964962f0 100644 --- a/library/files/template +++ b/library/files/template @@ -40,7 +40,8 @@ options: default: "no" validate: description: - - validation to run before copying into place + - validation to run before copying into place. The command is passed + securely so shell features like expansion and pipes won't work. required: false default: "" version_added: "1.2"