Re: [core] WG Last Call on draft-ietf-core-conditional-attributes

Christian Amsüss <christian@amsuess.com> Tue, 28 March 2023 03:48 UTC

Return-Path: <christian@amsuess.com>
X-Original-To: core@ietfa.amsl.com
Delivered-To: core@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AF747C14CE5F for <core@ietfa.amsl.com>; Mon, 27 Mar 2023 20:48:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-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 IAqU3Gvc8uIh for <core@ietfa.amsl.com>; Mon, 27 Mar 2023 20:48:34 -0700 (PDT)
Received: from smtp.akis.at (smtp.akis.at [IPv6:2a02:b18:500:a515::f455]) (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 D5F9DC14CE46 for <core@ietf.org>; Mon, 27 Mar 2023 20:48:32 -0700 (PDT)
Received: from poseidon-mailhub.amsuess.com (095129206250.cust.akis.net [95.129.206.250]) by smtp.akis.at (8.17.1/8.17.1) with ESMTPS id 32S3mS6r077528 (version=TLSv1.2 cipher=ECDHE-ECDSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 28 Mar 2023 05:48:29 +0200 (CEST) (envelope-from christian@amsuess.com)
X-Authentication-Warning: smtp.akis.at: Host 095129206250.cust.akis.net [95.129.206.250] claimed to be poseidon-mailhub.amsuess.com
Received: from poseidon-mailbox.amsuess.com (hermes.amsuess.com [10.13.13.254]) by poseidon-mailhub.amsuess.com (Postfix) with ESMTP id 73A761DAD0; Tue, 28 Mar 2023 05:48:28 +0200 (CEST)
Received: from hephaistos.amsuess.com (hephaistos.amsuess.com [IPv6:2a02:b18:c13b:8010::907]) by poseidon-mailbox.amsuess.com (Postfix) with ESMTPSA id 360DB201E8; Tue, 28 Mar 2023 05:48:28 +0200 (CEST)
Received: (nullmailer pid 26581 invoked by uid 1000); Tue, 28 Mar 2023 03:48:27 -0000
Date: Tue, 28 Mar 2023 05:48:27 +0200
From: Christian Amsüss <christian@amsuess.com>
To: Klaus Hartke <hartke@projectcool.de>
Cc: "core@ietf.org WG (core@ietf.org)" <core@ietf.org>
Message-ID: <ZCJjiw7LVzyoOjzk@hephaistos.amsuess.com>
References: <5a8317ff-c7ff-ffda-da15-94bb04700451@ri.se> <CAAzbHvaqrM6v+Ls+MfVENAhxSWNdJbRFFbfeu+7-jT-uGFdwEg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha256"; protocol="application/pgp-signature"; boundary="DtWh6ITWw6nqDJkB"
Content-Disposition: inline
In-Reply-To: <CAAzbHvaqrM6v+Ls+MfVENAhxSWNdJbRFFbfeu+7-jT-uGFdwEg@mail.gmail.com>
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/e414-zsL0ZX6hyNAymmf9f38CBo>
Subject: Re: [core] WG Last Call on draft-ietf-core-conditional-attributes
X-BeenThere: core@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "Constrained RESTful Environments \(CoRE\) Working Group list" <core.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/core>, <mailto:core-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/core/>
List-Post: <mailto:core@ietf.org>
List-Help: <mailto:core-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/core>, <mailto:core-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 28 Mar 2023 03:48:35 -0000

Hello Klaus, hello conditional-attributes authors,

On Thu, Jan 12, 2023 at 05:48:23PM +0100, Klaus Hartke wrote:
> Rather than interfering with under what conditions and in what way the
> server generates notifications, the draft should define how the server
> updates resource state and how the client can influence this.

I agree with most of the points in this review -- and would have
expected that after the 2019 Montreal meeting, the "projection" model
was already agreed on.

> An exception is an attribute like pmax, which makes a server send
> notifications that it wouldn't have to send with RFC7641. Such a
> change in protocol cannot be achieved by changing the way resource
> states are updated. Instead, the draft should describe it for what it
> is: a protocol extension. And for protocol extensions, we use options
> in CoAP.

I disagree on this point: pmax *can* be modelled as a projection. The
projected resource just needs to have a shorter Max-Age, and a
well-behaved CoAP server that maintains an observation will make sure
that observing clients receive a fresh representation (even if it was
unmodified) before. While that behavior of the server is not mandated,
there is a "server MAY wish to prevent that by sending a new
notification with the unchanged representation" in 7641, which I'm
regarding as best practice, and conditional-attributes server just have
even more reason to adhere to.

Thanks for writing this review
Christian

-- 
There's always a bigger fish.
  -- Qui-Gon Jinn