api icon indicating copy to clipboard operation
api copied to clipboard

Add Postgres adapter for socket communication

Open AndreyGoranov opened this issue 2 years ago • 4 comments

Since we have more than one server, we need this adapter as a way to notify all the servers of a specific event, so all the clients get notified. For example, we receive a valid payment on only one server, and we need a way to notify all the users who might be connected to the other server.

AndreyGoranov avatar Feb 22 '23 15:02 AndreyGoranov

✅ Tests will run for this PR. Once they succeed it can be merged.

github-actions[bot] avatar Feb 22 '23 15:02 github-actions[bot]

Please provide context about why this is needed and what purpose does it serve

imilchev avatar Feb 27 '23 18:02 imilchev

Would like to know more about this before it gets merged

https://docs.nestjs.com/websockets/adapter https://socket.io/docs/v4/postgres-adapter/

AndreyGoranov avatar Mar 06 '23 11:03 AndreyGoranov

If i correctly understood the socket.io adapter docs, this is only needed in cases when we want to emit messages across multiple server instances, which does not seem to be the case for us, at least for now.

sashko9807 avatar Jul 09 '23 16:07 sashko9807