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

Updates to API calls for CSAF output #19

Merged
merged 2 commits into from
Sep 7, 2021

Conversation

sei-vsarvepalli
Copy link
Contributor

This is further to issue #17 , the API example get_vince.py was not updated in the last PR merge.

Vijay

@sei-vsarvepalli sei-vsarvepalli self-assigned this Aug 20, 2021
Copy link

@tschmidtb51 tschmidtb51 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please replace cvrf with csaf.

@@ -51,11 +67,12 @@ def create_response(key,turl):
result[key] = json.loads(str(r.text))

def vince_to_cvrf(vince):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
def vince_to_cvrf(vince):
def vince_to_csaf(vince):

cvrf = { "document": {
"acknowledgments": [
{
"urls": [
"https://kb.cert.org/vuls/id/"+vince["get_case"]["vuid"]
"https://kb.cert.org/vuls/id/"+case_id

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should include at least the anchor to acknowledgments section (as extracting the single values is not supported yet). The anchor must be supported by the website as well.

Suggested change
"https://kb.cert.org/vuls/id/"+case_id
"https://kb.cert.org/vuls/id/"+case_id+"#Acknowledgements"

product_name = "Unknown"
if vince["get_original_report"]["product_name"]:
product_name = vince["get_original_report"]["product_name"]
product_version = "1.0.0"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the product version is not present, it should not be specified.

@sei-vsarvepalli
Copy link
Contributor Author

Hello @tschmidtb51

Thanks for these suggestions. I will merge them soon. I was going with our own SBOM recommendations that basically decided the author of the SBOM, should provide a semver complaint version number starting at 1.0.0 if no version number is provided. But I can totally see how this can confuse the recipients if we arbitrarily give version numbers to software that we don't own.

Let me chat with our SBOM WG on this feedback and dilemma we face at times with software that has no version number specified anywhere. I will merge in your commits soon, right after guessing there will be actually no better solution than just leaving the information as "unknown"

Vijay

@sei-vsarvepalli sei-vsarvepalli merged commit 75bf364 into CERTCC:main Sep 7, 2021
@sei-vsarvepalli
Copy link
Contributor Author

Hello @tschmidtb51 @santosomar

We have now pushed this code into production. You can use your VINCE API credentials to collect CSAF output for cases you are part of. I hope you have already generated API keys.

Once you have reviewed it, we can also provide the same to the public advisories that go out today as vulnerability notice at https://kb.cert.org/

Simple collect https://kb.cert.org/vince/comm/api/case/636397/csaf/ using the API key you have for VINCE comm.

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