Use template for Caddyfile

REFERENCE.md

@@ -354,17 +354,17 @@ Default value: `true`
 
 ##### <a name="-caddy--caddyfile_source"></a>`caddyfile_source`
 
-Data type: `Stdlib::Filesource`
+Data type: `Optional[Stdlib::Filesource]`
 
 Caddyfile source.
 
-Default value: `'puppet:///modules/caddy/etc/caddy/Caddyfile'`
+Default value: `undef`
 
 ##### <a name="-caddy--caddyfile_content"></a>`caddyfile_content`
 
 Data type: `Optional[String[1]]`
 
-Caddyfile content. Always preferred over caddyfile_source.
+Caddyfile content.
 
 Default value: `undef`
 

manifests/config.pp

@@ -29,14 +29,24 @@
   }
 
   if $caddy::manage_caddyfile {
-    # caddyfile_content is always preferred over caddyfile_source when set
+    # Prefer source over content if both are defined
+    # Fallback to the bundled template if both are unset
+    $real_source  = $caddy::caddyfile_source
+    $real_content = if $caddy::caddyfile_source { undef } else {
+      $caddy::caddyfile_content.lest || {
+        epp('caddy/etc/caddy/caddyfile.epp',
+          config_dir => $caddy::config_dir,
+        )
+      }
+    }
+
     file { '/etc/caddy/Caddyfile':
       ensure  => file,
       mode    => '0444',
       owner   => $caddy::caddy_user,
       group   => $caddy::caddy_group,
-      source  => if $caddy::caddyfile_content { undef } else { $caddy::caddyfile_source },
-      content => $caddy::caddyfile_content,
+      source  => $real_source,
+      content => $real_content,
     }
   }
 }

manifests/init.pp

@@ -118,7 +118,7 @@
 #   Caddyfile source.
 #
 # @param caddyfile_content
-#   Caddyfile content. Always preferred over caddyfile_source.
+#   Caddyfile content.
 #
 # @param config_dir
 #  Where to store Caddy configs
@@ -160,7 +160,7 @@
   String[1]                      $package_name                    = 'caddy',
   String[1]                      $package_ensure                  = $version,
   Boolean                        $manage_caddyfile                = true,
-  Stdlib::Filesource             $caddyfile_source                = 'puppet:///modules/caddy/etc/caddy/Caddyfile',
+  Optional[Stdlib::Filesource]   $caddyfile_source                = undef,
   Optional[String[1]]            $caddyfile_content               = undef,
   Boolean                        $purge_config_dir                = true,
 ) {

spec/classes/init_spec.rb

@@ -116,7 +116,8 @@
             'owner' => 'caddy',
             'group' => 'caddy',
             'mode' => '0444',
-            'source' => 'puppet:///modules/caddy/etc/caddy/Caddyfile'
+            'source' => nil,
+            'content' => %r{^import /etc/caddy/config/\*\.conf$}
           ).
             that_requires('File[/etc/caddy]')
         end
@@ -337,6 +338,19 @@
 
         it { is_expected.to contain_file('/etc/caddy/Caddyfile').with_source(nil).with_content("localhost\nfile_server\n") }
       end
+
+      context 'with both caddyfile_source and caddyfile_content set' do
+        let(:params) do
+          {
+            caddyfile_source: 'http://example.com/Caddyfile',
+            caddyfile_content: "localhost\nfile_server\n",
+          }
+        end
+
+        it 'prefers source over content' do
+          is_expected.to contain_file('/etc/caddy/Caddyfile').with_source('http://example.com/Caddyfile').with_content(nil)
+        end
+      end
     end
   end
 end

templates/etc/caddy/caddyfile.epp

@@ -0,0 +1,8 @@
+<%- |
+  String[1] $config_dir,
+| -%>
+#
+# THIS FILE IS MANAGED BY PUPPET
+#
+
+import <%= $config_dir %>/*.conf