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

Quote presence/absence in object versioning #732

Closed
fdur opened this issue Aug 27, 2019 · 2 comments · Fixed by #774
Closed

Quote presence/absence in object versioning #732

fdur opened this issue Aug 27, 2019 · 2 comments · Fixed by #774
Labels
new feature New feature from LWM2M specification server Impact LWM2M server

Comments

@fdur
Copy link

fdur commented Aug 27, 2019

In the different LwM2M TS versions, the syntax for object versioning seems to be different (with or without quote for version string

  • OMA-TS-LightweightM2M-V1_0-20170208-A and OMA-TS-LightweightM2M-V1_0_2-20180209-A
    §5.2.7.3: lwm2m="2.0",</0/0>,</0/1>;ssid=101, </1/0/>;ssid=101, </3/0>,</5>,</4>,</55>;ver=1.9, </55/0>
    §6.2.3: ep=nodename,</1/0>,</1/1>,</3/0>,</44>;ver= “2.2”,</44/0>

  • OMA-TS-LightweightM2M_Core-V1_1-20180710-A
    §6.1.7.3: lwm2m="1.1",</0/0>,</0/1>;ssid=101;uri="coaps://server_1.example.com", </1/0/>;ssid=101, </3/0>,</5>,</4>,</55>;ver="1.9",</55/0>
    §7.2.3: ep=nodename,</1/0>,</1/1>,</3/0>,</44>;ver="2.2",</44/0>

  • OMA-TS-LightweightM2M_Core-V1_1_1-20190617-A
    §6.1.7.3: </>;lwm2m=1.1,</0/0>,</0/1>;ssid=101;uri="coaps://server_1.example.com", </1/0/>;ssid=101, </3/0>,</5>,</4>,</55>;ver=1.9,</55/0>
    §7.2.3: ep=nodename,&lt;/1/0&gt;,&lt;/1/1&gt;,&lt;/3/0&gt;,&lt;/44&gt;;ver=2.2,&lt;/44/0&gt;

Better for Leshan to support both versions ? (with and without quote)

@fdur fdur changed the title Quote presnece/absence in object versioning Quote presence/absence in object versioning Aug 28, 2019
@sbernard31 sbernard31 added new feature New feature from LWM2M specification server Impact LWM2M server labels Oct 17, 2019
@sbernard31
Copy link
Contributor

It should be fixed by #774

@sbernard31
Copy link
Contributor

#774 is now integrated in master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature from LWM2M specification server Impact LWM2M server
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants