From 3a44d60fbc18829ebe5fc5edacebdee7d62fd605 Mon Sep 17 00:00:00 2001 From: Evan Kaufman Date: Wed, 10 Aug 2016 14:43:58 -0700 Subject: [PATCH] Fixes #3791 cron always returning changed state for multiline jobs (#4285) Strip only newlines and carriage returns. Instead of stripping ALL whitespace, which may have unintended side effects --- lib/ansible/modules/system/cron.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/ansible/modules/system/cron.py b/lib/ansible/modules/system/cron.py index 6aca8a7fd04..7df46469b8b 100644 --- a/lib/ansible/modules/system/cron.py +++ b/lib/ansible/modules/system/cron.py @@ -383,6 +383,9 @@ class CronTab(object): return [] def get_cron_job(self,minute,hour,day,month,weekday,job,special,disabled): + # normalize any leading/trailing newlines (ansible/ansible-modules-core#3791) + job = job.strip('\r\n') + if disabled: disable_prefix = '#' else: