Skip to content
This repository has been archived by the owner on Feb 11, 2020. It is now read-only.

Redis-based Persistence Doesn't Accept Database in Options #150

Closed
mocheng opened this issue Jun 17, 2014 · 2 comments
Closed

Redis-based Persistence Doesn't Accept Database in Options #150

mocheng opened this issue Jun 17, 2014 · 2 comments

Comments

@mocheng
Copy link
Contributor

mocheng commented Jun 17, 2014

* The current options include:
 *  - `port`, the Redis' port.
 *  - `host`, the Redis' host.
 *  - `password`, the Redis' password.
 *  - `redisOpts`, the options for the Redis client.
 *  - `channel`, the pub/sub channel that will be used to synchronize
 *    the various clients. Defaults to `'moscaSync'`.
 *  - `ttl`, an object containing three values:
 *    * `subscriptions`, the time (ms) after which subscriptions
 *      will expire. It defaults to 1 hour.
 *    * `packets`, the time (ms) after which packets will expire.

The Redis-based persistence has no option about "database". It would be better to have database selected by options. Then, one host & port instance can supports multiple mosca instances.

Thanks.

@mcollina
Copy link
Collaborator

Definitely an issue, thanks for reporting. Can you please submit a pull-request on this?

@mocheng
Copy link
Contributor Author

mocheng commented Jun 23, 2014

Pull request #152 submitted.

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

No branches or pull requests

2 participants