Skip to content

js: Replace dependency uuid with native crypto.randomUUID()#2286

Merged
svix-jplatte merged 1 commit into
svix:mainfrom
ulrichstark:refactor(javascript)--replace-dependency-`uuid`-with-native-`crypto.randomUUID()`
Apr 23, 2026
Merged

js: Replace dependency uuid with native crypto.randomUUID()#2286
svix-jplatte merged 1 commit into
svix:mainfrom
ulrichstark:refactor(javascript)--replace-dependency-`uuid`-with-native-`crypto.randomUUID()`

Conversation

@ulrichstark

Copy link
Copy Markdown
Contributor

Motivation

Same motivation as #2285 to fix the reported security vulnerability, but instead of upgrading uuid, I propose to replace it with the native crypto.randomUUID(). This also reduces dependencies by 2.

I'm not sure what the baseline requirements for this project are, but crypto.randomUUID is well established.

Solution

Replace dependency uuid with native crypto.randomUUID()

@ulrichstark ulrichstark requested a review from a team April 23, 2026 08:00
@ulrichstark ulrichstark requested a review from a team as a code owner April 23, 2026 08:00

@svix-jplatte svix-jplatte left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the PR!
This makes sense to me and I prefer it over the other solution, though I'll wait to hear from others on the team before merging.
As another piece of data: caniuse.com page

@svix-jplatte svix-jplatte changed the title refactor(javascript): replace dependency uuid with native crypto.randomUUID() js: Replace dependency uuid with native crypto.randomUUID() Apr 23, 2026
@svix-jplatte svix-jplatte merged commit 51ae972 into svix:main Apr 23, 2026
25 checks passed
@ulrichstark ulrichstark deleted the refactor(javascript)--replace-dependency-`uuid`-with-native-`crypto.randomUUID()` branch April 23, 2026 11:32
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.

2 participants