[riot-notifications] [RIOT-OS/RIOT] nanocoap: make coap_get_block2() actually fill struct (#11802)

Ken Bannister notifications at github.com
Mon Jul 8 13:36:04 CEST 2019


kb2ma approved this pull request.

This PR is a straightforward extension of the existing block1 getter (server request handler) to also get block2 (client response handler). Tested block2 by running [block client](https://github.com/kb2ma/riot-apps/tree/kb2ma-master/nano-block-client) test app. Confirmed no regression with block1 by running nanocoap server for /sha256 resource.

In coap_get_block(), the blknum and szx variables are not necessary. However, this PR is intended to be a minimal impact stopgap until we merge #11057 in 2019.10, which addresses this issue.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11802#pullrequestreview-258845015
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190708/0b2d0b4b/attachment.html>


More information about the notifications mailing list