Baguette/dnsmanager
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Delegation servers: only accept FQDN.

Browse source
This commit is contained in:
Philippe Pittoli 2025-10-04 16:34:25 +02:00
parent 9388663688
commit 6ff63b172b
1 changed files with 14 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/storage.cr
View file

@ -460,9 +460,21 @@ class DNSManager::Storage
# Wipes the zone from dnsmanager (zone db entry, generated zone file, tokens). # Wipes the zone from dnsmanager (zone db entry, generated zone file, tokens).
wipe_zone user_id, domain_name wipe_zone user_id, domain_name
# Make sure both name servers are "absolute" domain names.
ns1 = if nameserver1.ends_with? '.'
nameserver1
else
"#{nameserver1}."
end
ns2 = if nameserver2.ends_with? '.'
nameserver2
else
"#{nameserver2}."
end
# Creates the new zone. # Creates the new zone.
zone = Zone.new domain_name zone = Zone.new domain_name
zone.delegation = Zone::Delegation.new nameserver1, nameserver2 zone.delegation = Zone::Delegation.new ns1, ns2
zone.update_delegation @delegationdir zone.update_delegation @delegationdir
# Once the new delegation file has been written, the script generating the (root) zone file must # Once the new delegation file has been written, the script generating the (root) zone file must
@ -471,7 +483,7 @@ class DNSManager::Storage
zones_by_domain.update_or_create zone zones_by_domain.update_or_create zone
Response::DomainDelegated.new domain_name, nameserver1, nameserver2 Response::DomainDelegated.new domain_name, ns1, ns2
end end
# Removes user data. # Removes user data.