Closed snripa closed 4 years ago
In general, looks good to me.
I would only change following non-crucial thing:
GatewaySession
interface (currently with method sessionId()
only) and implement it by WebsocketGatewaySession
and RSocketGatewaySession
GatewaySession
interface type instead of SESSION
generic type in SessionEventsHandler
(in this case you also can safely remove sessionId(SESSION session)
method)
Generalize gateway events handler and apply same logic to RSocket Gateway.
Fixes https://github.com/scalecube/scalecube-gateway/issues/91