Skip to content

writer-json-sarif: propagate the "imp" flag as level #173

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

Merged
merged 1 commit into from
Apr 23, 2024

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