Gorzej jak się zmieni impl. getActiveConnections, bo dojdzie jakaś logika związana np. z rozróżnianiem hostów od ich procesów. Ja rozumiem że to w tym momencie działa ale to jest zły design. Eventy connection są dla ConnectionManagera. On powienien robić co należy (za niedługo będzie to dodanie socketa do odpowiedniej listy pewnie) i emitować jego gotowość. I na ten event powinien reagować ten callback. W tym momencie będzie to dokładnie to samo ale nie będzie podatne na zmiany logiki.