SanKumar2015 / EST-coaps

EST over CoAPs IETF draft
1 stars 1 forks source link

version -13 for AD comments #151

Closed petervanderstok closed 5 years ago

petervanderstok commented 5 years ago

Hi Panos,

thanks for the reminder. Why don't you send these comments to the list with a suggestion for improvement. Then Benjamin can decide what he prefers.

cheerio, and all the best,

peter Panos K. schreef op 2019-09-10 16:48:

@CSOSTO-PK commented on this pull request.

About "The corresponding longer URIs from MAY be supported." In a review someone (I think Carsten) told us that the way it reads is like we cannot make up our minds and we support both URI (long and short) which makes it harder for implementers. That made sense so we removed the support for longer URIs.

About "This approach allows future servers to incorporate currently not specified content-formats and resources." It does not add much to the content. The Discovery and the Accept option was added in COAP for a reason and that is why we are using it. I don't think using the mechanism has to do with future content-formats.

-- You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub [1], or mute the thread [2].

Links:

[1] https://github.com/SanKumar2015/EST-coaps/pull/151?email_source=notifications&email_token=ADCZGQO2RXQGYZOJ6KZGTSDQI6XTVA5CNFSM4IVFKLH2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCEHT4SA#pullrequestreview-286211656 [2] https://github.com/notifications/unsubscribe-auth/ADCZGQMT3QCAPWMACY5Z4QLQI6XTVANCNFSM4IVFKLHQ