NAPALM/napalm-amt

0 Fork 0

chore: adopt the NAPALM driver standard #2

Merged

christianmanivong merged 4 commits from chore/driver-standard into main

2026-10-23 09:28:20 +00:00

Author	SHA1	Message	Date
christianmanivong	0aec58fe06	chore(release): 0.2.0 driver / standard (pull_request) Successful in 18s Details driver / lint (pull_request) Successful in 16s Details driver / types (pull_request) Successful in 55s Details driver / test (3.10) (pull_request) Successful in 1m8s Details driver / test (3.11) (pull_request) Successful in 1m7s Details driver / test (3.12) (pull_request) Successful in 59s Details driver / test (3.13) (pull_request) Successful in 49s Details driver / lowest (pull_request) Successful in 23s Details driver / security (pull_request) Successful in 47s Details driver / build (pull_request) Successful in 23s Details CI / driver (pull_request) Successful in 7m5s Details driver / publish (pull_request) Skipped Details	2026-10-23 10:28:59 +02:00
christianmanivong	c9a7ee4378	chore: adopt the NAPALM driver standard driver / standard (pull_request) Successful in 13s Details driver / lint (pull_request) Successful in 12s Details driver / types (pull_request) Successful in 43s Details driver / test (3.10) (pull_request) Successful in 46s Details driver / test (3.11) (pull_request) Successful in 48s Details driver / test (3.12) (pull_request) Successful in 43s Details driver / test (3.13) (pull_request) Successful in 39s Details driver / lowest (pull_request) Successful in 23s Details driver / security (pull_request) Successful in 46s Details driver / build (pull_request) Successful in 22s Details CI / driver (pull_request) Successful in 5m37s Details driver / publish (pull_request) Skipped Details - CI arrives in .gitea/workflows/ci.yml and calls the shared driver workflow of NAPALM/driver-standard; the repo had none before. - setuptools>=77 and the SPDX license string PEP 639 wants, without the License classifier; napalm-device-types>=4.21.1 from PyPI; Issues and Changelog links on git.netork.io. LICENSE, NOTICE and py.typed were already there and stay. - The test extra becomes the dev extra of the standard, plus responses, which the tests mock AMT with. - mypy strict with the untyped imports (napalm, requests, defusedxml) named instead of ignored globally; napalm_amt.driver is exempt for now (5 errors: getters return dicts where the role declares TypedDicts), recorded in tests/quality_baseline.json with get_event_log's complexity of 11. - Coverage floor 95 % (measured 95.87 %). - CHANGELOG.md in Keep a Changelog form; the README's development steps install from PyPI.	2026-10-23 08:37:32 +02:00
christianmanivong	76f85051cc	fix: what bandit and the remaining lint rules found - WS-Management answers are parsed with defusedxml (new dependency, defusedxml>=0.7) with DTDs forbidden. The check for "<!DOCTYPE" in the raw bytes it replaces missed a DTD in a UTF-16 answer, which expat reads too; on plain HTTP (16992) anyone in the path can send one. An answer with a DTD in any encoding is now refused as no WS-Management answer, as a UTF-8 one already was (B314, B405). - xml.sax.saxutils only escapes the request envelopes, nothing parses with SAX (nosec B406). - python -m napalm_amt.record fetches the landing page without certificate checks, as before: it sends no credentials and records what discovery sees, and AMT's certificate is self-signed unless one was provisioned (nosec B501). The driver itself still verifies by default (verify_ssl).	2026-10-23 08:35:39 +02:00
christianmanivong	87f0dff43f	style: ruff format and the lint rules of the driver standard Mechanical: the rule set every NAPALM driver uses (E, F, I, UP, B at line length 100, Python 3.10), with E501 off under tests/. The code was already formatted and clean under it; what ruff cannot fix comes in the next commit.	2026-10-23 08:32:58 +02:00