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 delays when in active order #1293

Open
Purchase73 opened this issue May 14, 2024 · 3 comments
Open

Chat delays when in active order #1293

Purchase73 opened this issue May 14, 2024 · 3 comments
Labels
Android bug 🐛 Something isn't working ⚡Eligible for Sats ⚡ This issue or pull request rewards bitcoin

Comments

@Purchase73
Copy link
Contributor

Describe the bug
Chat delays in an active order

To Reproduce

Steps to reproduce the behavior:

This is in the new apk, v0.6.1-alpha.
I'm not sure it can be reproduced, but in an active order, once I got to the 3 of 4 screen (chat with seller), I was able to type a message, but was NOT able to hit send. It was disabled, grayed out.
Then all at once, I got a bunch of messages from the other party in the trade, indicating that the other party had been typing messages previously, and I had not been responding. Like "hello?, anyone there" type messages.
I was then able to send messages. The trade completed. The whole thing just probably took a couple more minutes than normal, but it's disconcerting to have the chat window open but not be able to send a message.

This has never happened in any other place I've used robosats.

Smartphone (please complete the following information):

  • Device: google pixel 7a
  • OS: android
  • Browser graphene os
  • Version - build number 2024050700

Additional context
Add any other context about the problem here.

@Reckless-Satoshi
Copy link
Collaborator

Hey @Purchase73 . Indeed, this is in fact a legacy problem.

Will further explain the causes here in case anyone feels jumping into fixing this one.

The Android app uses the HTTP API of the chatroom. On browser, we use Websockets for instant messaging, but the HTTP API is also available (under the name "Turtle mode" the small icon between the two chat status indicators).

That means in Android the app polls for messages very X seconds. This coordinator endpoint might have some other issues as well that make it slow.

The Android app didn't use the Websockets for instant chatting because that was not possible with our previous Tor library. However, it is likeky that the websockets might work with the new native Tor library @KoalaSat implemented. So this issue might not be too difficult to solve anymore.

@Reckless-Satoshi Reckless-Satoshi added bug 🐛 Something isn't working ⚡Eligible for Sats ⚡ This issue or pull request rewards bitcoin Android labels May 15, 2024
@Purchase73
Copy link
Contributor Author

FYI - same issue again, second time I tried.
When you go to a different screen (like I switch from Order to Robot and back to Order again) it appears to kick-start something, and I get the messages that have been queued up.
Otherwise, I don't know if I'd ever get them.

@KoalaSat
Copy link
Member

This should be fixed with v0.7.3-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android bug 🐛 Something isn't working ⚡Eligible for Sats ⚡ This issue or pull request rewards bitcoin
Projects
None yet
Development

No branches or pull requests

3 participants