9c0d390365
Verified dead-code + duplication cleanup over the WireGuard feature surface (found via an audit workflow, every item adversarially re-verified by grep): Dead code removed: - lang keys not_configured_title / not_configured_body (both locales) — orphaned when the unconfigured empty state became the in-page setup form (setup_title / setup_intro). 0 references anywhere. de/en parity kept (83/83). - WgTraffic::series() 'buckets' return key — written, never read by blade/route/test. - blade $fmtB byte-formatter — byte-identical duplicate of the top-level $fmtBytes already in scope; collapsed onto $fmtBytes. DRY / readability (behavior-preserving): - Index: PEER_NAME_RE constant (was the same regex literal at 5 sites), portInRange() + clampWindow() helpers (window default now self::WINDOWS[0]), openConfirm() helper (5 confirm-modal openers shared one dispatch), onFlag() helper (2 apply handlers). - WgBridge: validPort() helper (port check was duplicated verbatim). - clusev-wg.sh: _write_server_conf() shared by both setup paths (the byte-identical server bring-up was copy-pasted); single CIDR_RE constant for the 3 subnet checks; fixed a stale header comment that still claimed SSH is "NEVER matched" (the v0.9.40 ssh-lock can match 22); documented the defence-in-depth re-validation in _set_*. - WgStatus: documented server.pubkey is part of the /wg-status.json contract (kept). No DB/migration changes (peer_name write-only column left intact — needs a coordinated migration). 348 tests pass, shellcheck clean, Pint clean, R12 verified in both the configured + unconfigured states (200, no console errors, no leaked keys, traffic formatter renders 1.5 MB / 200 KB correctly). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| Index.php | ||