Fix freeradius::module::files

acctusersfile and preproxy_usersfile should be configured by default as
undef.

manifests/module/files.pp

@@ -3,16 +3,16 @@
 # Create e file module configuration for FreeRADIUS
 #
 define freeradius::module::files (
-  $ensure                     = 'present',
-  String $moddir              = "\${modconfdir}/\${.:instance}",
-  Optional[String] $key       = undef,
-  String $filename            = "\${moddir}/authorize",
-  Optional[String] $usersfile = undef,
-  String $acctusersfile       = "\${moddir}/accounting",
-  String $preproxy_usersfile  = "\${moddir}/pre-proxy",
-  Array[Hash] $users          = [],
-  Optional[String] $source    = undef,
-  Optional[String] $content   = undef,
+  $ensure                              = 'present',
+  String $moddir                       = "\${modconfdir}/\${.:instance}",
+  Optional[String] $key                = undef,
+  String $filename                     = "\${moddir}/authorize",
+  Optional[String] $usersfile          = undef,
+  Optional[String] $acctusersfile      = undef,
+  Optional[String] $preproxy_usersfile = undef,
+  Array[Hash] $users                   = [],
+  Optional[String] $source             = undef,
+  Optional[String] $content            = undef,
 ) {
   $fr_moduleconfigpath = $::freeradius::params::fr_moduleconfigpath
   $fr_group            = $::freeradius::params::fr_group

templates/files.erb

@@ -34,6 +34,10 @@ files <%= @name %> {
 
   #  These are accepted for backwards compatibility.
   #  They will be renamed in a future release.
+<%- if @acctusersfile -%>
   acctusersfile = ${moddir}/accounting
+<%- end -%>
+<%- if @preproxy_usersfile -%>
   preproxy_usersfile = ${moddir}/pre-proxy
+<%- end -%>
 }