search

status-im / status-console-client

Status messaging console user interface
Mozilla Public License 2.0
10 stars 2 forks source link

Console client gets stuck on sending messages to whisper proto layer #50

Closed dshulyak closed 5 years ago

dshulyak commented 5 years ago

To reproduce sent multiple messages. Around 5-10 client will be stuck

adambabik commented 5 years ago

I can't send even one so it's a regression. Will assign to myself because it's the same root cause as in #58.

dshulyak commented 5 years ago

btw i can't make it hang with updated status-go https://github.com/status-im/status-console-client/pull/49 . i fixed one issue with hanged history retrieval. maybe it has effect on some other parts as well

dshulyak commented 5 years ago

yeah i think it is fixed with status-go update. there was an issue where one of the whisper feed subscribers wasn't consuming events from feed. and thus all publishers hanged. because each whisper processQueue publishes to feed - they all got stuck. and whisper.Send got stuck as well because it tries to send to the channel that should be consumed by whisper.processQueue.

dshulyak commented 5 years ago

@adambabik i merged status-go update. can you retest this issue please?

adambabik commented 5 years ago

@dshulyak just pulled master and it seems like the problem is fixed.