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

Use SBOM descriptor version #1011

Merged
merged 5 commits into from
May 25, 2022

Conversation

jonasagx
Copy link
Contributor

@jonasagx jonasagx commented May 23, 2022

Fix lack of version info when Syft is used a library.

Format Pass/Fail Notes
syft-json 🟢
cyclonedx-xml 🟢
cyclonedx-json 🟢
github-0-json 🟢 Reports 0.0.0-dev
spdx-tag-value 🟢
spdx-json 🟢
table 🟢 N/A
text 🟢 N/A

Fix: #1010

Signed-off-by: Jonas Xavier [email protected]

Signed-off-by: Jonas Xavier <[email protected]>
@jonasagx jonasagx marked this pull request as draft May 23, 2022 18:46
@jonasagx jonasagx self-assigned this May 23, 2022
@jonasagx jonasagx marked this pull request as ready for review May 23, 2022 20:55
@jonasagx jonasagx requested a review from wagoodman May 23, 2022 20:55
jonasagx added 2 commits May 24, 2022 15:52
Signed-off-by: Jonas Xavier <[email protected]>
@jonasagx jonasagx requested review from wagoodman and spiffcs May 25, 2022 05:37
Copy link
Contributor

@spiffcs spiffcs left a comment

Choose a reason for hiding this comment

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

Thanks for updating the descriptor here from the earlier comments. LGTM

@jonasagx jonasagx merged commit 7cb8e1f into anchore:main May 25, 2022
@jonasagx jonasagx deleted the 1010-lib-syft-vendor-version branch May 25, 2022 21:40
@wagoodman
Copy link
Contributor

didn't get to this before the merge, can we add some explicit tests to ensure we grab the version from the descriptor and not the lib approach (so not lean on snapshot tests here)?

GijsCalis pushed a commit to GijsCalis/syft that referenced this pull request Feb 19, 2024
* Use SBOM descriptor version

Signed-off-by: Jonas Xavier <[email protected]>

* Update tests

Signed-off-by: Jonas Xavier <[email protected]>

* CycloneDX extract tools metadata in decoding stage

Signed-off-by: Jonas Xavier <[email protected]>

* add descriptor to spdx tag-value test

Signed-off-by: Jonas Xavier <[email protected]>

* remove comment

Signed-off-by: Jonas Xavier <[email protected]>
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.

Version is [not provided] when encoding to most formats
3 participants