Re: [Ntp] Antw: Re: Antw: [EXT] NTPv5 Loop Detection without Stratum

Harlan Stenn <stenn@nwtime.org> Tue, 06 September 2022 03:40 UTC

Return-Path: <stenn@nwtime.org>
X-Original-To: ntp@ietfa.amsl.com
Delivered-To: ntp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 05455C1524B2 for <ntp@ietfa.amsl.com>; Mon, 5 Sep 2022 20:40:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.897
X-Spam-Level:
X-Spam-Status: No, score=-1.897 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, NICE_REPLY_A=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, T_SCC_BODY_TEXT_LINE=-0.01, T_SPF_TEMPERROR=0.01, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id x4xPH49Cb3Bp for <ntp@ietfa.amsl.com>; Mon, 5 Sep 2022 20:40:22 -0700 (PDT)
Received: from chessie.everett.org (chessie.everett.org [66.220.13.234]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id CC5DEC1522DB for <ntp@ietf.org>; Mon, 5 Sep 2022 20:40:18 -0700 (PDT)
Received: from [10.208.75.149] (071-084-168-128.res.spectrum.com [71.84.168.128]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by chessie.everett.org (Postfix) with ESMTPSA id 4MMB3c10zYzMQ0n; Tue, 6 Sep 2022 03:40:12 +0000 (UTC)
Message-ID: <d3f0c3f4-df95-77ca-07ae-6d5e857a81d7@nwtime.org>
Date: Mon, 05 Sep 2022 20:40:10 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0
Content-Language: en-US
To: Danny Mayer <mayer@pdmconsulting.net>, Hal Murray <halmurray@sonic.net>
Cc: "ntp@ietf.org" <ntp@ietf.org>
References: <20220905215236.ED4D728C1D8@107-137-68-211.lightspeed.sntcca.sbcglobal.net> <0c5ac027-1be2-8f1f-ca52-badb7eb40eea@pdmconsulting.net>
From: Harlan Stenn <stenn@nwtime.org>
In-Reply-To: <0c5ac027-1be2-8f1f-ca52-badb7eb40eea@pdmconsulting.net>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/ntp/tqCIox9dy3d-HqzIe6rbCtmwCfs>
Subject: Re: [Ntp] Antw: Re: Antw: [EXT] NTPv5 Loop Detection without Stratum
X-BeenThere: ntp@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Network Time Protocol <ntp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ntp>, <mailto:ntp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ntp/>
List-Post: <mailto:ntp@ietf.org>
List-Help: <mailto:ntp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ntp>, <mailto:ntp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 06 Sep 2022 03:40:27 -0000

On 9/5/2022 3:24 PM, Danny Mayer wrote:
> 
> On 9/5/22 5:52 PM, Hal Murray wrote:
>> mayer@pdmconsulting.net said:
>>> SNTP is defined in RFC5905 Section 14. You don't need anything more than
>>> that.
>> There is also RFC 4330 and a lot of crappy code out there in firmware 
>> that
>> will never get updated.
> RFC5905 obsoleted RFC4330.
>> RFC 5905 is over 100 pages.  Section 14 is less than a page.  How many of
>> those other pages would somebody have to read and understand in order to
>> implement a SNTP client?
> Read Section 14 to find out.
>> If you were a junior programmer assigned to implement a SNTP client, 
>> would you
>> go throgh RFC 5905 or google around and find some code to copy and 
>> ship it
>> when it worked?
> You don't give this to a junior programmer. RFC's are difficult enough 
> to understand and if you were to do so, you would expect a poor 
> implementation and incorrect code. Don't blame the programmer, blame the 
> person who thought that was a good, or maybe a cheap, idea.

Who cares what is assigned to a junior programmer except the programmer 
and the programmer's management?

The more interesting point could be that the Reference Implementation 
has come with am open-source fully-featured SNTP client for many years' 
time.

There's nothing stopping anybody from cribbing from that code and 
throwing out the bits they don't want.  There are also several other 
(likely much less featured) published implementations out there.

>> I think we should have a separate document for SNTP clients.  So far, I
>> haven't convinced (m)any other people.  There are two goals:
>>
>> One is to have a clear and simple document so we can point people at 
>> it when
>> their current code is sending ancient format packets.  Some of those 
>> ancient
>> packets are not using a format described in any RFC and they worked just
>> because of the undocumented details in the ntpd implementation.
>>
>> Another is to make us think and document a packet format that we will 
>> support for a long time.
>>
>> We should setup servers that don't support old format packets so SNTP 
>> implementors can test their code.
>
> Why bother? Developers of such clients will just use existing servers. 
> You can have newer servers not support such packets.

I believe that creating a separate SNTP spec will just be another source 
of bitrot and confusion.

> However you don't 
> say what those undocumented details are.

Yes,  please support your assertion.

If folks implement their own SNTP client and it doesn't work with some 
class of servers, they can fix their code or not.

There is a well-defined SNTP spec already out there.

If somebody chooses to propagate a poor implementation, that's their 
choice.  If others choose to use that implementation, that's their choice.

> Danny
> 
> _______________________________________________
> ntp mailing list
> ntp@ietf.org
> https://www.ietf.org/mailman/listinfo/ntp
> 

-- 
Harlan Stenn <stenn@nwtime.org>
http://networktimefoundation.org - be a member!