Skip to content

feature: colorblind mode#12

Merged
marksie1988 merged 3 commits intoperiphery-security:mainfrom
NullRabbitLabs:feature/colorblind-mode
Mar 9, 2026
Merged

feature: colorblind mode#12
marksie1988 merged 3 commits intoperiphery-security:mainfrom
NullRabbitLabs:feature/colorblind-mode

Conversation

@simonmorley
Copy link
Copy Markdown

I present to you, colourblind mode, for people like me.

You're welcome.

Simon Morley added 3 commits March 6, 2026 13:12
- cve_scan/scanner.py: remove unreachable duplicate code block after return
  in _scan_service (dead code from a bad copy-paste refactor)

- cve_scan/scanner.py: fix NVD rate-limit detection using wrong HTTP status
  code; NVD returns 429 (Too Many Requests) for rate limits, not 403
  (Forbidden). Retrying on 403 was pointless and indicated an auth problem.

- password_scan/scanner.py: fix UnboundLocalError when credentials list is
  empty; variable i was unbound if the for-loop never executed, causing
  tested_count=i+1 to crash. Initialize i=-1 before the loop.

- password_scan/scanner.py: fix backward-compat scan_host() calling
  scanner.scan_host(host, ports) without the required mac argument,
  passing ports into the mac parameter and leaving ports missing.
Threads unprivileged=True from CLI through to nmap, bypassing the
privilege check and passing --unprivileged to all nmap invocations
(ping sweep, port scan, service probe) so users on macOS can run
without sudo using TCP connect scans.

Also threads verbose=True from CLI through to the scanner so
--verbose prints phase-by-phase nmap progress, discovered hosts,
and open ports as they are found.
Adds a colorblind.yaml skin using the Okabe-Ito (2008) palette —
the scientific standard for colorblind accessibility, safe for
deuteranopia, protanopia, and tritanopia.

Sky blue replaces cyan, orange replaces yellow warnings, vermillion
replaces red for danger, and all icons fall back to ASCII text forms
([OK], [X], [!]) so meaning is never conveyed by color alone.

--colorblind hot-swaps the theme before the scan runs so all
output including results, risk grades, and CVE badges renders in
the safe palette.
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 96.96970% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/edgewalker/modules/port_scan/scanner.py 94.73% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@marksie1988 marksie1988 changed the title Feature/colorblind mode feature: colorblind mode Mar 6, 2026
@marksie1988 marksie1988 merged commit 9d97e8b into periphery-security:main Mar 9, 2026
6 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants