diff options
| author | Haim Gelfenbeyn <haim@g8n.me> | 2024-05-30 09:30:27 -0400 |
|---|---|---|
| committer | Haim Gelfenbeyn <haim@g8n.me> | 2024-05-30 09:30:27 -0400 |
| commit | 19d8415512dcf87dc3a87feabf128652ffc74594 (patch) | |
| tree | f80c0b78a542494e41db13238032c9a3e00006a7 | |
| parent | 5978273c679b053dab2b878b476f0fd350a3e243 (diff) | |
| download | vyos-1x-19d8415512dcf87dc3a87feabf128652ffc74594.tar.gz vyos-1x-19d8415512dcf87dc3a87feabf128652ffc74594.zip | |
dns: T6422: allow multiple redundant NS records
NS is unlike CNAME or PTR, multiple NS records are perfectly valid and is a common use case: multiple redundant DNS servers is a common configuration and should be supported.
| -rw-r--r-- | interface-definitions/service_dns_forwarding.xml.in | 1 | ||||
| -rwxr-xr-x | src/conf_mode/service_dns_forwarding.py | 15 |
2 files changed, 15 insertions, 1 deletions
diff --git a/interface-definitions/service_dns_forwarding.xml.in b/interface-definitions/service_dns_forwarding.xml.in index b52b4bda3..5667028b7 100644 --- a/interface-definitions/service_dns_forwarding.xml.in +++ b/interface-definitions/service_dns_forwarding.xml.in @@ -311,6 +311,7 @@ <constraint> <regex>[-_a-zA-Z0-9.]{1,63}(?<!\.)</regex> </constraint> + <multi/> </properties> </leafNode> #include <include/dns/time-to-live.xml.i> diff --git a/src/conf_mode/service_dns_forwarding.py b/src/conf_mode/service_dns_forwarding.py index 7e863073a..e8318a83e 100755 --- a/src/conf_mode/service_dns_forwarding.py +++ b/src/conf_mode/service_dns_forwarding.py @@ -102,7 +102,7 @@ def get_config(config=None): 'ttl': rdata['ttl'], 'value': address }) - elif rtype in ['cname', 'ptr', 'ns']: + elif rtype in ['cname', 'ptr']: if not 'target' in rdata: dns['authoritative_zone_errors'].append(f'{subnode}.{node}: target is required') continue @@ -113,6 +113,19 @@ def get_config(config=None): 'ttl': rdata['ttl'], 'value': '{}.'.format(rdata['target']) }) + elif rtype == 'ns': + if not 'target' in rdata: + dns['authoritative_zone_errors'].append(f'{subnode}.{node}: at leaast one target is required') + continue + + for target in rdata['target']: + zone['records'].append({ + 'name': subnode, + 'type': rtype.upper(), + 'ttl': rdata['ttl'], + 'value': '{}.'.format(target) + }) + elif rtype == 'mx': if not 'server' in rdata: dns['authoritative_zone_errors'].append(f'{subnode}.{node}: at least one server is required') |