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

Move event triggers to after the connection timer is reset #92

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

felippeduran
Copy link
Member

Hey there,

We’re seeing a bad behaviour and we need your help to pinpoint the issue. The symptom is the following:

  1. The app goes to background and we force a disconnection on a connected pitaya instance.
  2. The app returns to foreground and we receive the disconnection callback
  3. We try to call connect again, but it fails with “connect timeout”

Notes:

  • We have connectivity at all times
  • The flow surprisingly works if we delay the “connect” asynchronously for 50ms

All of this leads me to believe that the issue might be related to a bad clean up/reset of the connection timer, and investigating the code, I found several places where we actually trigger the disconnection events before we reset it (see the changes in this PR). So I wonder if it might get triggered and interrupt the newly started connection before being reset.

Also, I see that when initializing the timer, we don't reset its state here, but I couldn't fully track if this is called when we initialize the connection flow. I think it's called only during the static initialization flow, so this might not be related to the issue 🤔.

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

Successfully merging this pull request may close these issues.

1 participant