-
-
Notifications
You must be signed in to change notification settings - Fork 267
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
Use rust crypto instead of libolm #1965
base: dev
Are you sure you want to change the base?
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
@ajbura I took a look at why this error was ocurring. Found this npm/cli#4828. Regenerating the package-lock.json as advised has worked. Also vite-plugin-top-level-await needs to be updated to Also looking at fe4d0be, there are some duplicate lines, so changes of that commit might need to be revised. I have done these changes (https://github.com/marekvospel/cinny/commits/js-sdk-v34/), and the build started passing. (I am seeing a lot of I hope this isn't too bold move from me. Since this is a security concern, I'd love this to be resolved as fast as possible, and this PR seemed stale. Feel free to contact me if there is anything I can help with 😉 |
Why are new features added (e.g. pinned messages) while there are open security vulnerabilities? Is there a particular blocker for this refactor? |
Adding rust crypto needs all crypto related features to be tested, this PR is waiting for #1988 so related work can be shared. |
Description
Fixes #1869
Type of change
Checklist: