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

include agent protocol in topic (MQTT) #374

Closed
AlvaroVega opened this issue Jan 23, 2019 · 1 comment
Closed

include agent protocol in topic (MQTT) #374

AlvaroVega opened this issue Jan 23, 2019 · 1 comment

Comments

@AlvaroVega
Copy link
Member

AlvaroVega commented Jan 23, 2019

twin issue telefonicaid/iotagent-ul#287

Currently topics used by iotagent-json and iotagent-ul are the same.
//<device_id>/attrs
There is no way to know if a topic (MQTT) is for UL or JSON agent.

Topic is fixed in agent source code:

https://github.com/telefonicaid/iotagent-json/blob/master/lib/bindings/MQTTBinding.js#L45
https://github.com/telefonicaid/iotagent-ul/blob/master/lib/bindings/MQTTBinding.js#L46

Topic is knowed by ACL files:

https://github.com/telefonicaid/iotagent-node-lib/blob/master/docker/Mosquitto/aclfile#L3
https://github.com/telefonicaid/rabbit-topic-acl/blob/master/docker/aclfile#L2

Possible solution: extending topics

/<protocol>/<apikey>/<device_id>/attrs

but is not backward compatible

@fgalan
Copy link
Member

fgalan commented Nov 26, 2019

Implemented in PR #450

Can this issue be closed? Or is there anything pending?

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

No branches or pull requests

2 participants