Skip to content

Conversation

schlenk
Copy link
Contributor

@schlenk schlenk commented Aug 14, 2024

The code mistreated hashes for Blake2b and SHA3.
Code for explicitly handling SHA1 & BLAKE3 was added, as those have no variants defined in the CycloneDX specification.

fixes #652

@schlenk schlenk requested a review from a team as a code owner August 14, 2024 10:16
@jkowalleck jkowalleck changed the title Fix broken HashType.from_composite_str method for Blake2b, SHA3, Blake3 fix: HashType.from_composite_str for Blake2b, SHA3, Blake3 Aug 14, 2024
@jkowalleck jkowalleck added the bug Something isn't working label Aug 14, 2024
@jkowalleck jkowalleck changed the title fix: HashType.from_composite_str for Blake2b, SHA3, Blake3 feat: HashType.from_composite_str for Blake2b, SHA3, Blake3 Aug 14, 2024
@jkowalleck jkowalleck added enhancement New feature or request and removed bug Something isn't working labels Aug 14, 2024
Copy link
Member

@jkowalleck jkowalleck 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 the contribution, @schlenk

@jkowalleck jkowalleck added the bug Something isn't working label Aug 14, 2024
@jkowalleck jkowalleck merged commit c59036e into CycloneDX:main Aug 14, 2024
41 checks passed
@schlenk schlenk deleted the iss_652 branch August 14, 2024 13:32
@jkowalleck
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] HashType.from_composite_str fails to recognize BLAKE2b
2 participants