The maximum number of subscribed DynDNS domains is configurable

src/domain.py

@@ -182,10 +182,10 @@ def domain_add(operation_logger, domain, subscribe=None, no_subscribe=False):
         if ((subscribe==None) == (no_subscribe==False)):
             raise YunohostValidationError("domain_dyndns_instruction_unclear")
 
-        from yunohost.dyndns import _guess_current_dyndns_domain
+        from yunohost.dyndns import is_subscribing_allowed
 
         # Do not allow to subscribe to multiple dyndns domains...
-        if _guess_current_dyndns_domain() != (None, None):
+        if not is_subscribing_allowed():
             raise YunohostValidationError("domain_dyndns_already_subscribed")
 
     operation_logger.start()

src/dyndns.py

@@ -48,6 +48,16 @@ logger = getActionLogger("yunohost.dyndns")
 
 DYNDNS_PROVIDER = "dyndns.yunohost.org"
 DYNDNS_DNS_AUTH = ["ns0.yunohost.org", "ns1.yunohost.org"]
+MAX_DYNDNS_DOMAINS = 1
+
+def is_subscribing_allowed():
+    """
+    Check if the limit of subscribed DynDNS domains has been reached
+
+    Returns:
+        True if the limit is not reached, False otherwise
+    """
+    return len(glob.glob("/etc/yunohost/dyndns/*.key"))<MAX_DYNDNS_DOMAINS
 
 
 def _dyndns_available(domain):
@@ -98,7 +108,7 @@ def dyndns_subscribe(operation_logger, domain=None, key=None, password=None):
         operation_logger.data_to_redact.append(password)
         assert_password_is_strong_enough("admin", password)
 
-    if _guess_current_dyndns_domain() != (None, None):
+    if not is_subscribing_allowed():
         raise YunohostValidationError("domain_dyndns_already_subscribed")
 
     if domain is None:
@@ -122,7 +132,6 @@ def dyndns_subscribe(operation_logger, domain=None, key=None, password=None):
     key_file = f"/etc/yunohost/dyndns/K{domain}.+165+1234.key"
 
     if key is None:
-        if len(glob.glob("/etc/yunohost/dyndns/*.key")) == 0:
         if not os.path.exists("/etc/yunohost/dyndns"):
             os.makedirs("/etc/yunohost/dyndns")
 
@@ -275,16 +284,27 @@ def dyndns_update(
     import dns.tsigkeyring
     import dns.update
 
-    # If domain is not given, try to guess it from keys available...
+    # If domain is not given, update all DynDNS domains
-    key = None
     if domain is None:
-        (domain, key) = _guess_current_dyndns_domain()
 
-    if domain is None:
+        from yunohost.domain import domain_list
+
+        domains = domain_list(exclude_subdomains=True,auto_push=True)["domains"]
+        pushed = 0
+        for d in domains:
+            if is_yunohost_dyndns_domain(d):
+                dyndns_update(d, force=force, dry_run=dry_run)
+                pushed+=1
+        if pushed==0:
             raise YunohostValidationError("dyndns_no_domain_registered")
+        return
+
+    elif type(domain).__name__ in ["list","tuple"]:
+        for d in domain:
+            dyndns_update(d, force=force, dry_run=dry_run)
+        return
 
     # If key is not given, pick the first file we find with the domain given
-    elif key is None:
     keys = glob.glob(f"/etc/yunohost/dyndns/K{domain}.+*.key")
 
     if not keys:
@@ -424,34 +444,3 @@ def dyndns_update(
         print(
             "Warning: dry run, this is only the generated config, it won't be applied"
         )
-
-
-def _guess_current_dyndns_domain():
-    """
-    This function tries to guess which domain should be updated by
-    "dyndns_update()" because there's not proper management of the current
-    dyndns domain :/ (and at the moment the code doesn't support having several
-    dyndns domain, which is sort of a feature so that people don't abuse the
-    dynette...)
-    """
-
-    DYNDNS_KEY_REGEX = re.compile(r".*/K(?P<domain>[^\s\+]+)\.\+165.+\.key$")
-
-    # Retrieve the first registered domain
-    paths = list(glob.iglob("/etc/yunohost/dyndns/K*.key"))
-    for path in paths:
-        match = DYNDNS_KEY_REGEX.match(path)
-        if not match:
-            continue
-        _domain = match.group("domain")
-
-        # Verify if domain is registered (i.e., if it's available, skip
-        # current domain beause that's not the one we want to update..)
-        # If there's only 1 such key found, then avoid doing the request
-        # for nothing (that's very probably the one we want to find ...)
-        if len(paths) > 1 and _dyndns_available(_domain):
-            continue
-        else:
-            return (_domain, path)
-
-    return (None, None)