-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Added sarif template #558
Added sarif template #558
Conversation
@knqyf263 kindle review |
@simar7 Could you review this PR? We need to upload a SARIF file to GitHub actually and check how it looks on the UI. |
Is this valid SARIF? For me it doesn't pass the validator checks. You can try this here https://sarifweb.azurewebsites.net/Validation I think you could also use a simple JSON validator as it should pass that too. |
Furthermore, could you also add a test that would exercise this template? You can see an example here trivy/integration/client_server_test.go Lines 78 to 87 in aa20adb
|
@simar7 fixed all comments, also validated the final output in Sarif validator |
* Added sarif template * Updated readme * fixed tests * Added integration tests and fixed all sarif validations issues * Added tests for endWithPeriod * Fixed tests, and added sarif golden file * removed optional newline sequence
* chore(deps): update defsec * fix: update tests with changes * fix test * fix: lint fixes Signed-off-by: Owen Rumney <owen.rumney@aquasec.com>
Issue