feat(named.conf.local.jinja): Allow "allow-update" to have multiple e…

…lements This allows to have multiple keys or a mix of keys and IP ...
saltstack-formulas · Nov 30, 2022 · c44804c · c44804c
1 parent c3b1fb4
commit c44804c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/bind/files/named.conf.local.jinja b/bind/files/named.conf.local.jinja
@@ -46,7 +46,12 @@ zone "{{ key }}" {
   inline-signing yes;
   {%- endif %}
   {%- if args['allow-update'] is defined %}
-  allow-update { {{ args['allow-update'] }}; };
+  {%- if args['allow-update'] is string %}
+  {%- set allow_update = [args['allow-update']] %}
+  {%- else %}
+  {%- set allow_update = args['allow-update'] %}
+  {%- endif %}
+  allow-update { {{ allow_update | join('; ') }}; };
   {%- endif %}
   {%- if args.update_policy is defined %}
   update-policy {

diff --git a/pillar.example b/pillar.example
@@ -261,7 +261,9 @@ bind:
 
     dynamic.domain.com:                           # Our ddns zone
       type: master                                # As above
-      allow-update: "key core_dhcp"               # Who we allow updates from (refers to above key)
+      allow-update:
+        - "key core_dhcp"                         # Who we allow updates, could be a string or an array
+        - "key other_dhcp"
       notify: true                                # Notify NS RRs of changes
 
     sub.anotherdomain.com:                        # Another domain zone