Skip to content

feat: allow computing specific digests instead of all digests#776

Merged
nscuro merged 1 commit intoCycloneDX:masterfrom
jeremylong:scratch/selective-hashing
Jan 21, 2026
Merged

feat: allow computing specific digests instead of all digests#776
nscuro merged 1 commit intoCycloneDX:masterfrom
jeremylong:scratch/selective-hashing

Conversation

@jeremylong
Copy link
Copy Markdown
Contributor

@jeremylong jeremylong commented Jan 21, 2026

Implement the ability to only generate the hash/digest for specific algorithms instead of all available algorithms. On extremely large projects, the digest calculation can add a significant amount of time.

This allows you to specify exactly which algorithms to use:

      List<Hash> hashes = BomUtils.calculateHashes(
          myFile,
          Version.VERSION_16,
          Arrays.asList(Hash.Algorithm.SHA_256, Hash.Algorithm.SHA_512) 
      );

@jeremylong jeremylong requested a review from a team as a code owner January 21, 2026 18:47
@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Jan 21, 2026

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for ca1f2221 ✅ 77.50%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (ca1f222) Report Missing Report Missing Report Missing
Head commit (6c105bd) 6678 5063 75.82%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#776) 40 31 77.50%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct. ↩

stevespringett
stevespringett approved these changes Jan 21, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@stevespringett stevespringett left a comment

Choose a reason for hiding this comment

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

LGTM

nscuro
nscuro reviewed Jan 21, 2026
View reviewed changes
@nscuro nscuro added the enhancement New feature or request label Jan 21, 2026
@nscuro nscuro merged commit 0031339 into CycloneDX:master Jan 21, 2026
7 checks passed
@jeremylong-snc
Copy link
Copy Markdown

jeremylong-snc commented Feb 5, 2026

@nscuro any chance this will be included in a release soon?

@nscuro
Copy link
Copy Markdown
Member

nscuro commented Feb 5, 2026

@jeremylong-snc Just released 12.1.0. Should be on Central soon.

https://github.com/CycloneDX/cyclonedx-core-java/releases/tag/cyclonedx-core-java-12.1.0

@jeremylong-snc
Copy link
Copy Markdown

jeremylong-snc commented Feb 5, 2026

thank you! Really appreciate all your work maintaining this (and other projects).

@ctran ctran mentioned this pull request Feb 23, 2026
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stevespringett stevespringett stevespringett approved these changes

@nscuro nscuro nscuro approved these changes

@mr-zepol mr-zepol Awaiting requested review from mr-zepol

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jeremylong @jeremylong-snc @nscuro @stevespringett