From murch@fastmail.com Mon Apr 8 06:16:24 2024 Return-Path: X-Original-To: extra@ietfa.amsl.com Delivered-To: extra@ietfa.amsl.com Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0CE18C14F6F0 for ; Mon, 8 Apr 2024 06:16:24 -0700 (PDT) X-Virus-Scanned: amavisd-new at amsl.com X-Spam-Flag: NO X-Spam-Score: -9.159 X-Spam-Level: X-Spam-Status: No, score=-9.159 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, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, NICE_REPLY_A=-2.064, RCVD_IN_DNSWL_HI=-5, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-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=fastmail.com header.b="R/hw0/f6"; dkim=pass (2048-bit key) header.d=messagingengine.com header.b="AeTc6/Lp" 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 7AwkXSHVjQaV for ; Mon, 8 Apr 2024 06:16:19 -0700 (PDT) Received: from fhigh7-smtp.messagingengine.com (fhigh7-smtp.messagingengine.com [103.168.172.158]) (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 21103C14F6E4 for ; Mon, 8 Apr 2024 06:16:19 -0700 (PDT) Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailfhigh.nyi.internal (Postfix) with ESMTP id EB35E11400FA; Mon, 8 Apr 2024 09:16:17 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Mon, 08 Apr 2024 09:16:17 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastmail.com; h= cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:subject :subject:to:to; s=fm2; t=1712582177; x=1712668577; bh=hAO14ahdU5 dk00UI+I1YkukfqDlu34iVGkq9l1xGi3w=; b=R/hw0/f69HZexlbCiwBhNRWSA+ 3HfS6ghUKETjSVlRcYjkvItJEGRVq0O34YOtxJ0dNnydylaTUhjUVioYWtRS2okL nkdDdD0FI9UVbQMqjLuFYy3328AEdpucq7favOa3JgT8tlGWfrx5PE5/Qfw0ll1f FX7ReYT+eRMtFzt7/14tBAhslHW0Ark4IjLnICCp4/f/1WYt7CK6CqoUVtT43fjQ 6JUiigHV6RlvT8kIfAGtR4PTwIMFYvkK+5coVMO1a2kJB3vTDbVIRmYbuOu9I96P n92TMFGoexoZFkEdHaGmgmCa830+1LQhI/3THJFU3mJ5n0TAisAk6otpagkQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc: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:subject:subject:to :to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm2; t=1712582177; x=1712668577; bh=hAO14ahdU5dk00UI+I1YkukfqDlu 34iVGkq9l1xGi3w=; b=AeTc6/Lp9IBVYgpVp15jBdwosYKIv9VzqPkXomMbMEse mZb6jHUY6E5FUqL8mtKrBZZ0CDYCBqaLKt35h2Rhx4q7shi/Ncomh3ZTG6o6VsZS RzsbBoC+8D8RYskItc57LOHUoAkuRgo8xskXEu2XgqA+btntnHYCjeJgef6+PhzD tcuSNrDDHWv+4LzkhvSQAtJDGfRzFZBWoJA6BZskvGnHGQ9oXd+YGYtp/j+FALNJ +/d063aRuVRvEfY7KMKqQ2AJh73kR+xs1m8WS5O1lbAYXF7HOpGw97HVpvC0ZnoI HgIFqHRzo7PJBxTE4nlOeFEkYJVp9fUTzQ5EAM2gSg== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvledrudegiedgiedvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurheptgfkffggfgfuvfhfhfgjsegrtderredtfeejnecuhfhrohhmpefmvghnucfo uhhrtghhihhsohhnuceomhhurhgthhesfhgrshhtmhgrihhlrdgtohhmqeenucggtffrrg htthgvrhhnpeffteekteeguedvveetjedtudfgffekteduteduheeiheegveffgfelfeej ffetudenucffohhmrghinhepihgvthhfrdhorhhgnecuvehluhhsthgvrhfuihiivgeptd enucfrrghrrghmpehmrghilhhfrhhomhepmhhurhgthhesfhgrshhtmhgrihhlrdgtohhm X-ME-Proxy: Feedback-ID: ibf914243:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 8 Apr 2024 09:16:17 -0400 (EDT) Content-Type: multipart/alternative; boundary="------------ph0TWL6Fl4K8ojoeJNrbMTI9" Message-ID: Date: Mon, 8 Apr 2024 09:16:16 -0400 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Content-Language: en-US To: Alexey Melnikov , extra@ietf.org References: <671d2e4c-de7c-45dd-bec4-6fc10aae4e46@app.fastmail.com> <7cc76fd6-dbb3-455c-8aeb-13430f843b3b@isode.com> From: Ken Murchison In-Reply-To: <7cc76fd6-dbb3-455c-8aeb-13430f843b3b@isode.com> Archived-At: Subject: Re: [Extra] Working Group Last Call: draft-ietf-extra-processimip (and call for shepherd) X-BeenThere: extra@ietf.org X-Mailman-Version: 2.1.39 Precedence: list List-Id: Email mailstore and eXtensions To Revise or Amend List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Apr 2024 13:16:24 -0000 This is a multi-part message in MIME format. --------------ph0TWL6Fl4K8ojoeJNrbMTI9 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Hi Alexey, On 4/5/24 1:25 PM, Alexey Melnikov wrote: > > Hi, > > On 20/03/2024 05:48, Bron Gondwana wrote: >> Hi All, >> >> This email starts a working group last call for the document at: >> >> https://datatracker.ietf.org/doc/draft-ietf-extra-processimip/ >> >> If you have any feedback, change requests, or just "yes I support >> this document" please reply to this email *by Thursday, April 4th* (a >> little over 2 weeks from now). > > I am sorry I missed the deadline! > > The document is in a good shape. A small nit: > > 4.8. Error String Argument > > The optional :errstr argument specifies the name of a variable into > which a string describing the reason for the outcome will be stored. > If no reason for the outcome is available, implementations MUST set > the variable to the empty string. > > Is this trying to say that this is set to "" for any :outcome other than "error"? If yes, maybe it would be clearer to say something like this: > The optional :errstr argument specifies the name of a variable into > which a string describing the error reason will be stored (when the outcome is "error"). > For all other outcome values, implementations MUST set > the variable to the empty string. Technically, there could be a an :errstr for any of the :outcomes.  For instance, for "no_action" the :errstr could be "only processing updates".  But that leads to :errstr being a poorly chosen name.  I'm also fine stating that :errstr only applies if the outcome is "error". Thoughts? -- Kenneth Murchison Senior Software Developer Fastmail US LLC --------------ph0TWL6Fl4K8ojoeJNrbMTI9 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 8bit

Hi Alexey,


On 4/5/24 1:25 PM, Alexey Melnikov wrote:

Hi,

On 20/03/2024 05:48, Bron Gondwana wrote:
Hi All,

This email starts a working group last call for the document at:


If you have any feedback, change requests, or just "yes I support this document" please reply to this email by Thursday, April 4th (a little over 2 weeks from now).

I am sorry I missed the deadline!

The document is in a good shape. A small nit:

4.8.  Error String Argument

   The optional :errstr argument specifies the name of a variable into
   which a string describing the reason for the outcome will be stored.
   If no reason for the outcome is available, implementations MUST set
   the variable to the empty string.

Is this trying to say that this is set to "" for any :outcome other than "error"? If yes, maybe it would be clearer to say something like this:

   The optional :errstr argument specifies the name of a variable into
   which a string describing the error reason will be stored (when the outcome is "error").
   For all other outcome values, implementations MUST set
   the variable to the empty string.


Technically, there could be a an :errstr for any of the :outcomes.  For instance, for "no_action" the :errstr could be "only processing updates".  But that leads to :errstr being a poorly chosen name.  I'm also fine stating that :errstr only applies if the outcome is "error".

Thoughts?


-- 
Kenneth Murchison
Senior Software Developer
Fastmail US LLC
--------------ph0TWL6Fl4K8ojoeJNrbMTI9--