1 files changed, 9 insertions, 9 deletions
diff --git a/src/conf-mode/vyos-config-dns-forwarding.py b/src/conf-mode/vyos-config-dns-forwarding.py
index df273b321..be48cde60 100755
--- a/src/conf-mode/vyos-config-dns-forwarding.py
+++ b/src/conf-mode/vyos-config-dns-forwarding.py
@@ -23,7 +23,7 @@ import netifaces
 import jinja2
 
 from vyos.config import Config
-from vyos.util import ConfigError
+from vyos import ConfigError
 
 config_file = r'/etc/powerdns/recursor.conf'
 
@@ -31,7 +31,6 @@ config_file = r'/etc/powerdns/recursor.conf'
 # especially in the semicolon-separated lists of name servers.
 # Please be careful if you edit the template.
 config_tmpl = """
-
 ### Autogenerated by vyos-config-dns-forwarding.py ###
 
 # Non-configurable defaults
@@ -47,19 +46,20 @@ max-cache-entries={{ cache_size }}
 export-etc-hosts={{ export_hosts_file }}
 
 # listen-on
-local-address= {{ listen_on | join(',') }}
+local-address={{ listen_on | join(',') }}
 
 # domain ... server ...
 {% if domains -%}
 
-{% for d in domains -%}
-forward-zones = {{ d.name }} = {{ d.servers | join(";") }}
-{% endfor -%}
+forward-zones={% for d in domains %}
+{{ d.name }}={{ d.servers | join(";") }}
+{%- if loop.first %}, {% endif %}
+{% endfor %}
 
 {% endif %}
 
 # name-server
-forward-zones-recurse=.=  {{ name_servers | join(';') }}
+forward-zones-recurse=.={{ name_servers | join(';') }}
 
 """
 
@@ -113,7 +113,7 @@ def get_config():
 
     if conf.exists('name-server'):
         name_servers = conf.return_values('name-server')
-        dns.setdefault('name_servers', name_servers)
+        dns['name_servers'] = dns['name_servers'] + name_servers
 
     if conf.exists('system'):
         conf.set_level('system')
@@ -185,7 +185,7 @@ def generate(dns):
     if dns is None:
         return None
 
-    tmpl = jinja2.Template(config_tmpl)
+    tmpl = jinja2.Template(config_tmpl, trim_blocks=True)
 
     config_text = tmpl.render(dns)
     with open(config_file, 'w') as f: