looplab / eventhorizon

Event Sourcing for Go!
Apache License 2.0
1.59k stars 195 forks source link

[httputils] Possible memory leak in EventBusHandler #187

Open bweston92 opened 6 years ago

bweston92 commented 6 years ago

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

maxekman commented 6 years ago

Good find, thanks for the PR.