Re: [netconf] [Technical Errata Reported] RFC8040 (6342)

Martin Björklund <mbj+ietf@4668.se> Mon, 23 November 2020 08:37 UTC

Return-Path: <mbj+ietf@4668.se>
X-Original-To: netconf@ietfa.amsl.com
Delivered-To: netconf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7BBE53A1737 for <netconf@ietfa.amsl.com>; Mon, 23 Nov 2020 00:37:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 1.779
X-Spam-Level: *
X-Spam-Status: No, score=1.779 tagged_above=-999 required=5 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, PDS_NAKED_TO_NUMERO=1.999, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=4668.se header.b=dBieKQaE; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=hgwQt9xQ
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 cmsY__-IOmq3 for <netconf@ietfa.amsl.com>; Mon, 23 Nov 2020 00:37:34 -0800 (PST)
Received: from wout3-smtp.messagingengine.com (wout3-smtp.messagingengine.com [64.147.123.19]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DC1B83A1736 for <netconf@ietf.org>; Mon, 23 Nov 2020 00:37:34 -0800 (PST)
Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailout.west.internal (Postfix) with ESMTP id 8BF55E22; Mon, 23 Nov 2020 03:37:33 -0500 (EST)
Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Mon, 23 Nov 2020 03:37:34 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=4668.se; h=date :message-id:to:cc:subject:from:in-reply-to:references :mime-version:content-type:content-transfer-encoding; s=fm3; bh= 74hN7S7Fwjl5+goVYp8iagHbKVKJ6e+Rr0IG1jSVXX4=; b=dBieKQaEJ7HdB5mP CDl4Tqs/QMc9BY8weyD5QzzgIyceuXk5fsBnykBhVGLi6OoAlo3LNg6qB/LFWEnA 6Ls4ftQYPd9I7fOJ17yV9vqz5aEqKu7WNDhapVScttVZbPJK2TMZOxgFMGgerhF7 C/BmtEzc4mVix6uUW7Q1QzbMCzpV8/gHWUom8HGHWBR2fpF1xs6nEjLjL/1afsRO tgmzaDBCweM/Eb3YarJalvokn0VRhpxtHz8CYF6OckwVsC/g3n8c9k6Pi2+C0bim 8GXlS/ZLkFn6p+Uu6fxqd9L4/kn5pIUQ+k/a88ZKFs1LRokIw20KqJn8oXEf5O71 ZBvDTg==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=74hN7S7Fwjl5+goVYp8iagHbKVKJ6e+Rr0IG1jSVX X4=; b=hgwQt9xQjOKIzvBIsY1OcBPAMmfAmBnRUw8MiLqz16PD7O12fG+BjNCrl 6eF4w8J61KpdSc3mEhlYHHIHi0xRypwe72gPasIR2tRf9eDw6gV2MNxTYVh0twpl CfaCV2enPIXUeq4qaSEtynO33ewwqA0qsiOucREP4jZjglCqgXHzPxXqfY3nyhK7 AJIsOZbC7TaVYYwiKGr/njmzWx3Hma3TIEGSBbeaD/M8O9M0ulNwBj3IkxQ6EnOQ BP7A3Hn5uSTB+5qzxJcUHEj5A8HCwZRkvzQITdW4swi6XMmvlxWDkcSJ5ck9OtaQ vtiqgFnhbb4LlZDdqwPBZptSQ2SJw==
X-ME-Sender: <xms:zHS7XwUaiKk15sLeKaqZgBhD7Rhr1CxqY7pBrzUAajliWOfYz5cEMw> <xme:zHS7X0mu5RCM9qyDG6vMsIlzlbceDNO-dZUokeeIz9jbBlPfYtWXCgYZ4NRblkQMm C4LxfIvCT8HaihuGOA>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudeghedguddulecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecunecujfgurhepfffkvffuhfgjfhfogggtgfesth ejredtredtvdenucfhrhhomhepofgrrhhtihhnuceujhpnrhhklhhunhguuceomhgsjhdo ihgvthhfseegieeikedrshgvqeenucggtffrrghtthgvrhhnpeekudeuheduvdejtdduke eiheejleffvefhheffuefgueevhedvvdevieduheehgeenucffohhmrghinheprhhftgdq vgguihhtohhrrdhorhhgpdduhhhoshhtvgigrghmphhlvgdrtghomhdpvgigrghmphhlvg drtghomhdpihgvthhfrdhorhhgnecukfhppeduheekrddujeegrdegrddvudehnecuvehl uhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepmhgsjhdoihgvth hfseegieeikedrshgv
X-ME-Proxy: <xmx:zHS7X0YjDX3AVVvaYvsqjHAeRXN-EweR8p2q9HHg5sLqjRhjEAmdxQ> <xmx:zHS7X_VUtIlY0Do40Vgu4p49wxk22R94A_FGg16BFSRskKTQOp1DKQ> <xmx:zHS7X6kOPKqJO-qW6F4FfEfj-NNjPe4xjl7hH6vKRVfH7QxllXiTfw> <xmx:zXS7XxvEugeupZ19gG0qJWgnZTXEQV63PBS8XxuIyLtXdEnD3Ayjqw>
Received: from localhost (unknown [158.174.4.215]) by mail.messagingengine.com (Postfix) with ESMTPA id 1745A3280064; Mon, 23 Nov 2020 03:37:30 -0500 (EST)
Date: Mon, 23 Nov 2020 09:37:29 +0100
Message-Id: <20201123.093729.325420538444196813.id@4668.se>
To: rfc-editor@rfc-editor.org
Cc: andy@yumaworks.com, kwatsen@juniper.net, warren@kumari.net, rwilton@cisco.com, kent+ietf@watsen.net, mjethanandani@gmail.com, netconf@ietf.org
From: Martin Björklund <mbj+ietf@4668.se>
In-Reply-To: <20201122221445.1E11FF4071F@rfc-editor.org>
References: <20201122221445.1E11FF4071F@rfc-editor.org>
X-Mailer: Mew version 6.8 on Emacs 26.3
Mime-Version: 1.0
Content-Type: Text/Plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/9p0NnYcFh-zEheRTIq5z2nLpEIg>
Subject: Re: [netconf] [Technical Errata Reported] RFC8040 (6342)
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: NETCONF WG list <netconf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netconf>, <mailto:netconf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netconf/>
List-Post: <mailto:netconf@ietf.org>
List-Help: <mailto:netconf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netconf>, <mailto:netconf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 23 Nov 2020 08:37:36 -0000

Hi,

The issue boils down to if list keys are required in a plain patch.
Unfortunately, the RFC doesn't specifucy this.  From a technical pow,
list keys are not necessary.  In fact, if they are present in the
payload, they are redundant (since they are part of the URL) (this is
actually mentioned in the RFC).

Since it isn't clearly specified, I think we must assume that the keys
are not required.  Hence I think that this errata should be rejected.

In a future version of this document, the behaviour should be
clarified.


/martin


RFC Errata System <rfc-editor@rfc-editor.org> wrote:
> The following errata report has been submitted for RFC8040,
> "RESTCONF Protocol".
> 
> --------------------------------------
> You may review the report below and at:
> https://www.rfc-editor.org/errata/eid6342
> 
> --------------------------------------
> Type: Technical
> Reported by: Muly Ilan <muly_i@rad.com>
> 
> Section: 4.6.1
> 
> Original Text
> -------------
> To replace just the "year" field in the "album" resource (instead of
> replacing the entire resource with the PUT method), the client might
> send a plain patch as follows:
> PATCH /restconf/data/example-jukebox:jukebox/\
> library/artist=Foo%20Fighters/album=Wasting%20Light HTTP/1.1
> Host: example.com
> If-Match: "b8389233a4c"
> Content-Type: application/yang-data+xml
> <album xmlns="http://example.com/ns/example-jukebox">
> <year>2011</year>
> </album>
> 
> Corrected Text
> --------------
> To replace just the "year" field in the "album" resource (instead of
> replacing the entire resource with the PUT method), the client might
> send a plain patch as follows:
> PATCH /restconf/data/example-jukebox:jukebox/\
> library/artist=Foo%20Fighters/album=Wasting%20Light HTTP/1.1
> Host: example.com
> If-Match: "b8389233a4c"
> Content-Type: application/yang-data+xml
> <album xmlns="http://example.com/ns/example-jukebox">
> <name>Wasting Light</name>
> <year>2011</year>
> </album>
> 
> Notes
> -----
> Missing key leaf value in the message-body (<name>Wasting Light</name>)
> 
> Instructions:
> -------------
> This erratum is currently posted as "Reported". If necessary, please
> use "Reply All" to discuss whether it should be verified or
> rejected. When a decision is reached, the verifying party  
> can log in to change the status and edit the report, if necessary. 
> 
> --------------------------------------
> RFC8040 (draft-ietf-netconf-restconf-18)
> --------------------------------------
> Title               : RESTCONF Protocol
> Publication Date    : January 2017
> Author(s)           : A. Bierman, M. Bjorklund, K. Watsen
> Category            : PROPOSED STANDARD
> Source              : Network Configuration
> Area                : Operations and Management
> Stream              : IETF
> Verifying Party     : IESG
> 
> _______________________________________________
> netconf mailing list
> netconf@ietf.org
> https://www.ietf.org/mailman/listinfo/netconf