[dnssd] Tsvart early review of draft-ietf-dnssd-push-19

Brian Trammell via Datatracker <noreply@ietf.org> Tue, 04 June 2019 14:17 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: dnssd@ietf.org
Delivered-To: dnssd@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id D117912000E; Tue, 4 Jun 2019 07:17:08 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Brian Trammell via Datatracker <noreply@ietf.org>
To: <tsv-art@ietf.org>
Cc: draft-ietf-dnssd-push.all@ietf.org, dnssd@ietf.org, ietf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.97.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Brian Trammell <ietf@trammell.ch>
Message-ID: <155965782875.28092.15981061156595567480@ietfa.amsl.com>
Date: Tue, 04 Jun 2019 07:17:08 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/dnssd/ytdyHdu_QTdqRvKX-yY8FzPJoo4>
Subject: [dnssd] Tsvart early review of draft-ietf-dnssd-push-19
X-BeenThere: dnssd@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "Discussion of extensions to DNS-based service discovery for routed networks." <dnssd.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dnssd>, <mailto:dnssd-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dnssd/>
List-Post: <mailto:dnssd@ietf.org>
List-Help: <mailto:dnssd-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dnssd>, <mailto:dnssd-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 04 Jun 2019 14:17:09 -0000

Reviewer: Brian Trammell
Review result: Ready with Nits

This document appears to pose no transport concerns beyond those raised by the
protocols on which it is substantially based, DoT [RFC7858] and DSO [RFC8490].
There are of course connectivity risks associated with using client-established
server-push over TCP, but the design of DSO appears to adequately account for
these.

nit in 6.1: "default port for DNS-over-TLS DNS over TLS [RFC7858]"