-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow defining multiple outputs #11
Comments
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 |
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 GitHub<https://urldefense.com/v3/__https://github.com/GitGuardian/ggshield-action/issues/11*issuecomment-1774652213__;Iw!!EIXh2HjOrYMV!fOlHRurCwpNj8ZoIHbk_vTC7g0l6gYNfAdH2fP0LVzjjtFhIZ1m8D-Slx7w4oPtIi9Vpl5i0-JOFmqX2PmHo0GcW2WjK$>, or unsubscribe<https://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.
+=============================================================+
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
The text was updated successfully, but these errors were encountered: