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

MQTT: Use / instead of . as topic separator #35

Open
1 of 4 tasks
bergie opened this issue May 17, 2017 · 3 comments
Open
1 of 4 tasks

MQTT: Use / instead of . as topic separator #35

bergie opened this issue May 17, 2017 · 3 comments

Comments

@bergie
Copy link
Member

bergie commented May 17, 2017

Currently MsgFlo always creates default queue names like role.PORT. However, with MQTT it would be good to use / as the separator, as then wildcards work correctly.

http://www.hivemq.com/blog/mqtt-essentials-part-5-mqtt-topics-best-practices

  • msgflo-arduino
  • msgflo-nodejs
  • msgflo-python
  • msgflo-rust
@jonnor
Copy link
Member

jonnor commented May 18, 2017

Should fix all the libraries in one go

@jonnor
Copy link
Member

jonnor commented May 18, 2017

Must ensure that we don't have code relying on the naming convention in Msgflo. Like setup without - -discover

@jonnor
Copy link
Member

jonnor commented May 18, 2017

Since other systems may communicate using the topic name directly, this is a breaking change and must bump minor

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

No branches or pull requests

2 participants