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

Sent message was not marked as sent in the local database #1193

Closed
daniele-athome opened this issue Jul 26, 2018 · 1 comment
Closed

Sent message was not marked as sent in the local database #1193

daniele-athome opened this issue Jul 26, 2018 · 1 comment
Assignees
Labels
bug Something isn't working
Milestone

Comments

@daniele-athome
Copy link
Member

I had a reported case of a group message not set to "sent" status in the database, despite the SM ack was correctly received by the server. Check the whole message confirmation workflow.

What apparently happened:

  • message was sent from resending queue after connection was authenticated (log present: "resending message...")
  • ack came through but the message was not set to "sent"
  • a couple of minutes later network changes and connection is restarted
  • message is sent again right after authentication (no log this time. What?!?) and confirmed, but status was not sent (still pending)
  • message is sent once again after the multicast service replies with discovery information (this time it's the correct workflow, with "resending message" log line)
  • message was confirmed by server but still not set to "sent" status
  • at the next reconnection, the message is correctly sent again, this time the status was set to "sent"

Ack handled message count by the server was always correct, so I'm assuming the problem is client-side.

@daniele-athome daniele-athome added the bug Something isn't working label Jul 26, 2018
@daniele-athome daniele-athome self-assigned this Jul 26, 2018
daniele-athome added a commit that referenced this issue Jul 31, 2018
@daniele-athome daniele-athome added this to the Next milestone Jul 31, 2018
@daniele-athome
Copy link
Member Author

Possibly fixed in 32d6d14.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant