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

'capycli bom merge' removes important information #73

Closed
sachinshaji opened this issue Aug 20, 2024 · 1 comment · Fixed by #97
Closed

'capycli bom merge' removes important information #73

sachinshaji opened this issue Aug 20, 2024 · 1 comment · Fixed by #97

Comments

@sachinshaji
Copy link

Hello All,
I have tried using 'capycli bom merge' command and finds that it removes some information from the resultant file.

I have tried to merge an empty json file and a json file generated by 'trivy' in 'CycloneDX' format.

Command I used --> capycli bom merge empty.json trivy.json

When I compare the results, some informations are lost.

under the 'dependencies' section 'dependsOn' information is coming as empty in the resultant file.

Sharing a screenshot for reference.
image

Also sharing the empty.json and trivy.json files which I use to run bom merge command
trivy.json
empty.json

@tngraf
Copy link
Collaborator

tngraf commented Sep 13, 2024

I agree that it does not work.

bom merge empty.json empty.json => OK
bom merge trivy.json trivy.json => OK
bom merge trivy.json empty.json => OK
bom merge empty.json trivy.json => messed up

But bom merge just loads an SBOM, copies components and then saves the SBOM. CaPyCLI does not modify the dependencies - all this is done by the underlying cyclonedx-python-lib.

We are using version 3.1.5, the next version is 4.0.0, the latest version is 7.6.0 ... and they did a lot of changes.

@tngraf tngraf linked a pull request Nov 7, 2024 that will close this issue
@tngraf tngraf closed this as completed in #97 Nov 7, 2024
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 a pull request may close this issue.

2 participants