YunoHost/yunohost
1
0
Fork 0
mirror of https://github.com/YunoHost/yunohost.git synced 2024-09-03 20:06:10 +02:00
Code Issues Projects Releases Packages Wiki Activity

dyndns: fix dyndns_update, needs to construct the update object using the target zone, not the domain itself

Browse source
This commit is contained in:
Alexandre Aubin 2022-01-17 21:21:46 +01:00
parent 5479befabb
commit ecb8419ee0
1 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/dyndns.py
View file

@ -219,8 +219,8 @@ def dyndns_update(
return
# Extract 'host', e.g. 'nohost.me' from 'foo.nohost.me'
host = domain.split(".")[1:]
host = ".".join(host)
zone = domain.split(".")[1:]
zone = ".".join(zone)
logger.debug("Building zone update ...")
@ -229,7 +229,7 @@ def dyndns_update(
keyring = dns.tsigkeyring.from_text({f'{domain}.': key})
# Python's dns.update is similar to the old nsupdate cli tool
update = dns.update.Update(domain, keyring=keyring, keyalgorithm=dns.tsig.HMAC_SHA512)
update = dns.update.Update(zone, keyring=keyring, keyalgorithm=dns.tsig.HMAC_SHA512)
auth_resolvers = []
@ -322,11 +322,15 @@ def dyndns_update(
if not dry_run:
try:
dns.query.tcp(update, auth_resolvers[0])
r = dns.query.tcp(update, auth_resolvers[0])
except Exception as e:
logger.error(e)
raise YunohostError("dyndns_ip_update_failed")
if "rcode NOERROR" not in str(r):
logger.error(str(r))
raise YunohostError("dyndns_ip_update_failed")
logger.success(m18n.n("dyndns_ip_updated"))
else:
print(