Support the preferred content format for Bootstrap-Request #443
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Provides an improved range of conversions and handles scientific notation. Updates the tests. Ignores warning about comparing floating point numbers for equality. This is intentionally used to detect a NaN. Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
It is always enabled for LWM2M 1.0 clients and servers and LWM2M 1.1 servers, but disabled by default for LWM2M 1.1 clients. Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: David Navarro <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
size_t can be 64-bits while unsigned int is smaller. This causes issues when option_length is accessed through the pointer x in coap_parse_message(). The option length and delta are at most a 16-bit unsigned integer plus 269 according to RFC 7252. Changing option_number, option_delta, and option_length to uint32_t and x to uint32_t * fixes eclipse-wakaama#429. This also ensures sufficient size even if used on a platform with smaller integers. Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Observe cancel responses from the client and Create requests from the server were hard coded to use TLV. Observe cancel will now respect the format in the accept option or use the preferred format of the client if there is no accept option. Create requests will now use the preferred client format except when no object instance is specified. JSON and SenML JSON formats do not parse or serialize properly with a missing object instance, so TLV is forced in that case until it is fixed. Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Fix problem serializing resource instance reads. Fix handling of empty strings and opaque. Fix JSON serializing single instance resource as multiple. Fixes eclipse-wakaama#378. Fix JSON parsing of multiple instance resources. Fixes eclipse-wakaama#399. Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Fixes eclipse-wakaama#332 and prepares for writes of individual resource instances. Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
Signed-off-by: Scott Bertin <[email protected]>
|
This got ugly fast. I'll close this and wait until I can create a pull request that is cleaner. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Includes changes from #423, #430, #431, #432, and #435. Those should be merged before this. The changes intended for this are in 04e8219.