-
Notifications
You must be signed in to change notification settings - Fork 11
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 libsignal-service-rs ? #5
Comments
Hi @rubdos! Actually I simply missed this amazing crate when I was googling something similar. It can significantly simplify our code! My only concern is why are you still using bindings to libsignal-client-c whereas there're great pure Rust implementation of it? For instance, it provides async-friendly stores traits (PreKeyStore, etc). Maybe I can help you with whisperfish/libsignal-service-rs#72? |
I thought you would be raising that concern. The only reason is lack of time/putting focus on other things. Please do feel free to help us with whisperfish/libsignal-service-rs#72 ! And also, please do feel free to post issues and questions. If you're into Matrix or IRC, feel free to come have a chat on #whisperfish:rubdos.be or #whisperfish on Freenode, it's where we coordinate this effort. Another unspoken rule: we still very often break public interface in libsignal-service. So if you're patching stuff, feel free to break things! |
FYI, our move to Meanwhile, we've moved to https://github.com/whisperfish/, which encompasses a bunch of third-party Signal libraries, and we're in contact with Signal about moving some primitives over to them. |
Thank you for keeping up to date! We will switch on |
I saw all this story with choosing right license (GPLv3 / AGPLv3) and signing CLA. Tricky journey! |
Hey! Full disclosure: I'm one of the main people behind https://github.com/Michael-F-Bryan/libsignal-service-rs/. I wonder why you folks rolled your own, and what would be missing in our library such that we can help each other? We already have unidentified senders (#4), for instance.
Want to team up?
The text was updated successfully, but these errors were encountered: