#1980 - Introduce class adblockplus::web

modules/adblockplus/manifests/web.pp

Index: modules/adblockplus/manifests/web.pp
diff --git a/modules/adblockplus/manifests/web.pp b/modules/adblockplus/manifests/web.pp
new file mode 100644
index 0000000000000000000000000000000000000000..6a75fe3e063c510c9665baecd3133a421715f5a6
--- /dev/null
+++ b/modules/adblockplus/manifests/web.pp
@@ -0,0 +1,26 @@
+# == Class: adblockplus::web
+#
+# Default root namespace for integrating custom web resources.
+#
+# See http://hub.eyeo.com/issues/1980.
+#
+# === Parameters:
+#
+# [*directory*]
+#   Custom parameters for the /var/www directory.
+#
+# [*ensure*]
+#   General resource policy, i.e. "present" or "absent".
+#
+class adblockplus::web (
+  $directory = {},
+  $ensure = 'present',
+) {
+
+  ensure_resource('file', '/var/www', merge({
+    'mode' => '0755',

[Fred, 2017/07/06 12:33:00]

Shouldn't also a user (owner) be provided as a default?
Although that might depend on the actual webserver package in use.

[mathias, 2017/07/06 12:42:06]

In the current state this defaults to the user executing Puppet, which is always
root in our case. And no, the HTTPd package does not create that directory.

+  }, $directory, {
+    'ensure' => ensure_directory_state(pick($directory['ensure'], $ensure)),
+    'path' => '/var/www',
+  }))
+}