Re: [DNSOP] Brian Haberman's Discuss on draft-ietf-dnsop-edns-tcp-keepalive-04: (with DISCUSS)

Sara Dickinson <sara@sinodun.com> Tue, 05 January 2016 17:08 UTC

Return-Path: <sara@sinodun.com>
X-Original-To: dnsop@ietfa.amsl.com
Delivered-To: dnsop@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0427B1ACE3E; Tue, 5 Jan 2016 09:08:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.6
X-Spam-Level:
X-Spam-Status: No, score=-2.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id RkPA7j2wRJ5P; Tue, 5 Jan 2016 09:08:48 -0800 (PST)
Received: from shcp01.hosting.zen.net.uk (shcp01.hosting.zen.net.uk [88.98.24.67]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D89251ACE31; Tue, 5 Jan 2016 09:08:47 -0800 (PST)
Received: from [62.232.251.194] (port=16587 helo=virgo.sinodun.com) by shcp01.hosting.zen.net.uk with esmtpsa (TLSv1:DHE-RSA-AES256-SHA:256) (Exim 4.85) (envelope-from <sara@sinodun.com>) id 1aGV67-0007aA-7R; Tue, 05 Jan 2016 17:08:40 +0000
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 9.2 \(3112\))
From: Sara Dickinson <sara@sinodun.com>
In-Reply-To: <20160104181816.30839.63396.idtracker@ietfa.amsl.com>
Date: Tue, 05 Jan 2016 17:08:42 +0000
Content-Transfer-Encoding: quoted-printable
Message-Id: <1AF2CA67-B9AF-4DC3-A3EC-85D3AB958DC8@sinodun.com>
References: <20160104181816.30839.63396.idtracker@ietfa.amsl.com>
To: Brian Haberman <brian@innovationslab.net>
X-Mailer: Apple Mail (2.3112)
X-OutGoing-Spam-Status: No, score=-1.0
X-AntiAbuse: This header was added to track abuse, please include it with any abuse report
X-AntiAbuse: Primary Hostname - shcp01.hosting.zen.net.uk
X-AntiAbuse: Original Domain - ietf.org
X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse: Sender Address Domain - sinodun.com
X-Get-Message-Sender-Via: shcp01.hosting.zen.net.uk: authenticated_id: sara+sinodun.com/only user confirmed/virtual account not confirmed
Archived-At: <http://mailarchive.ietf.org/arch/msg/dnsop/1Wv6WIrTo-_92B14RNoqsWTrL4E>
Cc: tjw.ietf@gmail.com, draft-ietf-dnsop-edns-tcp-keepalive@ietf.org, dnsop@ietf.org, dnsop-chairs@ietf.org, The IESG <iesg@ietf.org>
Subject: Re: [DNSOP] Brian Haberman's Discuss on draft-ietf-dnsop-edns-tcp-keepalive-04: (with DISCUSS)
X-BeenThere: dnsop@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: IETF DNSOP WG mailing list <dnsop.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dnsop>, <mailto:dnsop-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dnsop/>
List-Post: <mailto:dnsop@ietf.org>
List-Help: <mailto:dnsop-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dnsop>, <mailto:dnsop-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 05 Jan 2016 17:08:49 -0000

> On 4 Jan 2016, at 18:18, Brian Haberman <brian@innovationslab.net> wrote:
> 
> ----------------------------------------------------------------------
> DISCUSS:
> ----------------------------------------------------------------------
> 
> I support the publication of this document, but I have a point I want to
> discuss to help with the clarity of the spec.
> 
> Section 3.2.1 says that clients send this option with the first query
> sent on a TCP connection and Section 3.2.2 says it should honor the
> timeout provided by the server and close the socket when appropriate.
> What is not discussed is how the client should manage the timer with
> respect to the reception of multiple query responses that may, or may
> not, include edns-tcp-keepalive option. Section 3.3.2 says the server MAY
> send the option, so it is up to the server to decide when to include the
> option and the corresponding timeout value. Should the client's timer
> simply reflect the value sent in the latest response? The smallest
> remaining time?
> 
> I think a few sentences on client timer management would be beneficial.

Hi, 

Thanks for the feedback. The intention in section 3.2.2 was that the client should update the timeout whenever receiving a new value in any response. Would it be enough to clarify by changing the last sentence in the second paragraph to:

  “It SHOULD honour the timeout received in that response (overriding any previous timeout) and
   initiate close of the connection before the timeout expires.”

Regards

Sara.