Skip to content

✨ Add support for operating_system/container asset parsing in cyclonedx sboms#6317

Merged
chris-rock merged 2 commits intomainfrom
preslav/improve-cyclonedx-conversion
Dec 17, 2025
Merged

✨ Add support for operating_system/container asset parsing in cyclonedx sboms#6317
chris-rock merged 2 commits intomainfrom
preslav/improve-cyclonedx-conversion

Conversation

@preslavgerchev
Copy link
Copy Markdown
Contributor

This PR ensures that operating_system and container sboms get an improved platform/platform ids so that we can improve the asset detection

…dx sboms.

Signed-off-by: Preslav <preslav@mondoo.com>

// if we have no generator info, fallback to trying tools. these are deprecated
// but might still be present
if sbom.Generator == nil && bom.Metadata.Tools.Tools != nil {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is purely fallback in case the tools are present and we couldnt get a generator

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 17, 2025

Test Results

4 703 tests  +3   4 699 ✅ +3   2m 22s ⏱️ +2s
  415 suites ±0       4 💤 ±0 
   31 files   ±0       0 ❌ ±0 

Results for commit d881d0c. ± Comparison against base commit 138f87d.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Member

@chris-rock chris-rock left a comment

Choose a reason for hiding this comment

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

Looks great @preslavgerchev Lets reduce the testdata to the specific parts we want to test.

Signed-off-by: Preslav <preslav@mondoo.com>
@chris-rock chris-rock merged commit d276853 into main Dec 17, 2025
20 checks passed
@chris-rock chris-rock deleted the preslav/improve-cyclonedx-conversion branch December 17, 2025 21:04
@github-actions github-actions bot locked and limited conversation to collaborators Dec 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants