Re: [Gen-art] Genart last call review of draft-ietf-httpapi-link-template-02

Mark Nottingham <mnot@mnot.net> Fri, 19 May 2023 12:24 UTC

Return-Path: <mnot@mnot.net>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4110AC151996; Fri, 19 May 2023 05:24:50 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.097
X-Spam-Level:
X-Spam-Status: No, score=-2.097 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=mnot.net header.b="PwmYJxk9"; dkim=pass (2048-bit key) header.d=messagingengine.com header.b="xZN0EiOq"
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 F6suNrGZBUbj; Fri, 19 May 2023 05:24:45 -0700 (PDT)
Received: from wout4-smtp.messagingengine.com (wout4-smtp.messagingengine.com [64.147.123.20]) (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 73EF8C151066; Fri, 19 May 2023 05:24:44 -0700 (PDT)
Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailout.west.internal (Postfix) with ESMTP id 7FB6F32003D3; Fri, 19 May 2023 08:24:41 -0400 (EDT)
Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Fri, 19 May 2023 08:24:41 -0400
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mnot.net; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm2; t= 1684499081; x=1684585481; bh=cktAh0gzh83la0/HFSwPybypLitdn+Kacla 5ACQFFPQ=; b=PwmYJxk9CMo85G89pBLJUCEYQQZ/oGMqVJI1O+WZbMKHnhtFFC5 mKEFBzWTFE8eK4H8eIm1O/EHeuGOYI/1N7RjzqB0Hw5Q8HBNVMAyXnXpMKJM3eas I7JIrE1bsRFXtwyljMDAlb8KQ9VaxztH2TcGcPuuGvxzaw+rREHJGJMPL0K0KJRn qyelqqQiyF7/mFm4f9lvxKwe5b/xyrkWllUM+Cmi8hpzTPal4gXUlhmZPrYZfjmI nkyEr0lR568YrH0guGnZsvpJLt4+Q4bQMuzqp2aniCJHFmGl5vGwTB5eqO6I69iu g5vDqlvn92GqYtQZrLrkcxY+setswBJV3+Q==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t= 1684499081; x=1684585481; bh=cktAh0gzh83la0/HFSwPybypLitdn+Kacla 5ACQFFPQ=; b=xZN0EiOqJqiKNpSKfoGYeKlgvXreIZjf8M5EERLbv7n/SZ3veoF bKl2bJcqnRIuoF4Bt9YX7pe5n18VGV9h0FHrOH75203QtjaRv9pXVsDrCoxaW7rv UHSOu8rGNwHjaur+Mh4By1TYsYe+fBf1k4kYgG5TGXanC2sbQJeAyRGdS8+rKNjQ IUS1Q+G5IzdP+G8N6pdmiDBE0vPai5ZN1WmTX0J6/TQF+S6EvO7mEEGH8dk/wYAn vraq4J6jGIc4CviYwgpaNhnZn9cKUTpJHs65jmWQ6B+voaOgnyxOI2YLzX6eUXph HY2/cmA7AoB/9Y/TsKLQ2L1LEyl3cVwX5iQ==
X-ME-Sender: <xms:iGpnZFIW8JH7UCkRVOWDzfie0ZWYeXm5yp2oqCsyYA1LErB2WN7S3w> <xme:iGpnZBJJ1c76l2zQdslM-2GDUSkbEyjUZ5P2k4tMIxH3Si7dARndQxg2mLkrkhiI7 mUQOiP1uANHpTaP3w>
X-ME-Received: <xmr:iGpnZNugbqAyKnK0jidJYW8q-hAqN-O9iEfeReNZNbXiVLL-LFAnu4srOMo2K1YwNwY>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrfeeihedgheefucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurheptggguffhjgffvefgkfhfvffosehtqhhmtdhhtddvnecuhfhrohhmpeforghr khcupfhothhtihhnghhhrghmuceomhhnohhtsehmnhhothdrnhgvtheqnecuggftrfgrth htvghrnheptddtgefgueevtddugfdtkeffudegveetffegjeelhfdvtedvueejteegueeg teetnecuffhomhgrihhnpehmnhhothdrnhgvthenucevlhhushhtvghrufhiiigvpedtne curfgrrhgrmhepmhgrihhlfhhrohhmpehmnhhothesmhhnohhtrdhnvght
X-ME-Proxy: <xmx:iGpnZGarZiwY8ntchDagWLqsUbteF1VW-Gd3GBwTNH1A27QffIWr6g> <xmx:iGpnZMZiANi1s_sGzgb2ZL-PylmIij9A6vQ_gx6drlCGX1fgFZtcYg> <xmx:iGpnZKCOVpQpu8fJ9VsYXkpiN1B9f3gDfMdoa1D8sddvxumB4rYIqQ> <xmx:iWpnZLnCzhJesOCHGXRSauQK_282c2yElPaDQXnYUI-aLhAUFH-7Gg>
Feedback-ID: ie6694242:Fastmail
Received: by mail.messagingengine.com (Postfix) with ESMTPA; Fri, 19 May 2023 08:24:40 -0400 (EDT)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3731.500.231\))
From: Mark Nottingham <mnot@mnot.net>
In-Reply-To: <HE1PR07MB4441A4329BA51DC42C9796DE937C9@HE1PR07MB4441.eurprd07.prod.outlook.com>
Date: Fri, 19 May 2023 13:24:39 +0100
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, HTTP APIs Working Group <httpapi@ietf.org>, Last Call <last-call@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <C5848939-4628-4760-B1B7-ECB7105B866A@mnot.net>
References: <168424064858.44084.3692769234533396629@ietfa.amsl.com> <34AA44A9-B351-4798-8920-90724F600D8D@mnot.net> <HE1PR07MB4441A4329BA51DC42C9796DE937C9@HE1PR07MB4441.eurprd07.prod.outlook.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>
X-Mailer: Apple Mail (2.3731.500.231)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/raA9_BPevmJR4QS2K9TvThP7ThA>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-httpapi-link-template-02
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 19 May 2023 12:24:50 -0000

Hi Christer,

Thanks, see responses below.

> On 19 May 2023, at 1:13 pm, Christer Holmberg <christer.holmberg@ericsson.com> wrote:
> 
> Hi Mark,
> 
> Please see inline.
> 
>>> Q2_1: There is no ABNF for the header field. There are examples using
>>> both quotes ("/{username}") and angle brackets
>>> (</books/{book_id}/author>), so please include the ABNF.
>> 
>> This is a Structured Field; in the HTTP community, we've agreed that 
>> documenting them with ABNF is not good practice.
> 
> Gotha.
> 
> (I was comparing this to RFC 8288, and assumed that you still would have to 
> define the ABNF, following the rules of [STRUCTURED-FIELDS])
> 
> But, then I have the following question:
> 
> Section 2 says:   "Its value is a List of Strings.  Each String is a URI 
> Template..."
> 
> According to [STRUCTURED-FIELDS]), the syntax for String is:  sf-string = 
> DQUOTE *chr DQUOTE
> 
> From the example in Section 2, </books/{book_id}/author> does not follow that 
> syntax, does it? You must use DQUOTEs.

Yes, that was a transposition error that's since been corrected.


>>> Q2_2: The text says "Parameter values MUST be Strings."
>>> 
>>> It is unclear what "Strings" means. Does it mean that parameter values
>>> must be encoded as quoted-strings? If so, why? RFC8288 says that
>>> parameter values can be encoded both as token and quoted-string.
>> 
>> S 1.1: "This specification uses the following terms from 
>> [STRUCTURED-FIELDS]: List, String, Parameter."
> 
> Yes. But, in Section 3.1.2 of [STRUCTURED-FIELDS] there is no restriction that 
> the Parameter value must by String. My question is why you are making that 
> restriction, instead of just allowing the different Parameter value encodings 
> defined in [STRUCTURED-FIELDS]?

Ah. The reason is that allowing any type would require creating a mapping of current values to SF types, and there are just too many potential (and undocumented) values already in use to do this. 

Cheers,

--
Mark Nottingham   https://www.mnot.net/