Skip to content

feat: use sha256 for cache entries in fips mode#375

Merged
pi0 merged 13 commits intounjs:mainfrom
kricsleo:fix/fips-hash
Jul 23, 2025
Merged

feat: use sha256 for cache entries in fips mode#375
pi0 merged 13 commits intounjs:mainfrom
kricsleo:fix/fips-hash

Conversation

@kricsleo
Copy link
Copy Markdown
Member

resolves #340

@kricsleo
Copy link
Copy Markdown
Member Author

Deno does not support crypto.getFips() denoland/deno#18455

Co-authored-by: Pooya Parsa <pyapar@gmail.com>
Copy link
Copy Markdown
Member

@pi0 pi0 left a comment

Choose a reason for hiding this comment

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

Thanks!

@pi0 pi0 changed the title fix: use sha256 in FIPS mode fix: use sha256 for cache entries in fips mode Apr 17, 2025
@pihish
Copy link
Copy Markdown

pihish commented May 1, 2025

Is there a timeline to merge this into the next release?

@lamerson28
Copy link
Copy Markdown

Hey @pi0 we're blocked on this too. Happy to lend a hand if I can, anything to get this out asap.

Copy link
Copy Markdown
Member

@pi0 pi0 left a comment

Choose a reason for hiding this comment

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

(tested locally)

@pi0 pi0 changed the title fix: use sha256 for cache entries in fips mode feat: use sha256 for cache entries in fips mode Jul 23, 2025
@pi0 pi0 merged commit 63e907f into unjs:main Jul 23, 2025
7 checks passed
@xnox
Copy link
Copy Markdown

xnox commented Jul 23, 2025

Really nice implementation!

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.

compatibility with node-fips

5 participants