netconf-wg / restconf

9 stars 4 forks source link

TP review of restconf-09 #45

Closed abierman closed 8 years ago

abierman commented 8 years ago

From Tom Petch

https://www.ietf.org/mail-archive/web/netconf/current/msg10888.html

abierman commented 8 years ago

-OPEN ISSUE

s.2.2 "Consistent with the exclusive use of X.509v3 certificates for NETCONF over TLS [RFC7589], use of certificates in RESTCONF is also limited to X.509v3 certificates."

I think that this is open to misinterpretation. It is the use of TLS by NETCONF that is limited to X.509v3 certificates, not that when TLS uses certificates, as opposed to e.g. PSK, then they must be X.509v3. I seem to recall the IESG getting this wrong with call-home so best to be precise.

s.3.2 In a similar vein, a media type of 'application/yang.errors+xml ' corresponds to a resource of 'errors', just what I would expect, until a media type of 'application/yang.operation+xml ' corresponds to 'operations'. Another fruitful source of coding errors:-)

abierman commented 8 years ago

OPEN: term: consider operational state; no changes yet; reusing NETCONF term state data; check if used consistently

OPEN s2.2: we think the RFC reference is clear; no change

OPEN well-known. : not adding new example; refer to the .well-known RFC

OPEN: operations -- no change ; matches operation; not the container called operations

abierman commented 8 years ago

TLS issues left for Kent

kwatsen commented 8 years ago

I fixed s.2.2 - K.

abierman commented 8 years ago

OPEN well-known. : not adding new example; refer to the .well-known RFC

Added example for /top/restconf

abierman commented 8 years ago

closed in draft-10