From e6331db50afceb08f1c466b2d9a35ebdb75cb5c3 Mon Sep 17 00:00:00 2001
From: Thomas Bellman <bellman@nsc.liu.se>
Date: Wed, 25 Mar 2020 11:32:32 +0100
Subject: [PATCH] Quote numeric file permission modes.

Numeric (octal) file permission modes must be strings to be properly
interpreted by Puppet in version 4 and later.  Fix the two instances
where we have a file mode parameter.
---
 manifests/include_file.pp | 2 +-
 manifests/logdir.pp       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/manifests/include_file.pp b/manifests/include_file.pp
index 3b4b94e..0a13fc0 100644
--- a/manifests/include_file.pp
+++ b/manifests/include_file.pp
@@ -31,7 +31,7 @@ define apache::include_file($content=undef, $source=undef, $ensure='present')
 	    file {
 		$includefile:
 		    ensure => file,
-		    owner => 'root', group => 'root', mode => 0444,
+		    owner => 'root', group => 'root', mode => '0444',
 		    content => $content, source => $source,
 		    notify => Class[apache::service];
 	    }
diff --git a/manifests/logdir.pp b/manifests/logdir.pp
index 924d2c7..f5a575a 100644
--- a/manifests/logdir.pp
+++ b/manifests/logdir.pp
@@ -11,7 +11,7 @@
  * apache::base class, so users need to include it themselves.
  */
 
-class apache::logdir($owner='root', $group='root', $mode=0700)
+class apache::logdir($owner='root', $group='root', $mode='0700')
 {
     include apache
 
-- 
GitLab