MIME-Version: 1.0
References: <171729930120.24161.6084740749493881206@ietfa.amsl.com>
In-Reply-To: <171729930120.24161.6084740749493881206@ietfa.amsl.com>
From: tirumal reddy <kondtir@gmail.com>
Date: Sun, 2 Jun 2024 19:01:34 +0530
Message-ID: 
 <CAFpG3gd-M3wD=yJ=W-XEyDktt_JMXcwJEhbG7SHzi9pKLyOicA@mail.gmail.com>
To: Jiankang Yao <yaojk@cnnic.cn>
Content-Type: multipart/alternative; boundary="000000000000782fc90619e83e25"
Message-ID-Hash: A2TIQH7L3ZPILCV4ZS34353ZDERNOCGQ
CC: art@ietf.org, add@ietf.org,
 draft-ietf-add-split-horizon-authority.all@ietf.org, last-call@ietf.org
Precedence: list
Subject: =?utf-8?q?=5BAdd=5D_Re=3A_Artart_last_call_review_of_draft-ietf-add-split-ho?=
	=?utf-8?q?rizon-authority-11?=
Archived-At: 
 <https://mailarchive.ietf.org/arch/msg/add/fIngo3WiwM7A-lfFbnL2OmlX06A>

--000000000000782fc90619e83e25
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Thanks for the review, please see inline

On Sun, 2 Jun 2024 at 09:05, Jiankang Yao via Datatracker <noreply@ietf.org=
>
wrote:

> Reviewer: Jiankang Yao
> Review result: Ready
>
> I am the assigned ART-ART reviewer for this draft. The Art Area
> Review Team (ART-ART) reviews all IETF documents being processed
> by the IESG.  Please treat these comments just
> like any other last call comments.
>
> Document: draft-ietf-add-split-horizon-authority-11
> Reviewer: Jiankang Yao
> Review Date: 2024-06-01
>
> Summary: Ready for publication.
>
> This document is clear and in good shape.
>
> One suggestion:
>
> In section 2 Terminology,
>
> "
>   Encrypted DNS
> A DNS protocol that provides an encrypted channel between a DNS client an=
d
> server (e.g., DNS over TLS (DoT) [RFC7858], HTTPS (DoH) [RFC8484], QUIC
> (DoQ)
> [RFC9250]).
>
> Split-Horizon DNS
> The DNS service provided by a resolver that also acts as an authoritative
> server for some names, providing resolution results that are meaningfully
> different from those in the Global DNS. (See "Split DNS" in Section 6 of
> [RFC9499].)
>
> "
> Encrypted =E2=80=98DNS=E2=80=99 is defined as "Protocol" while Split-Hori=
zon =E2=80=98DNS=E2=80=99 is
> defined
> as "service". Same "DNS" points to different things.
>
> How about changing "Encrypted DNS"  to "Encrypted DNS Protocols"?
>

"Encrypted DNS" is already used in the terminology of other RFCs, like
https://datatracker.ietf.org/doc/rfc9463/

-Tiru

>
> Best Regards
> Jiankang Yao
>
>
>
>

--000000000000782fc90619e83e25
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div>Thanks for the review, please see inline=C2=A0</div><=
br><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Sun,=
 2 Jun 2024 at 09:05, Jiankang Yao via Datatracker &lt;<a href=3D"mailto:no=
reply@ietf.org">noreply@ietf.org</a>&gt; wrote:<br></div><blockquote class=
=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rg=
b(204,204,204);padding-left:1ex">Reviewer: Jiankang Yao<br>
Review result: Ready<br>
<br>
I am the assigned ART-ART reviewer for this draft. The Art Area<br>
Review Team (ART-ART) reviews all IETF documents being processed<br>
by the IESG.=C2=A0 Please treat these comments just<br>
like any other last call comments.<br>
<br>
Document: draft-ietf-add-split-horizon-authority-11<br>
Reviewer: Jiankang Yao<br>
Review Date: 2024-06-01<br>
<br>
Summary: Ready for publication.<br>
<br>
This document is clear and in good shape.<br>
<br>
One suggestion:<br>
<br>
In section 2 Terminology,<br>
<br>
&quot;<br>
=C2=A0 Encrypted DNS<br>
A DNS protocol that provides an encrypted channel between a DNS client and<=
br>
server (e.g., DNS over TLS (DoT) [RFC7858], HTTPS (DoH) [RFC8484], QUIC (Do=
Q)<br>
[RFC9250]).<br>
<br>
Split-Horizon DNS<br>
The DNS service provided by a resolver that also acts as an authoritative<b=
r>
server for some names, providing resolution results that are meaningfully<b=
r>
different from those in the Global DNS. (See &quot;Split DNS&quot; in Secti=
on 6 of<br>
[RFC9499].)<br>
<br>
&quot;<br>
Encrypted =E2=80=98DNS=E2=80=99 is defined as &quot;Protocol&quot; while Sp=
lit-Horizon =E2=80=98DNS=E2=80=99 is defined<br>
as &quot;service&quot;. Same &quot;DNS&quot; points to different things.<br=
>
<br>
How about changing &quot;Encrypted DNS&quot;=C2=A0 to &quot;Encrypted DNS P=
rotocols&quot;?<br></blockquote><div><br></div><div>&quot;Encrypted DNS&quo=
t; is already used in the terminology of other RFCs, like=C2=A0<a href=3D"h=
ttps://datatracker.ietf.org/doc/rfc9463/">https://datatracker.ietf.org/doc/=
rfc9463/</a></div><div><br></div><div>-Tiru=C2=A0</div><blockquote class=3D=
"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(2=
04,204,204);padding-left:1ex">
<br>
Best Regards<br>
Jiankang Yao<br>
<br>
<br>
<br>
</blockquote></div></div>

--000000000000782fc90619e83e25--