Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace uuid package with built-in Crypto library for generating UUIDs #1152

Merged
merged 2 commits into from
Apr 6, 2023

Conversation

nelsonni
Copy link
Member

@nelsonni nelsonni commented Apr 6, 2023

Entities stored in the Redux store are currently uniquely identified using the uuidjs/uuid library, but this dependency can be entirely removed in favor of the built-in JavaScript Crypto library. Removing separate libraries in favor of built-ins reduces the package size, attack surface, and robustness of Synectic.

This PR resolves #1116.

Changes:

This PR makes the following changes:

@nelsonni nelsonni added the dependencies Issues or updates to dependency files label Apr 6, 2023
@nelsonni nelsonni merged commit cb9748e into main Apr 6, 2023
@nelsonni nelsonni deleted the fix/crypto-replaces-uuid branch April 6, 2023 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Issues or updates to dependency files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate and replace uuid library with built-in JavaScript Crypto library
1 participant