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

Inconsistency between SenML RFC and LWM2M specification. #577

Open
sbernard31 opened this issue Jun 7, 2024 · 0 comments
Open

Inconsistency between SenML RFC and LWM2M specification. #577

sbernard31 opened this issue Jun 7, 2024 · 0 comments

Comments

@sbernard31
Copy link

sbernard31 commented Jun 7, 2024

Reading the Sensor Measurement Lists (SenML) -RFC8428§4.5.1. Names, I see that :

"The Name value is concatenated to the Base Name value to yield the
name of the sensor. The resulting concatenated name needs to
uniquely identify and differentiate the sensor from all others. The
concatenated name MUST consist only of characters out of the set "A"
to "Z", "a" to "z", and "0" to "9", as well as "-", ":", ".", "/",
and "_"; furthermore, it MUST start with a character out of the set
"A" to "Z", "a" to "z", or "0" to "9".
"

AFAIK, all SenML examples in LWM2M specification have names which start by / which doesn't follow that requirement ☝️

e.g :

[{"bn":"/4/0/", "n":"0"}, // name is /4/0/0 
 {"n":"1"},               // name is /4/0/1 
 {"n":"2"},               // name is /4/0/2 
 {"bn":"/6/", "n":"0"}]   // name is /6/0 
// all name are invalid as they start by '/' instead of [A-Z | a-z | 0-9]

And the specifications says :

The Name value is prepended by the Base Name value to form the name of the resource instance. The resulting name uniquely identifies the Resource Instance from all others. Example:
• if Base Name is "/" , the Array entry Name of the Resource is {Object}/{Object Instance}/{Resource}/{Resource Instance}
• When Base Name is not present, the Array entry Name is the full URI of the requested Resource Instance

See LWM2M-v1.2.1@core§7.5.6. SenML JSON for more details.

So unless I missed something there is a conflict between both specification ?

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

1 participant