From bfb591dc7419ee0c9574b995b5b0fb6bf83d367e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9r=C3=A9my=20Lecour?= <jlecour@evolix.fr>
Date: Sun, 9 Sep 2018 23:01:43 +0200
Subject: [PATCH] apache: cleaner way to overwrite the server status suffix

---
 CHANGELOG.md                   | 1 +
 apache/tasks/server_status.yml | 6 +++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 7c038372..d9236219 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -16,6 +16,7 @@ The **patch** part changes incrementally at each release.
 ### Changed
 
 ### Fixed
+* apache: cleaner way to overwrite the server status suffix
 * packweb-apache: don't regenerate phpMyAdmin suffix each time
 
 ### Security
diff --git a/apache/tasks/server_status.yml b/apache/tasks/server_status.yml
index 696f2592..f9e1aed8 100644
--- a/apache/tasks/server_status.yml
+++ b/apache/tasks/server_status.yml
@@ -9,7 +9,11 @@
     state: directory
 
 - name: set apache serverstatus suffix if provided
-  shell: "echo {{ apache_serverstatus_suffix }} > {{ apache_serverstatus_suffix_file }}"
+  copy:
+    dest: "{{ apache_serverstatus_suffix_file }}"
+    # The last character "\u000A" is a line feed (LF), it's better to keep it
+    content: "{{ apache_serverstatus_suffix }}\u000A"
+    force: yes
   when: apache_serverstatus_suffix != ""
 
 - name: generate random string for server-status suffix