-
Notifications
You must be signed in to change notification settings - Fork 119
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
Error assembling blockwise response, passing on error NotImplemented() #55
Comments
thanks for reporting this. the easiest way to reproduce this would be
having such a contiki node mapped somewhere in the internet for me to
quickly test that, would that be possible? (you can send the address to
chrysn@fsfe.org if you don't want it to be public).
if that's not an option, could you send network dumps of of the
request/response messages?
|
Nodes are not in public network. Inside institute I don't have permissions to change that. Security is very tight here. I have sent you on an email tcpdumps of working comm (Firefox's Copper) and non working comm (aiocoap) right before it enters contiki sensor network. Hope it helps. |
The problem occurred because one of the early responses in the blockwise
transfer (that had Block2.M=1 set) but did had a payload size different
from the size advertised in Block2.
aiocoap should handle this earlier (when receiving the odd message, not
when receiving the next one), and give a clear error message.
|
After few exchanged emails with @chrysn, we figured out that issue is on server side. There is a bug at advertising Block2 as @chrysn mentioned. He even gratefully suggested workaround in aiocoap. Until this is fixed, workaround will be kept at my fork, "contiki27-workaround" branch. Issue can now be closed. |
Re-opening because aiocoap did not properly report the issue at the first corrupt message but gave a mislieading error description at a later message. |
Reading from
/.well-known/core
on contiki (v2.7) device raiseserror.NotImplemented()
. My guess is that something around block2 is not implemented.Code where this raises:
The text was updated successfully, but these errors were encountered: