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

Content format is not set for execute request with arguments #820

Closed
Swij opened this issue Mar 18, 2020 · 3 comments
Closed

Content format is not set for execute request with arguments #820

Swij opened this issue Mar 18, 2020 · 3 comments
Labels
bug Dysfunctionnal behavior server Impact LWM2M server

Comments

@Swij
Copy link

Swij commented Mar 18, 2020

When a execute request with arguments is sent the arguments (payload) should be encoded as plain text[1] and the CoAP content-format should be set to plain text[2].

The payload is encoded correctly but the content-format is not set at all.

missing-execute-content-format

I'm running the Leshan demo server (downloaded here 2020-03-11)

[1] - LwM2M specification 1.0.2 - 5.4.5 Execute parameters
[2] - LwM2M specification 1.0.2 - 8.1 Required Features

@sbernard31
Copy link
Contributor

I think your right.

The LWM2M specification v1.1 is even clearer :

Execute | POST Content Format: text/plain or none |

I will fix this, thx for reporting this ! 🙏

@sbernard31 sbernard31 added bug Dysfunctionnal behavior server Impact LWM2M server labels Mar 19, 2020
@sbernard31
Copy link
Contributor

commit d1cca91 integrated in master should fix this issue.
(Let me know if it works for you now)

@Swij
Copy link
Author

Swij commented Mar 19, 2020

Amazing, that was fast! Can confirm that everything works as expected now.
Thank you!

@Swij Swij closed this as completed Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Dysfunctionnal behavior server Impact LWM2M server
Projects
None yet
Development

No branches or pull requests

2 participants