Skip to content
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

HdfConverter: Map Impact=0 to Kind=Not Applicable #2701

Merged
merged 3 commits into from
Jul 27, 2023

Conversation

candrews
Copy link
Collaborator

Per @Amndeep7 and @aaronlippold of the MITRE SAF team, which is the team that manages the Heimdall Data Format (HDF), impact of 0 indicates not applicable. See: #2695 (comment)

@michaelcfanning
Copy link
Member

Can you add something like this to ReleaseHistory.md? Obviously, update this text as you see fit to reflect the work/intent. :)

BUG: Improve HdfConverter1 SARIF conversion to properly map various properties (e.g., kind, level, rank`) and generally prepare converted SARIF for ingestion to GitHub Advanced Security.

@candrews
Copy link
Collaborator Author

Can you add something like this to ReleaseHistory.md?

Done!

@michaelcfanning michaelcfanning enabled auto-merge (squash) July 27, 2023 01:24
@michaelcfanning
Copy link
Member

OK! Should automerge shortly. You need a release after this or do we have more good stuff coming?

@candrews
Copy link
Collaborator Author

A release would be fantastic!

I'm sure there's no more coming... But this is all I know of at the moment :-)

@michaelcfanning michaelcfanning merged commit 9acc5c8 into main Jul 27, 2023
7 checks passed
@michaelcfanning michaelcfanning deleted the hdfconverter-impact-0 branch July 27, 2023 01:31
@candrews
Copy link
Collaborator Author

I'm looking forward to the release, thank you again!

@michaelcfanning
Copy link
Member

In flight.

@michaelcfanning
Copy link
Member

@candrews https://www.nuget.org/packages/Sarif.Sdk/4.3.0. thanks for the contributions! really helpful.

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