Skip to content

Add combined prefix overall evaluation for prefix checks - #14

Merged
DeepZone merged 1 commit into
mainfrom
codex/implement-combined-prefix-evaluation
May 19, 2026
Merged

Add combined prefix overall evaluation for prefix checks#14
DeepZone merged 1 commit into
mainfrom
codex/implement-combined-prefix-evaluation

Conversation

@DeepZone

Copy link
Copy Markdown
Owner

Motivation

  • Prefix-Prüfungen sollen RPKI- und Registry/IRR-Ergebnisse nicht nur einzeln anzeigen, sondern zu einer verständlichen Gesamtbewertung zusammenführen.
  • Die Gesamtbewertung soll konservativ arbeiten (CRITICAL über WARNING über UNKNOWN über OK) und Sonderfälle wie fehlendes origin_as sinnvoll behandeln.

Description

  • Neue zentrale Bewertungsfunktion evaluate_prefix_overall in backend/app/core/prefix_evaluation.py, die RPKI- und Registry-Status kombiniert und status, summary, explanation, risk und recommendations liefert.
  • PrefixChecker (backend/app/services/prefix_checker.py) wurde so angepasst, dass es die kombinierte Bewertung als Top-Level-Ergebnis setzt und gleichzeitig die einzelnen Checks unter checks.rpki und checks.registry inklusive Rohdaten unverändert beibehält.
  • Report-Templates (backend/app/templates/report.md.j2, backend/app/templates/report.html.j2) und UI (frontend/src/components/ReportView.tsx) wurden minimal überarbeitet, um ausdrücklich zwischen Gesamtbewertung, Einzelprüfungen und Rohdaten zu unterscheiden.
  • Tests ergänzt: backend/tests/test_prefix_overall_evaluation.py deckt die geforderten Kombinationsfälle ab; README um Abschnitt Kombinierte Prefix-Bewertung erweitert.

Testing

  • Backend-Tests ausgeführt mit cd backend && pytest -q und alle Tests erfolgreich (35 passed).
  • Frontend-Build ausgeführt mit cd frontend && npm run build und Build erfolgreich.
  • Änderungen bewahren die vorhandenen Endpoints, Rohdaten bleiben zugänglich und das System bleibt read-only.

Codex Task

@DeepZone
DeepZone merged commit 88aa91a into main May 19, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant