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.

Sign up for GitHub

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

Jump to bottom

[httputils] Possible memory leak in EventBusHandler #187

Open
bweston92 opened this issue Feb 28, 2018 · 1 comment
Open

[httputils] Possible memory leak in EventBusHandler #187

bweston92 opened this issue Feb 28, 2018 · 1 comment
Labels
bug

Comments

@bweston92
Copy link

I'm not to sure but httputils.EventBusHandler adds an observer to event publisher but never removes it after the request has ended.

https://github.com/looplab/eventhorizon/blob/master/httputils/eventbus.go
@bweston92 bweston92 mentioned this issue Feb 28, 2018
Closed
@maxekman
Copy link
Member

maxekman commented Mar 1, 2018

Good find, thanks for the PR.

@maxekman maxekman changed the title Possible memory leak in httputils.EventBusHandler [httputils] Possible memory leak in EventBusHandler Mar 21, 2018
@maxekman maxekman added the bug label Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Milestone
No milestone
Development

No branches or pull requests
2 participants
@maxekman @bweston92