From: Kent Watsen <kent@watsen.net>
Message-ID: <0100016b48687180-05d46ca8-d13d-46ba-93e7-cac92b63e392-000000@email.amazonses.com>
Content-Type: multipart/alternative;
 boundary="Apple-Mail=_6BE0E4A0-0156-41DA-9B5D-3B107C46EBAA"
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.11\))
Date: Tue, 11 Jun 2019 21:18:52 +0000
In-Reply-To: <0537B9CE-83CF-4E5C-95AC-ABEB46C92804@cisco.com>
Cc: "netconf@ietf.org" <netconf@ietf.org>
To: "Reshad Rahman (rrahman)" <rrahman@cisco.com>
References: <156020896295.32199.7617469401232263908@ietfa.amsl.com>
 <F45C8148-7D85-4F9E-A2EA-CD58DBCA86E6@cisco.com>
 <0100016b477bd2b9-b33bafa5-224c-43d4-9467-0ffbdae769e6-000000@email.amazonses.com>
 <0537B9CE-83CF-4E5C-95AC-ABEB46C92804@cisco.com>
Feedback-ID: 1.us-east-1.DKmIRZFhhsBhtmFMNikgwZUWVrODEw9qVcPhqJEI2DA=:AmazonSES
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/nwA0VljCuAxFo1TcM9Yjs1Qlu1o>
Subject: Re: [netconf] I-D Action: draft-ietf-netconf-restconf-notif-14.txt
Precedence: list


--Apple-Mail=_6BE0E4A0-0156-41DA-9B5D-3B107C46EBAA
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=utf-8


Hi Reshad,

> Yes, this is over RESTCONF so http1.1/http2 are both ok. One change =
<https://www.ietf.org/rfcdiff?url2=3Ddraft-ietf-netconf-restconf-notif-08>=
 which was done ~8 months ago is to have 1 callflow =E2=80=9Cover =
RESTCONF/SSE=E2=80=9D as opposed to the previous separate callflows for =
http1.1 and http2. The reason http2 is mentioned in a few places is =
because of http2 streams. Section 4 already mentions =E2=80=9Cwhere =
HTTP2 is available to=E2=80=A6=E2=80=9D, so it looks like section 3.4 is =
the only place missing =E2=80=9Cwhen HTTP2 is used=E2=80=9D?

I recall that conversion and hence my alarm seeing all the HTTP2 text.  =
Yes, adding "when/if HTTP2 is used" everywhere, to constantly remind the =
reader that it may not always be the case would be good.   =20

For Section 4, would replacing "where HTTP2 transport is available" with =
"when the HTTP2 transport is used" be better?


Kent // contributor


--Apple-Mail=_6BE0E4A0-0156-41DA-9B5D-3B107C46EBAA
Content-Transfer-Encoding: quoted-printable
Content-Type: text/html;
	charset=utf-8

<html><head><meta http-equiv=3D"Content-Type" content=3D"text/html; =
charset=3Dutf-8"></head><body style=3D"word-wrap: break-word; =
-webkit-nbsp-mode: space; line-break: after-white-space;" class=3D""><br =
class=3D""><div>Hi Reshad,</div><div><br class=3D""><blockquote =
type=3D"cite" class=3D""><div class=3D""><span style=3D"font-family: =
Calibri, sans-serif; font-size: 11pt; caret-color: rgb(0, 0, 0);" =
class=3D"">Yes, this is over RESTCONF so http1.1/http2 are both ok. =
One</span>&nbsp;<a =
href=3D"https://www.ietf.org/rfcdiff?url2=3Ddraft-ietf-netconf-restconf-no=
tif-08" class=3D"" style=3D"font-family: Calibri, sans-serif; font-size: =
11pt; caret-color: rgb(0, 0, 0); color: purple;">change</a>&nbsp;<span =
style=3D"font-family: Calibri, sans-serif; font-size: 11pt; caret-color: =
rgb(0, 0, 0);" class=3D"">which was done ~8 months ago is to have 1 =
callflow =E2=80=9Cover RESTCONF/SSE=E2=80=9D as opposed to the previous =
separate callflows for http1.1 and http2. The reason http2 is mentioned =
in a few places is because of http2 streams. Section 4 already mentions =
=E2=80=9Cwhere HTTP2 is available to=E2=80=A6=E2=80=9D, so it looks like =
section 3.4 is the only place missing =E2=80=9Cwhen HTTP2 is =
used=E2=80=9D?</span></div></blockquote><div><br class=3D""></div><div>I =
recall that conversion and hence my alarm seeing all the HTTP2 text. =
&nbsp;Yes, adding "when/if HTTP2 is used" everywhere, to constantly =
remind the reader that it may not always be the case would be good. =
&nbsp; &nbsp;</div><div><br class=3D""></div><div>For Section 4, would =
replacing "where HTTP2 transport is available" with "when the HTTP2 =
transport is used" be better?</div><div><br class=3D""></div></div><div =
class=3D""><br class=3D""></div>Kent // contributor<div class=3D""><br =
class=3D""></div></body></html>=

--Apple-Mail=_6BE0E4A0-0156-41DA-9B5D-3B107C46EBAA--