python-netgear-switch-library 0.0.post346__tar.gz → 0.0.post347__tar.gz

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post347}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-netgear-switch-library
-Version: 0.0.post346
+Version: 0.0.post347
 Summary: Python library and CLI to query and control Netgear switches over SNMP, NSDP and HTTP.
 Author-email: Tim Ansell <me@mith.ro>
 License-Expression: Apache-2.0

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post347}/src/netgear_switch/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.0.post346'
-__version_tuple__ = version_tuple = (0, 0, 'post346')
+__version__ = version = '0.0.post347'
+__version_tuple__ = version_tuple = (0, 0, 'post347')
 
 __commit_id__ = commit_id = None

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post347}/src/netgear_switch/capabilities.py

@@ -175,16 +175,21 @@ WRITE_OPERATIONS: tuple[Operation, ...] = (
         "set_hostname",
         OperationKind.WRITE,
         "Set the switch's host name",
-        # SNMP (sysName, confirmed writable on all five reachable switches) and
-        # the FASTPATH CLI (`hostname`) are what the library writes today.
+        # SNMP (sysName), NSDP (tag 0x0003) and the FASTPATH CLI (`hostname`),
+        # each confirmed writable against real hardware:
         #
-        # This restriction makes a REAL GAP visible rather than hiding it: the
-        # Plus models have neither backend, so this library currently cannot
-        # rename a gs110emx, gs105pe or gs305ep at all. Both routes exist on the
-        # devices -- NSDP carries a writable HOSTNAME tag and two web UIs expose
-        # a switch_name field -- so this is unbuilt work, not a device limit,
-        # and the support table should show the hole until it is filled.
-        backends=frozenset({Backend.SNMP}) | _CLI_BACKENDS,
+        #   SNMP  all five reachable switches accepted a SET of the value they
+        #         already held -- a zero-impact writability probe
+        #   NSDP  live round trip on gs110emx3 (10.1.5.27): recorded the prior
+        #         name, wrote a throwaway, read it back, restored, re-read
+        #   CLI   round-trips on all four CLI models
+        #
+        # That closes the Plus-model hole this restriction used to expose: every
+        # registered model now has at least one backend that can rename it. HTTP
+        # is still absent -- two web UIs carry a switch_name field but the write
+        # is unbuilt -- so the restriction stays, and states what the library
+        # serves rather than what the devices can do.
+        backends=frozenset({Backend.SNMP, Backend.NSDP}) | _CLI_BACKENDS,
     ),
     Operation(
         "upload_certificate",

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post347}/src/netgear_switch/nsdp_write.py

@@ -33,6 +33,7 @@ from .models import VlanMode
 from .nsdp_read import AsyncNsdpReader, NsdpReader
 from .protocols.nsdp.protocol import Tag
 from .protocols.nsdp.write import (
+    hostname_tlv,
     ipv4_tlv,
     pvid_tlv,
     vlan_destroy_tlv,
@@ -132,6 +133,28 @@ class NsdpWriter:
     def _vlan(self, vlan: int) -> VLANInfo | None:
         return next((v for v in self._reader.get_vlans() if v.vlan_id == vlan), None)
 
+    def set_hostname(self, name: str, *, force: bool = False) -> None:
+        """Set the switch's host name over NSDP (tag 0x0003).
+
+        The Plus family's only write route for this: those switches have no SNMP
+        agent and no CLI, so without this they cannot be renamed at all.
+
+        Not force-gated -- renaming cannot strand a switch, and it is reversible
+        by writing the old name back. Verified by re-reading the tag.
+        """
+        del force  # accepted for a uniform writer signature; nothing to gate
+        if not name.strip():
+            raise ValueError("hostname must not be empty")
+        before = self._reader.get_hostname()
+        self.client.write([hostname_tlv(name)], password=self._password)
+        after = self._reader.get_hostname()
+        if after != name:
+            raise WriteVerificationError(
+                f"host name is {after!r} after writing {name!r}",
+                before=before,
+                after=after,
+            )
+
     def set_pvid(self, port: int, vlan: int, *, force: bool = False) -> None:
         # LIVE-VERIFIED over NSDP v2 on a GS110EMX (fw 1.0.2.8): PORT_PVID
         # (0x3000) IS writable (the reference spec's READ-ONLY marking was

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post347}/src/netgear_switch/protocols/nsdp/write.py

@@ -157,6 +157,17 @@ def port_name_tlv(port: int, name: str) -> TLVEntry:
     return TLVEntry(Tag.PORT_NAME, bytes([port]) + name.encode("utf-8"))
 
 
+def hostname_tlv(name: str) -> TLVEntry:
+    """Host-name write TLV (tag 0x0003), the same shape the read decodes.
+
+    The read side is measured -- ``parsers`` decodes this tag as plain text and
+    three live GS110EMX report their names through it -- so the write is that
+    encoding with nothing added: the bare name, ASCII, no length prefix and no
+    port byte (unlike ``port_name_tlv`` above, whose tag is indexed by port).
+    """
+    return TLVEntry(Tag.HOSTNAME, name.encode("ascii"))
+
+
 def ipv4_tlv(tag: Tag, dotted: str) -> TLVEntry:
     return TLVEntry(tag, socket.inet_aton(dotted))