Skip to content

Conversation

kdudka
Copy link
Member

@kdudka kdudka commented Apr 22, 2024

If the "imp" flag is set on a finding, set the SARIF level to "error", which means "Important".

Related: https://issues.redhat.com/browse/OSH-565

@kdudka kdudka requested a review from hanchuntao April 22, 2024 13:35
@kdudka kdudka self-assigned this Apr 22, 2024
kdudka added a commit to kdudka/csdiff that referenced this pull request Apr 22, 2024
If the "imp" flag is set on a finding, set the SARIF level to "error",
which means "Important".

Related: https://issues.redhat.com/browse/OSH-565
Closes: csutils#173
kdudka added a commit to kdudka/csdiff that referenced this pull request Apr 22, 2024
If the "imp" flag is set on a finding, set the SARIF level to "error",
which means "Important".

Related: https://issues.redhat.com/browse/OSH-565
Closes: csutils#173
@kdudka kdudka changed the title wrtier-json-sarif: propagate the "imp" flag as level writer-json-sarif: propagate the "imp" flag as level Apr 22, 2024
If the "imp" flag is set on a finding, set the SARIF level to "error",
which means "Important".

Related: https://issues.redhat.com/browse/OSH-565
Closes: csutils#173
@kdudka
Copy link
Member Author

kdudka commented Apr 23, 2024

Verified by Camilo Cota at https://issues.redhat.com/browse/PSSECAUT-395. Merging...

@kdudka kdudka merged commit cd5d2e5 into csutils:main Apr 23, 2024
@kdudka kdudka deleted the sarif-imp branch April 23, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants