modified: certbot_dns_netbox/dns_netbox.py

Fix bug

README.md

@@ -4,7 +4,7 @@ Certbot DNS-01 authenticator plugin that updates the challenge TXT records in
 [NetBox DNS](https://github.com/sys4/netbox-plugin-dns) (the `netbox-dns`
 NetBox plugin) via the bundled `nbapi.py` API helper.
 
-***DISCLAIMER: THIS PROJECT IS VIBE CODED USING AN LLM.***
+***DISCLAIMER: THIS PROJECT IS A 100% VIBE-CODED PRODUCT.***
 
 <a href="https://pypi.org/project/certbot-dns-netbox/">
 <img src="https://img.shields.io/pypi/v/certbot-dns-netbox" alt="PyPI">
@@ -163,11 +163,10 @@ for sub-domains (`-d sub.example.com`) use a record named
 
 ### 3. Create a service user
 
-* Create a user group, e.g. `ACME Clients`, and assign the permission from
+* Create a user, e.g. `certbot`, without a usable login, and assign the
-  step 2 to the **group** — any service user added to the group later
+  permission from step 2 directly to the **user**.
-  inherits it.
+* Optionally create a group, e.g. `Service Accounts`, purely for organizing
-* Create a user, e.g. `certbot`, without a usable login, and make it a member
+  service accounts, and make the user a member of it.
-  of the group.
 
 ### 4. Pre-provision the TXT records
 

certbot_dns_netbox/dns_netbox.py

@@ -148,9 +148,13 @@ class _NetboxDnsClient:
     def update_txt_record(self, record_fqdn: str, validation: str) -> None:
         """Write the challenge token into one of the pre-provisioned records.
 
-        Each challenge token claims its own record so that several tokens
+        Records claimed earlier in this certbot run are skipped so that
-        for the same name (e.g. apex and wildcard on one certificate) stay
+        several tokens for the same name (e.g. apex and wildcard on one
-        live simultaneously.
+        certificate) stay live simultaneously. Among the remaining records,
+        the least recently updated one is chosen, according to the
+        timestamps NetBox keeps on the records themselves — that selection
+        needs no shared state, so separate certbot invocations for domains
+        sharing a record name still claim different records.
         """
         fqdn = self._fqdn(record_fqdn)
         records = self._get_txt_records(fqdn)
@@ -164,23 +168,32 @@ class _NetboxDnsClient:
                 )
                 return
 
-        existing = sorted(
+        free = [
-            (record for record in records if not record.get("managed")),
+            record
-            key=lambda record: record["id"],
+            for record in records
-        )
+            if not record.get("managed") and record["id"] not in self._claimed
-        free = [record for record in existing if record["id"] not in self._claimed]
+        ]
         if not free:
             raise errors.PluginError(
                 f"Not enough TXT records for {record_fqdn} in view "
-                f"{search_view!r}: {len(existing)} record(s) exist, but all of "
+                f"{search_view!r}: all existing records are already in use "
-                "them are already in use by this certificate request. Add "
+                "by this certificate request. Add another TXT record with "
-                "another TXT record with the same name and rerun certbot."
+                "the same name and rerun certbot."
             )
 
-        target = free[0]
+        # least recently updated record first; ties break on the record id.
+        target = min(
+            free,
+            key=lambda record: (
+                record.get("last_updated") or "",
+                record["id"],
+            ),
+        )
         patch = {"id": target["id"], "value": validation}
         if self._ttl:
             patch["ttl"] = self._ttl
         self._nbapi.modify_dns_record(patch)
         self._claimed.add(target["id"])
-        logger.info("Updated TXT record %s (id %s).", target["fqdn"], target["id"])
+        logger.info(
+            "Updated TXT record %s (id %s).", target["fqdn"], target["id"]
+        )

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "certbot-dns-netbox"
-version = "1.0.1"
+version = "1.0.2"
 description = "NetBox DNS (netbox-plugin-dns) DNS-01 authenticator plugin for Certbot"
 readme = "README.md"
 requires-python = ">=3.9"