Merge pull request #355 from rtprio/gid0

remove the 'root' group and replace with group 0
voxpupuli · Jul 14, 2024 · e6acdf5 · e6acdf5
2 parents 9db952a + a753ec0
commit e6acdf5
Show file tree

Hide file tree

Showing 9 changed files with 9 additions and 9 deletions.
diff --git a/manifests/hook.pp b/manifests/hook.pp
@@ -20,7 +20,7 @@
   file { $hook_file:
     ensure  => file,
     owner   => 'root',
-    group   => 'root',
+    group   => 0,
     mode    => '0755',
     content => epp('letsencrypt/hook.sh.epp', {
         commands     => $commands,

diff --git a/manifests/plugin/dns_cloudflare.pp b/manifests/plugin/dns_cloudflare.pp
@@ -65,7 +65,7 @@
   file { $config_path:
     ensure  => file,
     owner   => 'root',
-    group   => 'root',
+    group   => 0,
     mode    => '0400',
     content => epp('letsencrypt/ini.epp', {
         vars => { '' => $ini_vars },

diff --git a/manifests/plugin/dns_rfc2136.pp b/manifests/plugin/dns_rfc2136.pp
@@ -43,7 +43,7 @@
   file { "${config_dir}/dns-rfc2136.ini":
     ensure  => file,
     owner   => 'root',
-    group   => 'root',
+    group   => 0,
     mode    => '0400',
     content => epp('letsencrypt/ini.epp', {
         vars => { '' => $ini_vars },

diff --git a/manifests/renew.pp b/manifests/renew.pp
@@ -48,7 +48,7 @@
     ensure  => directory,
     path    => "${letsencrypt::config_dir}/renewal-hooks-puppet",
     owner   => 'root',
-    group   => 'root',
+    group   => 0,
     mode    => '0755',
     recurse => true,
     purge   => true,

diff --git a/manifests/scripts.pp b/manifests/scripts.pp
@@ -9,7 +9,7 @@
   file { '/usr/local/sbin/letsencrypt-domain-validation':
     ensure  => file,
     owner   => 'root',
-    group   => 'root',
+    group   => 0,
     mode    => '0500',
     content => file("${module_name}/domain-validation.sh"),
   }

diff --git a/spec/classes/letsencrypt_spec.rb b/spec/classes/letsencrypt_spec.rb
@@ -40,7 +40,7 @@
                 with(ensure: 'directory',
                      path: '/usr/local/etc/letsencrypt/renewal-hooks-puppet',
                      owner: 'root',
-                     group: 'root',
+                     group: 0,
                      mode: '0755',
                      recurse: true,
                      purge: true)

diff --git a/spec/classes/plugin/dns_rfc2136_spec.rb b/spec/classes/plugin/dns_rfc2136_spec.rb
@@ -55,7 +55,7 @@ class { 'letsencrypt':
             is_expected.to contain_file("#{pathprefix}/etc/letsencrypt/dns-rfc2136.ini").
               with_ensure('file').
               with_owner('root').
-              with_group('root').
+              with_group('0').
               with_mode('0400').
               with_content(%r{^.*dns_rfc2136_server.*$})
           end

diff --git a/spec/defines/letsencrypt_certonly_spec.rb b/spec/defines/letsencrypt_certonly_spec.rb
@@ -27,7 +27,7 @@
           is_expected.to contain_file('/usr/local/sbin/letsencrypt-domain-validation').
             with_ensure('file').
             with_owner('root').
-            with_group('root').
+            with_group('0').
             with_mode('0500').
             with_content(%r{#!/bin/sh})
         end

diff --git a/spec/defines/letsencrypt_hook_spec.rb b/spec/defines/letsencrypt_hook_spec.rb
@@ -33,7 +33,7 @@
           is_expected.to contain_file('/etc/letsencrypt/renewal-hooks-puppet/foo.example.com-pre.sh').
             with(ensure: 'file',
                  owner: 'root',
-                 group: 'root',
+                 group: 0,
                  mode: '0755',
                  content: %r{^.*validate_env=0.*FooBar.*$}m).
             that_requires('File[letsencrypt-renewal-hooks-puppet]')