External threat-detection artifact omits retry diagnostics
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 82/100
Piste de recherche
Lisez pkg/workflow/threat_detection_external.go, en particulier buildUploadDetectionArtifactStep, et comparez-le à buildUploadDetectionLogStep dans pkg/workflow/threat_detection_steps.go. Examinez ensuite la couverture associée des chemins de téléversement générés. Le travail est terminé lorsque l’artefact externe inclut detection_harness_retry.json si le fichier est présent, respecte les conditions de masquage existantes et que les tests de régression vérifient les chemins générés.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
buildUploadDetectionArtifactStepuploads the detection result, execution evidence, and usage files, but omits the retry-diagnostics file.collect_usage_artifact_files.shexplicitly expects and copies that file.- The alternative
buildUploadDetectionLogStepalready includes it.
Steps to reproduce
- Use gh-aw v1.0.1.
- Compile a workflow.
- Inspect the generated Upload threat detection artifact step.
- 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.
- Langage dominant
- Go
- Étoiles
- 5.5k
- Forks
- 606
- Merge moyen
- 4 h 18 min
- PR mergées (30 j)
- 796
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de github/gh-aw
-
agent-research
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
automation cli cookie documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
github/gh-aw#72983 · 13 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
automation cli cookie documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
Pi on arc-dind: an aliased model reaches pi empty (`Model "aw-gateway/" not found`)Peut-être pris @pelikhan l’a pris aujourd’hui. Ouverteai-inspected community
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
github/gh-aw#72530 · 1 réaction · 2 personnes assignées ·
Les mainteneurs répondent en général sous 1 jour
-
[doc-healer] DDUw improvement: retry integrity-filtered issue reads with direct single-item lookupsOuverteautomation documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de github/gh-aw
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
orchestration.migration.data.tasks.retry values are ignored by the 8.8 data migration jobPeut-être pris @irisfield l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
camunda/camunda-platform-helm#7773 ·
Les mainteneurs répondent en général sous 1 jour
-
Add missing unit tests for IsAdmissionPolicyName and IsCertManagerResourceNamePeut-être pris @a1h8 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
kubernetes-sigs/security-profiles-operator#3644 ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: Anthropic Responses converter forwards empty text blocks, upstream rejects with 400Peut-être pris @david-choi-querypie l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
[DOCS]: enforce_admins description understates what it covers and omits push restrictionsPeut-être pris @dav-tb l’a pris aujourd’hui. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
integrations/terraform-provider-github#3707 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour