Re: [core] Genart last call review of draft-ietf-core-links-json-07

Carsten Bormann <cabo@tzi.org> Thu, 27 April 2017 12:30 UTC

Return-Path: <cabo@tzi.org>
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 2B685129463; Thu, 27 Apr 2017 05:30:39 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.199
X-Spam-Level:
X-Spam-Status: No, score=-4.199 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 nqUhh2g3iTSg; Thu, 27 Apr 2017 05:30:37 -0700 (PDT)
Received: from mailhost.informatik.uni-bremen.de (mailhost.informatik.uni-bremen.de [IPv6:2001:638:708:30c9::12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DF56A1273B1; Thu, 27 Apr 2017 05:30:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at informatik.uni-bremen.de
Received: from submithost.informatik.uni-bremen.de (submithost.informatik.uni-bremen.de [IPv6:2001:638:708:30c9::b]) by mailhost.informatik.uni-bremen.de (8.14.5/8.14.5) with ESMTP id v3RCURe0018959; Thu, 27 Apr 2017 14:30:27 +0200 (CEST)
Received: from client-0199.vpn.uni-bremen.de (client-0199.vpn.uni-bremen.de [134.102.107.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by submithost.informatik.uni-bremen.de (Postfix) with ESMTPSA id 3wDGW70pv5zDHMX; Thu, 27 Apr 2017 14:30:27 +0200 (CEST)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\))
From: Carsten Bormann <cabo@tzi.org>
In-Reply-To: <720C9B43-B803-4C99-A366-3C79F463FB23@tzi.org>
Date: Thu, 27 Apr 2017 14:30:26 +0200
Cc: gen-art@ietf.org, ietf@ietf.org, core@ietf.org, draft-ietf-core-links-json.all@ietf.org
X-Mao-Original-Outgoing-Id: 514989026.323576-92920dfcf09ad9fd924409fa3dea0ae4
Content-Transfer-Encoding: quoted-printable
Message-Id: <218E712A-7379-4764-B59A-FD0A186F4520@tzi.org>
References: <149315879365.13684.3263173090290877403@ietfa.amsl.com> <720C9B43-B803-4C99-A366-3C79F463FB23@tzi.org>
To: Elwyn Davies <elwynd@dial.pipex.com>
X-Mailer: Apple Mail (2.3273)
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/DaZk1p5Qys7kg6v_99QNMdweSMc>
Subject: Re: [core] Genart last call review of draft-ietf-core-links-json-07
X-BeenThere: core@ietf.org
X-Mailman-Version: 2.1.22
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: Thu, 27 Apr 2017 12:30:39 -0000

Hi Elwyn,

we tried one round at your comments.

We didn’t tackle the interaction with /.well-known/core — I continue to believe what I said yesterday, but we still have to find a good way to put this into the document.

We added a note that we need to decide whether there is an onus on a receiving CBOR implementation to check for the absence of the 13 strings as names (“Adam’s second issue”).

The title* issue also requires some more thinking — do we even want to have language tags on a constrained device.  Right now, the document simply leaves the whole subject out, but probably needs a position on that.

Finally, there is the issue that RFC 6690 percent-decodes and how that should be reflected in the JSON and CBOR variants.

We added a whole subsection on converting back to RFC 6690, which discusses the current pragmatic approach on solving the “to quote or not to quote” issue.

On the editorial side, we addressed the document title, added some language to make it clear that the CDDL is informative (and actually added build tool rules to check the examples against the CDDL, ouch), and quite a few nits.  Thank you for the text suggestions (and no, we didn’t add a reference to X.690 yet :-).  Based on Adam’s suggestions, we found a way to fix the bullet list in 2.2.

We still need to review the editorial comments on the detailed processing of strings etc. (see may previous comments why these didn’t just drop in).

Please have a look at the new text at:

https://tools.ietf.org/rfcdiff?url2=draft-ietf-core-links-json-08.txt

Thank you!

Grüße, Carsten