Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Full control on the key option and uid added #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

antongorodezkiy
Copy link

Key option: needed for socket.io-redis compatibility, because currently socket.io-redis is using prefix + '#' + namespace + '#' and prefix + '#' + namespace + '#' + room + '#' formats of keys.

uid option added to make compatible with socket.io-emitter (1.0.0)

Key option: needed for socket.io-redis compatibility, because currently socket.io-redis is using `prefix + '#' + namespace + '#'` and `prefix + '#' + namespace + '#' + room + '#'` formats of keys.

uid option added to make compatible with socket.io-emitter (1.0.0)
@flyon
Copy link

flyon commented Nov 1, 2016

hey @antongorodezkiy this is exactly what I was looking for. Im trying to get this library to work with socket.io-redis. What would you fill in for uid in the options? Im just using the default namespace. So I used socket.io#/# as key

@antongorodezkiy
Copy link
Author

@flyon I'm also using socket.io#/# as the key and default uid.

static::$emitter = new BaseSocketIO\Emitter(Yii::$app->redis, [
    'key' => 'socket.io#/#'
]);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants