-
Notifications
You must be signed in to change notification settings - Fork 424
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 Extended Tokens Feature(RFC 8974) in libcoap. #937
Comments
I actually have code in place to do this, but was waiting for some the other PRs to be merged that add in extra token usage. |
Thank you for replying. @mrdeep1 |
Ok,i will try it #938。 |
Thanks for doing this review,
RFC8973 Section 2.2. Discovering Support and in particular
This is the test packet for determining whether the server has Extended Token support or not.
It appears that wireshark does not know how to decode Extended Tokens and hence the reporting confusion. The libcoap logs show the correctly decoded packets. For example, wireshark reports the token length as 13, whereas the 13 indicates that you need the first byte of the token (in this case 01) added to 13 which gives 14 as per RFC8974 Section 2.1. Extended Token Length (TKL) Field
The example is correct and working as expected. |
I have updated the documentation for #938 to reflect the initial Extended Token test PDU that is sent. |
Thank you for your reply,i will check RFC8974 Section 2.2.2. Trial and Error to figure out the meaning of the first packet. |
@529433 How is it all going? |
Is your feature request related to a problem? Please describe.
I noticed that the serialization of state into tokens is limited by the fact that both Coap over UDP[RFC7252] and Coap over reliable transports[RFC8323] restrict the maximum token length to 8 bytes.But in RFC8974, to overcome this limitation, the token length can be expanded.When is this feature planned to be added to libcoap?
Describe the solution you'd like
Add in support for extended tokens.
Thanks in advance
The text was updated successfully, but these errors were encountered: