External threat-detection artifact omits retry diagnostics

Aperta Adatta ai principianti
#73,006 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
github-actions, go
Ambito
ci-cd, devops

Direzione di ricerca

Leggi pkg/workflow/threat_detection_external.go, in particolare buildUploadDetectionArtifactStep, e confrontalo con buildUploadDetectionLogStep in pkg/workflow/threat_detection_steps.go. Esamina poi la copertura correlata dei percorsi di caricamento generati. Il lavoro è completo quando l’artefatto esterno include detection_harness_retry.json se presente, rispetta le condizioni di oscuramento esistenti e i test di regressione verificano i percorsi generati.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

ai-inspected

In gh-aw v1.0.1, the threat-detection artifact upload omits /tmp/gh-aw/threat-detection/detection_harness_retry.json. However, the conclusion job’s usage collector expects that source file and copies it to /tmp/gh-aw/usage/detection/harness_retry.json.

Because detection and conclusion run in separate jobs, any retry diagnostics written to the source path cannot reach the usage artifact without being included in the detection upload.

Evidence
Steps to reproduce
  1. Use gh-aw v1.0.1.
  2. Compile a workflow.
  3. Inspect the generated Upload threat detection artifact step.
  4. Compare its paths with the conclusion job’s Upload usage artifact paths.

The usage upload lists usage/detection/harness_retry.json, but the detection upload does not transfer its source file.

Expected behavior

Include detection_harness_retry.json in the external detection artifact when present, preserving the existing redaction conditions.

Impact

Retry diagnostics can be lost between jobs, reducing troubleshooting visibility.

Suggested fix

Add the source path to buildUploadDetectionArtifactStep and regression coverage for its inclusion in the generated upload paths.

Lingua principale
Go
Stelle
5.5k
Fork
606
Merge medio
4h 47m
PR unite (30g)
797

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di github/gh-aw

Tutte le issue di github/gh-aw

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.