Skip to content

chore(deps): remove uuid, use built-in crypto#105

Merged
cacieprins merged 2 commits intomasterfrom
deps/rm-uuid
Apr 30, 2026
Merged

chore(deps): remove uuid, use built-in crypto#105
cacieprins merged 2 commits intomasterfrom
deps/rm-uuid

Conversation

@cacieprins
Copy link
Copy Markdown

@cacieprins cacieprins commented Apr 30, 2026

PR checklist

  • I have run npm test locally and all tests are passing.
  • I have added/updated tests for any new behavior.
  • If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link]

Summary

Removes the uuid dependency and generates UUIDs with Node’s built-in crypto.randomUUID():

  • Digest authentication (lib/auth.js): cnonce when qop is set — still strips hyphens as before (randomUUID().replace(/-/g, '')).
  • Multipart (lib/multipart.js): form boundary string.

Breaking / compatibility

Area Change
Engines package.json requires Node >= 14.17.0 (minimum version that provides crypto.randomUUID()).
Behavior Same high-level behavior (random UUID-shaped strings); dependency surface shrinks. Minor differences could only matter if callers depended on the exact uuid package implementation rather than the output shape.

Risk note

Medium: Engine requirement is stricter than before, and random values are used for auth and multipart boundaries — worth sanity-checking digest-auth and multipart uploads against real backends if you rely on those paths.

@cypress-app-bot
Copy link
Copy Markdown

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 70413a9. Configure here.

Comment thread lib/auth.js
@cacieprins cacieprins self-assigned this Apr 30, 2026
@cacieprins cacieprins merged commit 12b38c9 into master Apr 30, 2026
1 of 4 checks passed
@cypress-app-bot
Copy link
Copy Markdown

🎉 This PR is included in version 4.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants