python-netgear-switch-library 0.0.post349__tar.gz → 0.0.post351__tar.gz

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-netgear-switch-library
-Version: 0.0.post349
+Version: 0.0.post351
 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.post349 → python_netgear_switch_library-0.0.post351}/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.post349'
-__version_tuple__ = version_tuple = (0, 0, 'post349')
+__version__ = version = '0.0.post351'
+__version_tuple__ = version_tuple = (0, 0, 'post351')
 
 __commit_id__ = commit_id = None

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/cli_write.py

@@ -712,3 +712,14 @@ class CliWriter:
         except CliTransportError:
             # Expected: the switch tore the session down while rebooting.
             return
+
+    def set_syslog_enabled(self, enabled: bool, *, force: bool = False) -> None:
+        """This backend does not serve a remote-logging toggle.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "a remote-logging toggle"
+        )

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/http_read.py

@@ -45,6 +45,9 @@ if TYPE_CHECKING:
         PortStats,
         PortStatus,
         Sensor,
+        ServiceStatus,
+        SwitchUser,
+        SyslogConfig,
     )
     from .protocols.http.endpoints import HttpModelSpec
     from .protocols.http.session import AsyncHttpSession, HttpSession
@@ -681,6 +684,39 @@ class HttpReader:
             )
         return cfg
 
+    def get_users(self) -> list[SwitchUser]:
+        """This backend does not serve local user accounts.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "local user accounts (no such tag/page/table on this backend)"
+        )
+
+    def get_services(self) -> list[ServiceStatus]:
+        """This backend does not serve management-service state.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "management-service state (http/https/telnet/ssh)"
+        )
+
+    def get_syslog(self) -> SyslogConfig:
+        """This backend does not serve remote-logging configuration.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "remote-logging configuration"
+        )
+
 
 class AsyncHttpReader:
     """Asynchronous web-UI read facade (mirror of ``HttpReader``)."""
@@ -820,3 +856,46 @@ class AsyncHttpReader:
                 ),
             )
         return cfg
+
+    async def get_users(self) -> list[SwitchUser]:
+        """This backend does not serve local user accounts.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "local user accounts (no such tag/page/table on this backend)"
+        )
+
+    async def get_services(self) -> list[ServiceStatus]:
+        """This backend does not serve management-service state.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "management-service state (http/https/telnet/ssh)"
+        )
+
+    async def get_syslog(self) -> SyslogConfig:
+        """This backend does not serve remote-logging configuration.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "remote-logging configuration"
+        )
+
+    async def get_hostname(self) -> str:
+        """This backend does not serve a host name field.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose a host name field"
+        )

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/http_write.py

@@ -742,9 +742,7 @@ class HttpWriter:
         if page.vlan_id == vlan:
             return page
         body = forms.fastpath_membership_form(page, vlan=vlan)
-        shown = parse.parse_fastpath_membership(
-            self.session.post_form(post_path, body)
-        )
+        shown = parse.parse_fastpath_membership(self.session.post_form(post_path, body))
         _require_fastpath_membership_for(shown, vlan, post_path)
         return shown
 
@@ -985,6 +983,27 @@ class HttpWriter:
                 return r.admin_enabled
         return False
 
+    def set_hostname(self, name: str, *, force: bool = False) -> None:
+        """This backend does not serve a host-name write.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose a host-name write"
+        )
+
+    def set_syslog_enabled(self, enabled: bool, *, force: bool = False) -> None:
+        """This backend does not serve a remote-logging toggle.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "a remote-logging toggle"
+        )
+
 
 class AsyncHttpWriter:
     def __init__(
@@ -1302,9 +1321,7 @@ class AsyncHttpWriter:
             ),
         )
         _raise_on_fastpath_err_flag(applied, f"management IP -> {address}/{netmask}")
-        after = parse.parse_xui_form_page(
-            await self.session.get_page(path), page=path
-        )
+        after = parse.parse_xui_form_page(await self.session.get_page(path), page=path)
         got = (
             after.fields.get(fields.address),
             after.fields.get(fields.netmask),
@@ -1346,3 +1363,24 @@ class AsyncHttpWriter:
         _check_multipart_cert_response(
             await self.session.post_multipart(path, fields, payload)
         )
+
+    async def set_hostname(self, name: str, *, force: bool = False) -> None:
+        """This backend does not serve a host-name write.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose a host-name write"
+        )
+
+    async def set_syslog_enabled(self, enabled: bool, *, force: bool = False) -> None:
+        """This backend does not serve a remote-logging toggle.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "a remote-logging toggle"
+        )

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/models.py

@@ -38,6 +38,17 @@ class PortStatus:
     # older tests) keep constructing without it; a backend that cannot read
     # ifAlias (NSDP, HTTP) leaves it honestly None rather than fabricating "".
     description: str | None = None
