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.

Sign up for GitHub

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

Jump to bottom

Fix: Correctly map algorithm name in CycloneDx #2207

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Fix: Correctly map algorithm name in CycloneDx #2207

merged 1 commit into from
Oct 9, 2023

Conversation

sharief007
Copy link
Contributor

Change Log

  • Fixed the algorithm look up logic by converting the key (algorithm name) to lower case.
  • Added unit test for toCycloneDXAlgorithm.

Resolves: #2183

The example from the above issue now produces the following final output:
sboms.zip

@sharief007
Fix algo lookup by converting key to lower case
bff91a4 
Signed-off-by: Mohammad Sharief Baig <shariefmohammad007@gmail.com>
@spiffcs spiffcs merged commit d16ecdf into anchore:main Oct 9, 2023
9 checks passed
@chenrui333 chenrui333 mentioned this pull request Oct 10, 2023
Merged
GijsCalis pushed a commit to GijsCalis/syft that referenced this pull request Feb 19, 2024
@sharief007
Fix algo lookup by converting key to lower case (anchore#2207)
a2bc6e4 
Signed-off-by: Mohammad Sharief Baig <shariefmohammad007@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spiffcs spiffcs spiffcs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Converting a SBOM looses the algorithm type for added checksums
2 participants
@sharief007 @spiffcs