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

refactor: use @projectdysnomia/libsodium to perform voice encryption #179

Draft
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

TTtie
Copy link
Member

@TTtie TTtie commented Nov 2, 2024

⚠️ This PR isn't ready for production use. Memory WILL leak.

This PR refactors the voice backend to use @projectdysnomia/libsodium for voice encryption. This package provides an universal interface between sodium-native (preferred if available) and a WASM build of libsodium in hopes to provide a reasonably fast encryption alternative for those who cannot install native modules.

TODO:

  • figure out where and when to free allocated memory
  • add CI to the libsodium project
  • release a production version of libsodium to NPM
  • test if this doesn't leak in any way

@TTtie TTtie added type: enhancement New feature or request scope: voice Issues or pull requests related to voice functionality labels Nov 2, 2024
@TTtie TTtie added this to the 0.2.0 milestone Nov 2, 2024
@TTtie TTtie self-assigned this Nov 2, 2024
@TTtie TTtie added the needs testing Additional testing is needed label Nov 2, 2024
@TTtie TTtie modified the milestones: 0.2.0, 0.2.x Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs testing Additional testing is needed scope: voice Issues or pull requests related to voice functionality type: enhancement New feature or request
Projects
Status: WIP
Development

Successfully merging this pull request may close these issues.

1 participant