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

chat status don't change to seen when sender syncs after sometime #567

Closed
Assadbintahir opened this issue Mar 27, 2017 · 5 comments
Closed

Comments

@Assadbintahir
Copy link

This issue happens randomly. Sometime status changes to seen and sometimes it don't

Steps:

1- Turn internet on on both android and ios device.
2- send messages from ios to android.
3- don't read messages on android device.
4- turn internet off in ios device.
5- now read messages on android device.
6- turn internet on in ios device after sometime.
7- status don't change to seen from delivered.

(This issue happens randomly. Sometimes, status changes to seen)

@jekram
Copy link
Contributor

jekram commented Mar 27, 2017

@sumairasaeed

Since this issue is happening only on iOS this is not a server issue. Please look into this.

@sumairasaeed
Copy link
Contributor

Worked to Debug this and it found that the status list was big so the push size exceeded limit. Here is the log which shows the long list: https://papertrailapp.com/groups/1804763/events?focus=783624603049312281&q=program%3Adefault&selected=783624603049312281
and here is the error in logs which shows that the response push was discarded :
https://papertrailapp.com/groups/1804763/events?focus=783624603250638854&q=program%3Adefault&selected=783624603250638854

I have opened server side issue for solving payload size limit issue.
It runs fine in normal case when status list is small.
We will test it again after payload size issue is resolved Cloudkibo/CloudKibo#506

@jekram
Copy link
Contributor

jekram commented Mar 29, 2017

Please see my response on Chat status don't change from sent to delivered if sender syncs after sometimes. #568

@sumairasaeed
Copy link
Contributor

This has been fixed.

@jekram
Copy link
Contributor

jekram commented Apr 6, 2017

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants