Hi Achim, and welcome
I read your extensions. Nice work we need indeed. I have two main comments:
-
your branch cond_observe contains all the changes. It would be better to have one branch branched from the head of wakaama:master containing only the blockwise extension and another one containing only
the transaction extension. It would be much clearer for the integration.
-
I don’t get why the blockwise extension is not implemented the same way on the client and the server. Both the client and the server uses the transaction mechanism. Could you explain me ?
On a side note, I just realized there is a potential DOS attack vector with blockwise if a compromised peer keeps on sending packets with the More flag set. We need to limit the storage buffers per peer.
Regards,
David Navarro
From: wakaama-dev-bounces@xxxxxxxxxxx [mailto:wakaama-dev-bounces@xxxxxxxxxxx]
On Behalf Of Kraus Achim (INST/ESY4)
Sent: Friday, 20 February, 2015 13:06
To: wakaama-dev@xxxxxxxxxxx
Subject: [wakaama-dev] transaction & block wise transfer ready for first comments
Hello PL, Committers and Contributors,
I’m also new to this list. As my colleague Jörg already announced, I was working the last weeks on some extensions for wakaama.
The first one is an extension of the transaction mechanism. The original one takes the MID and matches an ACK with this. This is working fine for piggybacked responses, but it seems not to work for separate responses. Separate responses
may be useful, if it must be ensured, that the response is already received before sending any further messages. This may help to during the registration process.
The second extension is the block wise transfer. Though the spec is still a draft, it may be not the final one. The block wise transfer uses a buffer to accumulate or store larger payload. The COAP client uses transactions to maintain these
buffers, while the COAP server uses a list of buffers identified by client/method/uri triples.
For both extensions please refer to the comments in “transaction.c” and “blockwise.c”
The extensions are now pushed to the
https://github.com/bsinno/wakaama copy of wakaama for comments on it. The diff is unfortunately more the 300KB, so if wanted, I would prepare pull requests with selected smaller subsets (one by one,
J ). I would propose to start with some additional fixes. Is the preferred way for this project therefore to report a bug or to list the fixes her in the mailing list?
Mit freundlichen Grüßen / Best regards
Achim Kraus
Bosch Software Innovations GmbH
Communications (INST/ESY4)
Stuttgarter Straße 130
71332 Waiblingen
GERMANY
www.bosch-si.de
www.blog.bosch-si.com
Tel. +49 711 811-58139
achim.kraus@xxxxxxxxxxxx
Registered office: Berlin, Register court: Amtsgericht Charlottenburg, HRB 148411 B
Executives: Dr.-Ing. Rainer Kallenbach; Michael Hahn