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

Messages are delivered with delay #21850

Open
pavloburykh opened this issue Dec 19, 2024 · 2 comments
Open

Messages are delivered with delay #21850

pavloburykh opened this issue Dec 19, 2024 · 2 comments
Labels
mobile-core waku All issues relating to the Status Waku integration.
Milestone

Comments

@pavloburykh
Copy link
Contributor

cc @chaitanyaprem @richard-ramos @plopezlpz @qfran @ilmotta

Currently go develop seems to be broken in terms of data delivery. The presumed culprit commit is status-im/status-go@e7cc535

Issue has been initially reported here

Below you can see example of 1 min delay between messages.

Check out the video:

sender_receiver.mp4

Logs for sender: logs (27).zip
Logs for receiver: receiver.zip

Additional Information

  • Status version: nightly
  • Operating System: Android, iOS
@pavloburykh pavloburykh added the waku All issues relating to the Status Waku integration. label Dec 19, 2024
@chaitanyaprem
Copy link
Contributor

It could probably be due to rate-limiters added.
I have a fix for it here along with a bunch of other fixes.

I was waiting for more testing to be done, can you test the mobile PR which includes this fixes and update if things are working fine.
Then i will merge this so that develop will become sane again.

cc @pavloburykh

@pavloburykh
Copy link
Contributor Author

It could probably be due to rate-limiters added. I have a fix for it here along with a bunch of other fixes.

I was waiting for more testing to be done, can you test the mobile PR which includes this fixes and update if things are working fine. Then i will merge this so that develop will become sane again.

cc @pavloburykh

@chaitanyaprem thank you. We will perform testing of the PR and let you know about the results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mobile-core waku All issues relating to the Status Waku integration.
Projects
Status: No status
Development

No branches or pull requests

3 participants