Introduce an authz_core mod class

This removes duplication in default mods and makes it easy to use without default mods.
puppetlabs · Jan 16, 2023 · a678f3d · a678f3d
1 parent 210f319
commit a678f3d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 13 deletions.
diff --git a/manifests/default_mods.pp b/manifests/default_mods.pp
@@ -114,6 +114,7 @@
     }
     include apache::mod::alias
     include apache::mod::authn_file
+    include apache::mod::authz_core
     include apache::mod::autoindex
     include apache::mod::dav
     include apache::mod::dav_fs
@@ -124,11 +125,6 @@
     include apache::mod::setenvif
     include apache::mod::auth_basic
 
-    # authz_core is needed for 'Require' directive
-    ::apache::mod { 'authz_core':
-      id => 'authz_core_module',
-    }
-
     # lots of stuff seems to break without access_compat
     ::apache::mod { 'access_compat': }
 
@@ -138,14 +134,8 @@
   } elsif $mods {
     ::apache::default_mods::load { $mods: }
 
-    # authz_core is needed for 'Require' directive
-    ::apache::mod { 'authz_core':
-      id => 'authz_core_module',
-    }
+    include apache::mod::authz_core
   } else {
-    # authz_core is needed for 'Require' directive
-    ::apache::mod { 'authz_core':
-      id => 'authz_core_module',
-    }
+    include apache::mod::authz_core
   }
 }
diff --git a/manifests/mod/authz_core.pp b/manifests/mod/authz_core.pp
@@ -0,0 +1,13 @@
+# @summary
+#   Installs `mod_authz_core`.
+# 
+# @param apache_version
+#   The version of apache being run.
+# 
+# @see https://httpd.apache.org/docs/current/mod/mod_authz_core.html for additional documentation.
+#
+class apache::mod::authz_core {
+  apache::mod { 'authz_core':
+    id => 'authz_core_module',
+  }
+}