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

message_client.c does not compile #5

Closed
annerajb opened this issue May 30, 2017 · 3 comments
Closed

message_client.c does not compile #5

annerajb opened this issue May 30, 2017 · 3 comments

Comments

@annerajb
Copy link

It seems message_client.c is referencing old functions that are still on the .h files

axc_handshake_accept
axc_handshake_acknowledge

Are this suppose to be implemented are they needed?

@gkdr
Copy link
Owner

gkdr commented May 31, 2017

libsignal contained code for synchronous session establishment that was later removed.
I assume this is because you can use the existing asynchronous functions in a synchronous setting by simply sending a request for the data which usually gets uploaded to a server.

When I updated this library, I didn't update the demo code (because it seems this lib isn't too popular anyway, so I thought it's not a hurry. I wrote this on the readme:

Unfortunately it is currently broken as the synchronous code was removed.

In any case: No, they are not necessary, and compiling this demo is not necessary for the lib to work. But please leave this issue open as a reminder for me to fix it! Until then, I hope the documentation I wrote in the .h files is enough.

@fbrausse fbrausse mentioned this issue Aug 1, 2019
@hartwork
Copy link
Contributor

@gkdr can this be closed as fixed? Seems to be fixed by commit 1c60f49.

@gkdr
Copy link
Owner

gkdr commented Jan 31, 2022

4 years... better late than never 😬

@gkdr gkdr closed this as completed Jan 31, 2022
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 a pull request may close this issue.

3 participants