+    #: Whether the link negotiated FULL duplex. ``None`` when the backend cannot
+    #: tell -- a down port has no negotiated duplex, and not every backend
+    #: reports it at all. Measured 2026-08-02: the M4300's EtherLike-MIB dot3
+    #: table exposes only error counters, NOT dot3StatsDuplexStatus (column 19
+    #: is absent), so SNMP cannot answer this and leaves it None. The FASTPATH
+    #: CLI does: `show port all` reports "1000 Full" in its Physical Status
+    #: column, carrying speed and duplex together.
+    full_duplex: bool | None = None
+    #: Whether IEEE 802.3x flow control is enabled on the port ("Flow Mode" in
+    #: `show port all`). ``None`` where the backend does not report it.
+    flow_control: bool | None = None
 
 
 @dataclass(frozen=True)

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/nsdp_read.py

@@ -21,7 +21,15 @@ from .protocols.nsdp.types import LinkSpeed
 from .registry import Backend
 
 if TYPE_CHECKING:
-    from .models import LLDPNeighbor, MacEntry, PoEStatus, Sensor
+    from .models import (
+        LLDPNeighbor,
+        MacEntry,
+        PoEStatus,
+        Sensor,
+        ServiceStatus,
+        SwitchUser,
+        SyslogConfig,
+    )
     from .protocols.nsdp.client import AsyncNsdpClient, NsdpClient
     from .protocols.nsdp.types import NsdpDevice
     from .registry import SwitchModel
@@ -255,6 +263,39 @@ class NsdpReader:
     def get_poe(self) -> list[PoEStatus]:
         raise UnsupportedCapabilityError(_NO_POE)
 
+    def get_users(self) -> list[SwitchUser]:
+        """This backend does not serve local user accounts.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "local user accounts (no such tag/page/table on this backend)"
+        )
+
+    def get_services(self) -> list[ServiceStatus]:
+        """This backend does not serve management-service state.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "management-service state (http/https/telnet/ssh)"
+        )
+
+    def get_syslog(self) -> SyslogConfig:
+        """This backend does not serve remote-logging configuration.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "remote-logging configuration"
+        )
+
 
 class AsyncNsdpReader:
     """Asynchronous NSDP read facade (mirror of NsdpReader)."""
@@ -308,3 +349,36 @@ class AsyncNsdpReader:
 
     async def get_poe(self) -> list[PoEStatus]:
         raise UnsupportedCapabilityError(_NO_POE)
+
+    async def get_users(self) -> list[SwitchUser]:
+        """This backend does not serve local user accounts.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "local user accounts (no such tag/page/table on this backend)"
+        )
+
+    async def get_services(self) -> list[ServiceStatus]:
+        """This backend does not serve management-service state.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "management-service state (http/https/telnet/ssh)"
+        )
+
+    async def get_syslog(self) -> SyslogConfig:
+        """This backend does not serve remote-logging configuration.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "remote-logging configuration"
+        )

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/nsdp_write.py

@@ -305,6 +305,17 @@ class NsdpWriter:
                 after=after,
             )
 
+    def set_syslog_enabled(self, enabled: bool, *, force: bool = False) -> None:
+        """This backend does not serve a remote-logging toggle.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "a remote-logging toggle"
+        )
+
 
 class AsyncNsdpWriter:
     """Asynchronous NSDP write facade (mirror of NsdpWriter)."""
@@ -465,3 +476,24 @@ class AsyncNsdpWriter:
                 before=before,
                 after=after,
             )
+
+    async def set_hostname(self, name: str, *, force: bool = False) -> None:
+        """This backend does not serve a host-name write.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose a host-name write"
+        )
+
+    async def set_syslog_enabled(self, enabled: bool, *, force: bool = False) -> None:
+        """This backend does not serve a remote-logging toggle.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "a remote-logging toggle"
+        )

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/protocols/cli/parse.py

@@ -244,6 +244,11 @@ def parse_version(text: str, models: Mapping[str, SwitchModel]) -> DetectedModel
 #   Link Status | Link Trap | LACP Mode | Flow Mode
 _PORT_INTF, _PORT_TYPE, _PORT_ADMIN = 0, 1, 2
 _PORT_PHYS_MODE, _PORT_PHYS_STATUS, _PORT_LINK = 3, 4, 5
+# Flow Mode is the LAST column of `show port all` (Intf, Type, Admin Mode,
+# Physical Mode, Physical Status, Link Status, Link Trap, LACP Mode, Flow
+# Mode). Indexed from the end so a firmware that omits an intermediate
+# column cannot silently shift it.
+_PORT_FLOW = -1
 
 _SPEED_RE = re.compile(r"(\d+)\s*([GgMm]?)")
 
@@ -261,13 +266,30 @@ def _speed_mbps(phys_status: str) -> int | None:
     return value * 1000 if m.group(2).upper() == "G" else value
 
 
+def _duplex(phys_status: str) -> bool | None:
+    """FULL/HALF from the Physical Status cell, e.g. "1000 Full" -> True.
+
+    Blank on a down port, which has no negotiated duplex -- honestly None rather
+    than a fabricated False.
+    """
+    text = phys_status.strip().lower()
+    if "full" in text:
+        return True
+    if "half" in text:
+        return False
+    return None
+
+
 def parse_port_status(text: str) -> list[PortStatus]:
     """``show port all`` -> per physical-port ``PortStatus``.
 
     ``admin_enabled`` = Admin Mode == "Enable"; ``link_up`` = Link Status ==
