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

publish subscribes and new client ids #92

Closed
3 tasks
mcollina opened this issue Feb 15, 2014 · 5 comments · Fixed by #182
Closed
3 tasks

publish subscribes and new client ids #92

mcollina opened this issue Feb 15, 2014 · 5 comments · Fixed by #182

Comments

@mcollina
Copy link
Collaborator

  • Publish each subscribe to "$SYS/broker-id/new/subscribes"
  • Publish each clientId to "$SYS/broker-id/new/clients"
  • Automatic disconnect clients connected to another broker
@mocheng
Copy link
Contributor

mocheng commented Jul 18, 2014

Automatic disconnect clients connected to another broker

Should this logic be implemented in ascoltatori?
I suppose this feature can only be done with help of external storage. Mosca, besides persistence layer, doesn't touch external storage.

@mcollina
Copy link
Collaborator Author

First, that feature is defined in the MQTT spec, so it belongs here. As I propose to implement it, it needs no persistence layer, just an ascoltatori subscription to '$SYS/+/new/clients'. Does it sound feasible?

@mocheng
Copy link
Contributor

mocheng commented Jul 18, 2014

Sounds good. I'll try on this.

@mocheng
Copy link
Contributor

mocheng commented Jul 18, 2014

Update RedisPersistance to subscribe to 'new/clients' for some of its internal behavior

What kind of "internal behavior"?

@mcollina
Copy link
Collaborator Author

oh forget about that, I made a mistake. What I wanted to replace with something on Ascoltatori was this: https://github.com/mcollina/mosca/blob/master/lib/persistence/redis.js#L126-L132.
However I don't know if it's a good idea. Anyway, it does not belong here (I'm updating this).

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

Successfully merging a pull request may close this issue.

2 participants