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

tox_friend_send_lossless_packet return true, but send message failed #558

Closed
randoms opened this issue Jun 5, 2017 · 1 comment
Closed
Labels
P3 Low priority
Milestone

Comments

@randoms
Copy link

randoms commented Jun 5, 2017

  1. Start two tox node, tox1 and tox2. Tox1 and tox2 are connected.
  2. Kill tox2 and call tox_friend_send_lossless_packet in tox1 immediately, tox_friend_send_lossless_packet will return true, but in fact the message was failed to send.
@iphydf iphydf added this to the v0.2.x milestone Jul 16, 2018
@iphydf iphydf added the P3 Low priority label Apr 27, 2020
@iphydf
Copy link
Member

iphydf commented Feb 4, 2022

Toxcore has no persistence. Lossless packets are only mildly guaranteed lossless. We need a better approach to actual lossless transmissions and toxcore is probably the wrong place to do this. I think lossless packets inside toxcore should go away entirely and be replaced by a layer on top of lossy packets.

@iphydf iphydf closed this as completed Feb 4, 2022
@iphydf iphydf modified the milestones: v0.2.x, meta Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 Low priority
Projects
None yet
Development

No branches or pull requests

2 participants