fix: parse relative zone record names safely

Author: MNametissa

o2switch_cli/core/dns_service.py

@@ -83,7 +83,13 @@ def _record_name(cls, item: dict[str, object], root_domain: str) -> str:
         raw_name = item.get("dname") or item.get("name") or item.get("domain")
         if raw_name in (None, ""):
             raw_name = cls._decode_b64_text(item.get("dname_b64")) or root_domain
-        return canonical_record_name(str(raw_name), root_domain)
+        token = str(raw_name).strip()
+        zone_name = normalize_hostname(root_domain)
+        if not token or token == "@":
+            return zone_name
+        if token.endswith("."):
+            return canonical_record_name(token, zone_name)
+        return f"{token.lower().rstrip('.')}.{zone_name}"
 
     @staticmethod
     def _require_serial(operation: str, root_domain: str, serial: int | None) -> int:

tests/test_dns_service.py

@@ -267,6 +267,25 @@ def test_find_records_decodes_base64_zone_entries() -> None:
     assert matches[0].value == "203.0.113.25"
 
 
+def test_find_records_treats_non_trailing_dot_names_as_relative_zone_tokens() -> None:
+    _, service = build_service(
+        [
+            {
+                "dname_b64": b64("name.ginutech.com"),
+                "record_type": "A",
+                "data_b64": [b64("203.0.113.25")],
+                "ttl": 300,
+                "line_index": 9,
+            }
+        ],
+        serial=None,
+    )
+    assert service.find_records("name.ginutech.com") == []
+    malformed = service.find_records("name.ginutech.com.ginutech.com")
+    assert len(malformed) == 1
+    assert malformed[0].name == "name.ginutech.com.ginutech.com"
+
+
 def test_delete_rejects_ambiguous_records_without_force() -> None:
     _, service = build_service(
         [