-
-
Notifications
You must be signed in to change notification settings - Fork 325
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 AppEUI #28
Comments
It's seems too overhead to call the API for obtain the AppEUI on every RX packet. |
Hi @ymkins I think there has been some misunderstanding what the Latest specs (1.0.2):
So it shouldn't be used to distinguish the payload format that you're using. E.g. two different Therefore the ApplicationID was introduced, so that you can define which nodes are serving which purpose. |
Well, I agree with that two equal AppEUI values could do something completely different, as we can't control vendors). I saw in the "Update MQTT handler so that it uses name of app and node" 249fe86 the "application/[applicationName]/node/[nodeName]/rx" schema. But in the "Update docs and changelog. Add app description in migration" 2b05632 there is the "application/[applicationID]/node/[devEUI]/rx" schema. The ApplicationID depends on the registration process. It can be troubled to have the same ApplicationIDs on different environments. |
Yes, the initial approach was to use names for both the application and node. However, the issue is that these names are mutable (thus can break any integration you made), where using the |
So, if we want provide integrations with the consistent ApplicationID on different environments, we can change the starting value of the id sequence in DB on deployment. |
Another option to have reserved ids after deployment, just create with API few application records for future using. |
So, no more sense to have ApplicationName and NodeName with restricted values because of ApplicationDescription and NodeDescription. Isn't it? |
Hi Brocaar
After the "Update MQTT handler so that it uses name of app and node" 249fe86 change it's harder to deploy configuration when a few existing applications are subscribed to own topics for data in particular formats. And there are no more AppEUI even in payload.
So, could you please add AppEUI info. For example, as field in payload.
The text was updated successfully, but these errors were encountered: