Created a PR to Support Redis for message passing between client #195
Conversation
|
I myself am in need of this feature as well. Perhaps it is however better to abstract the redis portion, to allow for different backends such as memcache. Also there's quite a few console logs left. |
|
yeah i will remove the console.log. Let me check how we can abstract the Radis part away. Thanks |
| } | ||
| cert: "", | ||
| }, | ||
| redis: false, |
There was a problem hiding this comment.
i'd rather suggest to use an object to describe redis config:
redis: {
host: "",
port: 0,
}
| @@ -8,6 +8,8 @@ import { Client, IClient } from "../../models/client"; | |||
| import { IRealm } from "../../models/realm"; | |||
| import { MyWebSocket } from "./webSocket"; | |||
|
|
|||
| const Redis = require("ioredis"); | |||
|
Like @KaiserKarel said, we should use an abstraction to transfer/handle messages. I have an idea how to do it. I suggest to move this PR to separate package to extends peerjs server with Redis. Others stuff like RabbitMQ, Memcache etc also should be separated packages. |
|
Hey just wondering if there are any updates on this PR? would be intersted to try it out |
|
THIS PROJECT SEEMS COOL THX |
Ability to use Redis Pub Sub to pass messaged between webrtc clients connected across mutiple servers.