-    "Up"; ``speed_mbps`` from Physical Status (None when blank/down). ``lag N``
-    aggregation rows are skipped (not physical ports). ``description`` is
-    honestly ``None``: this command carries no ifAlias column.
+    "Up"; ``speed_mbps`` and ``full_duplex`` both from Physical Status, which
+    carries them together ("1000 Full"); ``flow_control`` from the Flow Mode
+    column. All three are ``None`` on a down port, which has negotiated nothing.
+    ``lag N`` aggregation rows are skipped (not physical ports).
+    ``description`` is honestly ``None``: this command carries no ifAlias
+    column.
     """
     out: list[PortStatus] = []
     for cells in iter_table_rows(text):
@@ -288,6 +310,16 @@ def parse_port_status(text: str) -> list[PortStatus]:
                     else None
                 ),
                 description=None,
+                full_duplex=(
+                    _duplex(cells[_PORT_PHYS_STATUS])
+                    if cells[_PORT_LINK].strip().lower() == "up"
+                    else None
+                ),
+                flow_control=(
+                    cells[_PORT_FLOW].strip().lower() == "enable"
+                    if len(cells) > _PORT_LINK + 1
+                    else None
+                ),
             )
         )
     return out

{python_netgear_switch_library-0.0.post349 → python_netgear_switch_library-0.0.post351}/src/netgear_switch/snmp_read.py

@@ -23,6 +23,8 @@ if TYPE_CHECKING:
         PortStats,
         PortStatus,
         Sensor,
+        ServiceStatus,
+        SwitchUser,
         SyslogConfig,
         VLANInfo,
     )
@@ -243,6 +245,28 @@ class SnmpReader:
         """
         return read_system_info(self.client)
 
+    def get_users(self) -> list[SwitchUser]:
+        """This backend does not serve local user accounts.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "local user accounts (no such tag/page/table on this backend)"
+        )
+
+    def get_services(self) -> list[ServiceStatus]:
+        """This backend does not serve management-service state.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "management-service state (http/https/telnet/ssh)"
+        )
+
 
 class AsyncSnmpReader:
     def __init__(self, client: AsyncSnmpClient, model: SwitchModel) -> None:
@@ -359,6 +383,30 @@ class AsyncSnmpReader:
             await w(oids.IP_ADDRESS_IFINDEX),  # RFC-4293 fallback (M4300)
         )
 
+    async def get_syslog(self) -> SyslogConfig:
+        """Async twin of ``SnmpReader.get_syslog`` -- see there."""
+        if not oids.has_vendor_oids(self.model):
+            raise UnsupportedCapabilityError(
+                f"model {self.model.key!r} registers no Netgear vendor OID "
+                "subtree, and the logging columns are vendor-only; an empty "
+                "result here would be indistinguishable from a switch with no "
+                "syslog collectors configured"
+            )
+        vo = oids.vendor_oids(self.model)
+        w = self.client.walk
+        return parse.parse_syslog(
+            await self.client.get([vo.syslog_admin_mode]),
+            await self.client.get([vo.syslog_local_port]),
+            await w(vo.syslog_host_addr),
+            await w(vo.syslog_host_port),
+            await w(vo.syslog_host_severity),
+            await w(vo.syslog_host_status),
+            addr_base=vo.syslog_host_addr,
+            port_base=vo.syslog_host_port,
+            severity_base=vo.syslog_host_severity,
+            status_base=vo.syslog_host_status,
+        )
+
     async def get_hostname(self) -> str:
         """Async twin of ``SnmpReader.get_hostname`` -- see there."""
         return parse.parse_hostname(await self.client.get([oids.SYS_NAME]))
@@ -366,3 +414,25 @@ class AsyncSnmpReader:
     async def get_system_info(self) -> DetectedModel:
         """Async twin of ``SnmpReader.get_system_info`` -- see there."""
         return await async_read_system_info(self.client)
+
+    async def get_users(self) -> list[SwitchUser]:
+        """This backend does not serve local user accounts.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "local user accounts (no such tag/page/table on this backend)"
+        )
+
+    async def get_services(self) -> list[ServiceStatus]:
+        """This backend does not serve management-service state.
+
+        Refused by name rather than returned empty: an empty answer here
+        would be indistinguishable from a switch that genuinely has none.
+        """
+        raise UnsupportedCapabilityError(
+            f"model {self.model.key!r}: this backend does not expose "
+            "management-service state (http/https/telnet/ssh)"
+        )