Moar dyndns yagni: we don't need to be able to specify ipv4/ipv6/keyfile during dyndns update

data/actionsmap/yunohost.yml

@@ -1501,12 +1501,6 @@ dyndns:
                     help: Full domain to update
                     extra:
                         pattern: *pattern_domain
-                -k:
-                    full: --key
-                    help: Public DNS key
-                -i:
-                    full: --ipv4
-                    help: IP address to send
                 -f:
                     full: --force
                     help: Force the update (for debugging only)
@@ -1515,9 +1509,6 @@ dyndns:
                     full: --dry-run
                     help: Only display the generated zone
                     action: store_true
-                -6:
-                    full: --ipv6
-                    help: IPv6 address to send
 
         ### dyndns_installcron()
         installcron:

src/yunohost/dyndns.py

@@ -174,9 +174,6 @@ def dyndns_subscribe(
 def dyndns_update(
     operation_logger,
     domain=None,
-    key=None,
-    ipv4=None,
-    ipv6=None,
     force=False,
     dry_run=False,
 ):
@@ -185,15 +182,12 @@ def dyndns_update(
 
     Keyword argument:
         domain -- Full domain to update
-        key -- Public DNS key
-        ipv4 -- IP address to send
-        ipv6 -- IPv6 address to send
-
     """
 
     from yunohost.dns import _build_dns_conf
 
     # If domain is not given, try to guess it from keys available...
+    key = None
     if domain is None:
         (domain, key) = _guess_current_dyndns_domain()
 
@@ -201,8 +195,7 @@ def dyndns_update(
         raise YunohostValidationError("dyndns_no_domain_registered")
 
     # If key is not given, pick the first file we find with the domain given
-    else:
+    elif key is None:
-        if key is None:
         keys = glob.glob("/etc/yunohost/dyndns/K{0}.+*.private".format(domain))
 
         if not keys:
@@ -267,14 +260,8 @@ def dyndns_update(
     old_ipv6 = resolve_domain(domain, "AAAA")
 
     # Get current IPv4 and IPv6
-    ipv4_ = get_public_ip()
+    ipv4 = get_public_ip()
-    ipv6_ = get_public_ip(6)
+    ipv6 = get_public_ip(6)
-
-    if ipv4 is None:
-        ipv4 = ipv4_
-
-    if ipv6 is None:
-        ipv6 = ipv6_
 
     logger.debug("Old IPv4/v6 are (%s, %s)" % (old_ipv4, old_ipv6))
     logger.debug("Requested IPv4/v6 are (%s, %s)" % (ipv4, ipv6))
@@ -379,6 +366,7 @@ def _guess_current_dyndns_domain():
     # Retrieve the first registered domain
     paths = list(glob.iglob("/etc/yunohost/dyndns/K*.private"))
     for path in paths:
+        # MD5 is legacy ugh
         match = RE_DYNDNS_PRIVATE_KEY_MD5.match(path)
         if not match:
             match = RE_DYNDNS_PRIVATE_KEY_SHA512.match(path)