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

[FIX] Some fixes #110

Open
wants to merge 5 commits into
base: multiple-clients
Choose a base branch
from
Open

[FIX] Some fixes #110

wants to merge 5 commits into from

Conversation

djorkaeffalexandre
Copy link

@djorkaeffalexandre djorkaeffalexandre commented Mar 19, 2020

  • Configurable Custom Headers to rest/websocket
  • Resolve any send (unsub need an id emit)
  • Clear timeout/interval at close function instead onClose, apparently sometimes onClose is called with code 1000 by other handler
  • reopen immediately when called instead after timeout
  • send some message await an open event when send before the websocket readyState equal 1
  • resubscribe with the same id, no duplicate streams
    • Open a stream, minimize the app, back after 60 seconds, it'll call subscribeAll and send the same id of the previous streams
  • Handle open retry

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