ggshield-action icon indicating copy to clipboard operation
ggshield-action copied to clipboard

Allow defining multiple outputs

Open Jiri-Stary opened this issue 1 year ago • 2 comments

Hello,

i have a suggestion - currently it is only possible to choose output either json or human readable output. I would like to get both. Other tools commonly support possibility to define multiple formats e.g. human, json ,sarif

Human output is feasible for console logs or to attach to a pull, request, while machine readable output is usable for further processing or to determine if /when to fail a build. In my use case i would like to have both

Jiri-Stary avatar Oct 10 '23 10:10 Jiri-Stary

Hi Jiri, sorry for the late reply. Adding multiple output sounds a bit tricky: I don't see how it could work with stdout piping (the ability to run ggshield secret scan path . --json | jq [...]). Do you have any ideas on how to deal with that?

agateau-gg avatar Oct 23 '23 08:10 agateau-gg

hi aurelien,

I dont have an idea, execpt for running the tool twice. In my scenario i run multiple scan tools with both human and machine output and then the goal would be to decide automatically the status of pull request and add all the info as comment.

I currently do this

ggshield secret scan path . —json | tee result.json

ggshield secret scan path . | tee result.txt

Odesláno z iPhonu

Best regards Jiri

23. 10. 2023 v 10:11, Aurelien Gateau @.***>:

 Hi Jiri, sorry for the late reply. Adding multiple output sounds a bit tricky: I don't see how it could work with stdout piping (the ability to run ggshield secret scan path . --json | jq [. . . ]). Do you have any ideas on how to deal with that?

Hi Jiri, sorry for the late reply. Adding multiple output sounds a bit tricky: I don't see how it could work with stdout piping (the ability to run ggshield secret scan path . --json | jq [...]). Do you have any ideas on how to deal with that?

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/GitGuardian/ggshield-action/issues/11*issuecomment-1774652213__;Iw!!EIXh2HjOrYMV!fOlHRurCwpNj8ZoIHbk_vTC7g0l6gYNfAdH2fP0LVzjjtFhIZ1m8D-Slx7w4oPtIi9Vpl5i0-JOFmqX2PmHo0GcW2WjK$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ARIUMG2XMZSJNCHNZ4JQKA3YAYRERAVCNFSM6AAAAAA52CS6YKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZUGY2TEMRRGM__;!!EIXh2HjOrYMV!fOlHRurCwpNj8ZoIHbk_vTC7g0l6gYNfAdH2fP0LVzjjtFhIZ1m8D-Slx7w4oPtIi9Vpl5i0-JOFmqX2PmHo0PN0ujLo$. You are receiving this because you authored the thread.Message ID: @.***>

+=============================================================+ This email is confidential and may be privileged. If you have received it in error, please notify us immediately, delete the email, and do not copy it, disclose its contents or use it for any purpose. +=============================================================+

Jiri-Stary avatar Oct 23 '23 09:10 Jiri-Stary