Skip to content

feat(report): Advisory field for pending severity (#1765) - #1768

Merged
FabioLeitao merged 1 commit into
mainfrom
feat/1765-report-advisory-field
Aug 26, 2026
Merged

FabioLeitao merged 1 commit into
mainfrom
feat/1765-report-advisory-field

Conversation

@FabioLeitao

Copy link
Copy Markdown
Collaborator

Summary

  • Add Advisory to the Excel Report info About table, next to Integrity state (#856 family): about.get("advisory", "unknown").
  • Loud text only — same spirit as -alpha. No refuse path, no trust clamp, no change to trust_level.
  • Air-gap / missing payload → unknown (never empty, never none). Beacon-shaped payload with no sev → none. Pending sev → e.g. sev=critical · 1.7.4 → 1.7.6 · fix disponível · visto em ….
  • Scanner still does not call the beacon; _release_advisory_payload() is fail-open (None today) so reports keep generating.

Closes #1765

Test plan

  • Negative proof: with production Advisory row absent, test_report_info_advisory_* failed (KeyError: 'Advisory').
  • After the row: pending severity shows the alert string; absent key → unknown and _build_report_info / generate_report still succeed.
  • ./scripts/check-all.sh OK on this branch.

Pending release severity is a loud About-table line next to Integrity
state. Missing data stays unknown; generation never fails on this field.

Refs: #1765
@strix-security

strix-security Bot commented Aug 26, 2026 •

Copy link
Copy Markdown

Strix Security Review

No security issues found.

Updated for dd813c7.


Reviewed by Strix
Re-run review · Configure security review settings

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FabioLeitao has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@FabioLeitao
FabioLeitao merged commit 613d00a into main Aug 26, 2026
25 checks passed
@FabioLeitao
FabioLeitao deleted the feat/1765-report-advisory-field branch August 26, 2026 12:55
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.

[P2][feature][report] Campo Advisory no relatório — severidade pendente visível pro auditor, sem bloquear nada

1 participant