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

Syft is dropping the "group" field from imported CycloneDX #1202

Open
kairabiensda opened this issue Sep 14, 2022 · 2 comments
Open

Syft is dropping the "group" field from imported CycloneDX #1202

kairabiensda opened this issue Sep 14, 2022 · 2 comments
Labels
bug Something isn't working format:cyclonedx CycloneDX related enhancement or bug needs-discussion

Comments

@kairabiensda
Copy link

kairabiensda commented Sep 14, 2022

This is based on research with @wurstbrot. It might be an issue with syft, or the CycloneDX library.

By taking a look at the output of the javascript cataloger, it doesn't use group (specified in . It places the group in front of the name.
I am using the following components/flow:
cyclonedx/cyclonedx-npm (uses group) -> image -> syft (doesn't support group) with https://github.com/CycloneDX/cyclonedx-go (does support groups) -> new-bom.json (without group attribute/prefix in name)

Is there a reason that syft doesn't use the specified group in a package? The cycloneDX specification uses
a group https://cyclonedx.org/docs/1.4/json/#components_items_group?

A workaround might be to do make the bom.json it like the javascript cataloger uses it and add the group in front of the name and remove the group attribute.

Originally posted by @wurstbrot in #1029 (comment)

@kzantow
Copy link
Contributor

kzantow commented Oct 6, 2022

Thanks @kairabiensda , this sounds like something we should be able to get done at some point in the not too distant future. A good idea!

@kairabiensda
Copy link
Author

a workaround for this is now here: #1345

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working format:cyclonedx CycloneDX related enhancement or bug needs-discussion
Projects
Status: Backlog
Development

No branches or pull requests

5 participants