Hello,
Great to see some activity on here again :)
On 9 Jul 2020, at 16:10, Stefan Schantl stefan.schantl@ipfire.org wrote:
The provider entirely changed the update API, which now is not longer DynDNS2 compatible, but supports IPv6 and cleaning records.
Sadly they do not provide any detailed return codes in case an update fails.
Signed-off-by: Stefan Schantl stefan.schantl@ipfire.org
src/ddns/providers.py | 46 +++++++++++++++++++++++++++++++++++++++---- 1 file changed, 42 insertions(+), 4 deletions(-)
diff --git a/src/ddns/providers.py b/src/ddns/providers.py index 46d8a67..690516f 100644 --- a/src/ddns/providers.py +++ b/src/ddns/providers.py @@ -802,16 +802,54 @@ class DDNSProviderDtDNS(DDNSProvider): raise DDNSUpdateError
-class DDNSProviderDuckDNS(DDNSProtocolDynDNS2, DDNSProvider): +class DDNSProviderDuckDNS(DDNSProvider): handle = "duckdns.org" name = "Duck DNS" website = "http://www.duckdns.org/"
- protocols = ("ipv4",)
- protocols = ("ipv6", "ipv4",)
Yay, IPv6!
# Information about the format of the request is to be found
- # https://www.duckdns.org/spec.jsp
- # ipv4 / ipv6 records are automatically removed during the update
- # process ( clear=true ) and the ipv4 / ipv6 addresses appended.
- url = "https://www.duckdns.org/update"
- def update(self):
# Raise an error if no auth details are given.
if not self.token:
raise DDNSConfigurationError
In general, it would be nice to add a reason for this error like this:
raise DDNSConfigurationError(“Missing token”)
data = {
"domains" : self.hostname,
"token" : self.token,
"ipv6" : self.get_address("ipv6", "-"),
"ip" : self.get_address("ipv4", "-"),
"clear" : "true",
}
# Send update to the server.
response = self.send_request(self.url, data=data)
# Get the full response message.
output = response.read().decode()
# Remove all leading and trailing whitespace.
output = output.strip()
I think this is a rather pretty solution.
Could you give this to another person for testing?
Acked-by: Michael Tremer michael.tremer@ipfire.org
Best, -Michael
print(output)
# Handle success messages.
if output == "OK":
return
# The provider does not give detailed information
# if the update fails. Only a "KO" will be sent back.
if output == "KO":
raise DDNSUpdateError
# If we got here, some other update error happened.
raise DDNSUpdateError
class DDNSProviderDyFi(DDNSProtocolDynDNS2, DDNSProvider):
2.20.1
ddns mailing list ddns@lists.ipfire.org https://lists.ipfire.org/mailman/listinfo/ddns