search

SanKumar2015 / EST-coaps

EST over CoAPs IETF draft
1 stars 1 forks source link

Nits and minor issues (Klaus WGLC review 2/27/2019) #140

Closed csosto-pk closed 5 years ago

csosto-pk commented 5 years ago

(Sections 3.1 and 3.2.2 of [RFC7030]."

is missing ")"

Also

The EST-coaps server URIs, obtained through discovery of the EST-coaps root resource(s) as shown below, are of the form:

There is no longer a 'root resource' if a client discovers the full paths. Remove the "root".

It is up to the implementation to choose its root resource

Similarly, there is no root resource. Need to rephrase.

The Content-Formats in the response allow the client to request one that is supported by the server.

Better to state explicitly that these are the values accepted by the server in the Accept option

Option Length = 0x9 Option Value = est-coaps.example.org"

The value of 0x9 does not match the length of "est-coaps.example.org". Need to update the length.

Also in other examples and for other options.

csosto-pk commented 5 years ago

All rephrased, or text removed. Also fixed option lengths in the example.

Two commits address this issue. They are e243222 and b3f60d3.