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

[feat] Add support for '/' in MQTT prefix #213

Closed
kipe opened this issue Jan 12, 2021 · 0 comments · Fixed by #214
Closed

[feat] Add support for '/' in MQTT prefix #213

kipe opened this issue Jan 12, 2021 · 0 comments · Fixed by #214
Assignees
Labels
enhancement New feature or request

Comments

@kipe
Copy link

kipe commented Jan 12, 2021

Is your feature request related to a problem? Please describe.
Currently the prefix for MQTT topics only allows "a-z", "A-Z", "0-9", and "_". With the current prefix limitations, it's not possible to just subscribe to zwavejs2mqtt/+/# (for example, using zwavejs2mqtt/uuid as prefix). This would be useful when implementing a server with multiple zwavejs2mqtt instances connected, allowing the prefix to differentiate instances from each other.

Describe the solution you'd like
By allowing "/" as part of prefix, it would be possible use prefixes such as zwavejs2mqtt/55976116-d9f0-4c16-891c-e6bd771f550d. This would allow for more simple listener subscriptions.

Describe alternatives you've considered
Currently I'm subscribing to each "prefix-topic" separately, using prefixes like zwavejs2mqtt-55976116-d9f0-4c16-891c-e6bd771f550d. This requires subscriptions to each instance of zwavejs2mqtt separately.

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

Successfully merging a pull request may close this issue.

2 participants