.on() doesnt register multiple listeners

[yesitsfebreeze]

Discussed in https://github.com/geckosio/geckos.io/discussions/202

^{Originally posted by **yesitsfebreeze** May 13, 2022} Would it be possible to implement the on listener in a different way to this? Currently the problem is that you cant register multiple listeners via `.on`, due to the fact that the callback is passed into the lambda. https://github.com/geckosio/geckos.io/blob/master/packages/server/src/geckos/channel.ts#L272 this would be sufficient i guess: ``` listener = (callback, ....) => { ... }.bind(callback) this.eventEmitter.on(eventName, listener) ```

[github-actions[bot]]

This issue is stale because it has been open 300 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[bananu7]

Actually, does that mean that only one listener can be active at a time? I was struggling a bit because there's no removeListener/removeAllListeners exposed...

[bananu7]

I've just tested and it seems that the subsequent calls to .on() are ignored. For reference, I worked around it by binding via a proxy:

this.channel.on('spectating', (data: Data) => {
    if (this._onSpectating)
        this._onSpectating(data);
});

That's a bit more annoying than just being able to override the handler.

[github-actions[bot]]

This issue is stale because it has been open 500 days with no activity. Remove stale label or comment or this will be closed in 10 days.