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

Hub invocation callbacks are not cleaned up in JS client if no response is received causing a memory leak #2271

Closed
DamianEdwards opened this issue Jul 8, 2013 · 2 comments
Assignees
Milestone

Comments

@DamianEdwards
Copy link
Member

Note: This is the 2.0 version of #2270

There's a memory leak when the hub proxy never receives a valid response from the server as the callback isn't cleaned up from the global callbacks map. Suggested changes:

  • Hub invocation callbacks should be stored on the connection instance metadata member (connection._). Note it needs to be separate to the existing callback map on there used to store hub proxy client callbacks
  • For the transports that use Ajax sends, the fail delegate should clear the callback delegate
  • For WebSockets, the callback map for that connection should be completely cleared on the reconnect event
  • For all transports, the callback map for that connection should be completely cleared on the disconnect event
@ghost ghost assigned NTaylorMullen Jul 8, 2013
@ghost ghost assigned DamianEdwards Jul 19, 2013
@DamianEdwards
Copy link
Member Author

Fixed in 42c7575

@ghost ghost assigned gustavo-armenta Jul 19, 2013
@gustavo-armenta
Copy link
Contributor

tested behavior is the same as in #2270

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

No branches or pull requests

3 participants