Skip to content

Validate network topology telemetry - #54

Merged
vikranthreddimasu merged 2 commits into
masterfrom
validate-network-telemetry
Aug 3, 2026
Merged

vikranthreddimasu merged 2 commits into
masterfrom
validate-network-telemetry

Conversation

@vikranthreddimasu

@vikranthreddimasu vikranthreddimasu commented Aug 3, 2026 •

Copy link
Copy Markdown
Owner
  • Summary: strictly validate peer bandwidth, latency, packet-loss, and MTU telemetry before route scoring.
  • Summary: omit invalid local telemetry from advertisements, emit standards-compliant JSON, and reject link lists above the eight-link protocol limit.
  • Security changes: malformed or oversized unauthenticated open-fleet topology data now fails closed; authentication and credential handling are unchanged.
  • Usability changes: bad sensor or peer data can no longer create misleading route scores or invalid discovery payloads.
  • Tests/checks run: focused topology and discovery tests passed (47 passed).
  • Tests/checks run: make lint passed (Ruff plus mypy).
  • Tests/checks run: full suite passed (1069 passed, 3 skipped, 2 warnings).
  • Tests/checks run: wheel build, git diff --check, and diff-only secret scan passed.
  • Tests/checks run: all 11 GitHub Actions checks passed.
  • Commit: 4cc7c3a Validate peer network telemetry.
  • Commit: 9a3ef5a Harden topology advertisements.
  • Risks / manual review notes: low-risk parsing and serialization boundary change; older valid numeric JSON remains accepted; no auth, authorization, credential, destructive-action, or deployment-secret code changed.
  • Merge status: merged into master after local and GitHub checks passed.

@vikranthreddimasu
vikranthreddimasu merged commit 19589d2 into master Aug 3, 2026
11 checks passed
@vikranthreddimasu
vikranthreddimasu deleted the validate-network-telemetry branch August 3, 2026 17:00
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.

1 participant