fix: set owner/group to defaults

- defaults set via hiera, works over multiple OS's

manifests/defaults.pp

@@ -25,7 +25,7 @@ class bind::defaults (
     $chroot_class           = undef,
     $chroot_dir             = undef,
 ) {
-    unless is_bool($supported) {
+    unless $supported.is_a(Boolean) {
         fail('Please ensure that the dependencies of the bind module are installed and working correctly')
     }
 

manifests/key.pp

@@ -4,7 +4,7 @@ define bind::key (
     $secret      = undef,
     $secret_bits = 256,
     $algorithm   = 'hmac-sha256',
-    $owner       = 'root',
+    $owner       = $::bind::defaults::bind_user,
     $group       = $::bind::defaults::bind_group,
     $keydir      = $::bind::keydir::keydir,
     $keyfile     = undef,
@@ -35,7 +35,7 @@ define bind::key (
         ensure  => present,
         owner   => $owner,
         group   => $group,
-        mode    => '0640',
+        mode    => '0400',
         replace => $replace,
         content => template('bind/key.conf.erb'),
     }

manifests/keydir.pp

@@ -2,9 +2,13 @@
 
 class bind::keydir (
     $keydir,
+    $owner = $::bind::defaults::bind_user,
+    $group = $::bind::defaults::bind_group,
 ) {
     file { $keydir:
         ensure => directory,
         mode   => '0755',
+        owner  => $owner,
+        group  => $group,
     }
 }

metadata.json

@@ -1,12 +1,12 @@
 {
   "name": "inkblot-bind",
   "version": "7.4.0",
-  "author": "inkblot",
+  "author": "souldo",
   "license": "Apache-2.0",
-  "summary": "ISC BIND name server",
-  "source": "git://github.com/inkblot/puppet-bind.git",
-  "project_page": "https://github.com/inkblot/puppet-bind",
-  "issues_url": "https://github.com/inkblot/puppet-bind/issues",
+  "summary": "ISC BIND name server <-- forked from inkblot",
+  "source": "git://github.com/matthewrstone/puppet-bind.git",
+  "project_page": "https://github.com/matthewrstone/puppet-bind",
+  "issues_url": "https://github.com/matthewrstone/puppet-bind/issues",
   "tags": [ "bind", "dns", "dnssec", "nsupdate" ],
   "operatingsystem_support": [
     {
@@ -15,7 +15,7 @@
     },
     {
       "operatingsystem": "Ubuntu",
-      "operatingsystemrelease": [ "12.04", "14.04", "16.04" ]
+      "operatingsystemrelease": [ "22.04" ]
     },
     {
       "operatingsystem": "CentOS",
@@ -27,8 +27,7 @@
     }
   ],
   "dependencies": [
-    { "name": "puppetlabs/stdlib", "version_requirement": ">= 4.15.0 < 5.0.0" },
+    { "name": "puppetlabs/stdlib", "version_requirement": ">= 4.15.0 < 10.0.0" },
     { "name": "puppetlabs/concat", "version_requirement": ">= 2.2.1 < 5.0.0" }
-  ],
-  "data_provider": "hiera"
+  ]
 }

templates/named.conf.erb

@@ -27,14 +27,13 @@ options {
 	forward <%= @forward %>;
 <%- end -%>
 	auth-nxdomain <%= @auth_nxdomain ? 'yes' : 'no' %>;
+	listen-on { any; };
 	listen-on-v6 { any; };
-	dnssec-enable <%= @dnssec ? 'yes' : 'no' %>;
 <%- if @filter_ipv6 -%>
 	filter-aaaa-on-v4 yes;
 <%- end -%>
 <%- if @dnssec -%>
 	dnssec-validation yes;
-	dnssec-lookaside auto;
 <%-   if @isc_bind_keys -%>
         bindkeys-file "<%= @isc_bind_keys %>";
 <%-   end -%>