-
Notifications
You must be signed in to change notification settings - Fork 368
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
Californium does not support standard coap codes for ping 7.02 and pong 7.03 #1564
Comments
Yes, the TCP implementation RFC 8323 is still very experimental and more or less based on a early version of that RFC. Do you have interest in evolving it? My interest in CoAP over TCP is limited. See Next major version 3.0 - About CoAP over TCP / RFC 8323 and issue #1488. |
Thanks for the reply. Sure, we have a fairly conservative fix that simply forwards the 7.02 pings to the application layer as Requests, and allows the application to handle them. Will submit a PR! |
Please obey CONTRIBUTING. You will mainly need a ECA, using the same e-mail address as used for "signing" the commit. |
Just to ask: Do you want to apply this for 2.6.x? If so, is it possible that you give us/me your planed time-schedule? In order to sync with an upcoming leshan milestone release, I scheduled a 2.6.2 bugfix release for next Tuesday. If required, that could also be postponed for 1-2 days. |
Uups, sorry, I was mislead. |
Any news on this? |
Don't hesitate to create a PR, if you're still on this issue. |
Problem description
There seems no support in Californium for the ping/pong messages defined by CoAP RFC 8323:
See https://tools.ietf.org/html/rfc8323#section-5.4
Currently, if Calfornium receives a PING sent by another CoAP stack,
DataParser.parseMessage
throws an "illegal message code" exception.Because code 7.02 lies outside
REQUEST_CODE_LOWER_BOUND, REQUEST_CODE_UPPER_BOUND
andRESPONSE_CODE_LOWER_BOUND, RESPONSE_CODE_UPPER_BOUND
defined incalifornium.core.coap.CoAP
.This causes significant problems communicating with other CoAP stacks.
The text was updated successfully, but these errors were encountered: