-
Notifications
You must be signed in to change notification settings - Fork 407
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
Add updated IPSO objects specifications (version 1.1) to the Leshan Demo Server #972
Comments
Thx for reporting this. I did an update some time ago but maybe a new one is needed again.
Generally contributions are welcome but I have some code to update this more or less automatically, so I think this will be simple if I do it myself. I will look at this and let you know as soon as I have a PR ready for this.
Good question, I guess as |
Ok, understood, thank you. Just FYI, the only problem I faced was the Location object specification in version 1.1, it has a duplicated Timestamp resource which caused Leshan build error. I've reported the issue in the |
I just faced it too :) Thx for reporting this to OMA 🙏 |
I started to work on this but this raises some questions : see OpenMobileAlliance/lwm2m-registry#585 Do you have any opinion about that ? |
Hard to tell , perhaps 1) is indeed more intuitive as you suggest. On the other hand 2) gives a bit more flexibility, I don't have strong opinion here, let's see what spec gurus say. As for the bonus question, it seems to be covered pretty well in section 7.2.2 of the LwM2M specification (1.1.1):
Section 7.2.1 explains what is considered a backward and non-backward compatible change. |
See #973. Do not hesitate to tell me if it works for you and/or if it sounds good for you :) |
About model version, another idea would be : I can not see any issue for server-demo as the most it supports the better it is. I guess for now I will stay on 1) and we will rethink about this if more recent version is asked by users. |
Thank you @sbernard31 for such a prompt PR! I've tested it with a Zephyr client with updated IPSO object models and it worked fine. Thanks again! |
I personally work on improvements for the Zephyr LwM2M client implementation, where I typically use Leshan for testing (because it's a great project :)). But I'm not involved in any end-product utilizing Leshan. |
Great to know that Leshan is useful :-) |
Multiple IPSO objects (in 3xxx range) had their specifications updated to version 1.1. The update added Timestamp/Measurement Quality resources to the objects in most cases. The updated specifications can be found in the lwm2m-registry GH repository.
The new object specifications are not supported in the Leshan Demo Server yet. It would be good to add the 1.1 specifications to the Demo Server so that clients using the new object versions can view their objects properly (currently the server will show
Object 3xxx (model unknown)
).I've added the new IPSO objects specifications to the Leshan Demo Server locally, so I can create a PR if we agree it's a desired change. I also wonder whether Leshan Demo Client should be updated, i. e. switch to IPSO objects in version 1.1 or rather stick to version 1.0?
The text was updated successfully, but these errors were encountered: