IETF Logo Mail Archive

Re: [Jmap] Review of draft-ietf-jmap-portability-extensions-00

Joris Baum <joris@audriga.com> Wed, 20 March 2024 16:36 UTC

Return-Path: <joris@audriga.com>
X-Original-To: jmap@ietfa.amsl.com
Delivered-To: jmap@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 670A7C14F61E for <jmap@ietfa.amsl.com>; Wed, 20 Mar 2024 09:36:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.905
X-Spam-Level:
X-Spam-Status: No, score=-1.905 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=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 RBXdLQ0E4UlC for <jmap@ietfa.amsl.com>; Wed, 20 Mar 2024 09:36:03 -0700 (PDT)
Received: from mail.audriga.com (mail.audriga.com [176.221.42.35]) (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 00A15C151082 for <jmap@ietf.org>; Wed, 20 Mar 2024 09:35:45 -0700 (PDT)
Received: from localhost (localhost.localdomain [127.0.0.1]) by mail.audriga.com (Postfix) with ESMTP id 453A7A1E9 for <jmap@ietf.org>; Wed, 20 Mar 2024 17:35:43 +0100 (CET)
X-Virus-Scanned: Debian amavisd-new at mail.audriga.com
Received: from mail.audriga.com ([127.0.0.1]) by localhost (mail.audriga.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id vuReAZD2KU0z for <jmap@ietf.org>; Wed, 20 Mar 2024 17:35:40 +0100 (CET)
Received: from [192.168.10.127] (ip-109-090-161-242.um36.pools.vodafone-ip.de [109.90.161.242]) (Authenticated sender: joris@audriga.com) by mail.audriga.com (Postfix) with ESMTPSA id E6908A1E0 for <jmap@ietf.org>; Wed, 20 Mar 2024 17:35:40 +0100 (CET)
Content-Type: multipart/alternative; boundary="------------Bi74DCZBmL1tID7Jbyy5Az0N"
Message-ID: <7fc0a88f-230c-49d9-a5c2-668778ddcfe2@audriga.com>
Date: Wed, 20 Mar 2024 17:35:40 +0100
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Content-Language: en-US
To: jmap@ietf.org
References: <f4a8f1dd-0764-4241-a982-baa84a8cb924@fastmail.com> <b49c67a8-cfb3-4a67-a65b-1df50b5a23e9@fastmail.com>
From: Joris Baum <joris@audriga.com>
Autocrypt: addr=joris@audriga.com; keydata= xjMEXmiiVxYJKwYBBAHaRw8BAQdAdZHr1ErnL1M6znXii/tmQdbrX2WYv7z2IOX24nQI/ILN HkpvcmlzIEJhdW0gPGpvcmlzQGF1ZHJpZ2EuY29tPsKWBBMWCAA+FiEEcn/m2ZrBrKtT4eWN /E+o0tJIXvAFAl5oolcCGwMFCQlmAYAFCwkIBwIGFQoJCAsCBBYCAwECHgECF4AACgkQ/E+o 0tJIXvBxNwD9FTxAqK3hInm0FO8PkbKnoMs39U8uIsyWzZ6OQxNKAqoBAILUwL4+zZ27pJwr cpLaLrbGJ7jFH4gvaXD9pSsVrN4OzjgEXmiiVxIKKwYBBAGXVQEFAQEHQOwg+TuTO26r4K5V BzYwVGK9EXrx6UpBxiubgDlHdY0KAwEIB8J+BBgWCAAmFiEEcn/m2ZrBrKtT4eWN/E+o0tJI XvAFAl5oolcCGwwFCQlmAYAACgkQ/E+o0tJIXvC2HwD/VjK0qWcInLxsNA+4IpgsZeR6U3bO K0NUuXoxZLGiOG8BAJNp3V+nFFgguUohvpSzw7sI4h4QXKVuVMhpG43PHVsD
In-Reply-To: <b49c67a8-cfb3-4a67-a65b-1df50b5a23e9@fastmail.com>
Archived-At: <https://mailarchive.ietf.org/arch/msg/jmap/gwoP2375QDNUBlnMrBz7UGoPVQ8>
Subject: Re: [Jmap] Review of draft-ietf-jmap-portability-extensions-00
X-BeenThere: jmap@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: JSON Message Access Protocol <jmap.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/jmap>, <mailto:jmap-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/jmap/>
List-Post: <mailto:jmap@ietf.org>
List-Help: <mailto:jmap-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/jmap>, <mailto:jmap-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 20 Mar 2024 16:36:07 -0000

Hi,

thanks for your review. I uploaded a new version that should address all 
your comments.

I also added one additional property called capabilityOverrides for 
Backend Info that I had kept our until now.


On 20.03.24 12:03, Ken Murchison wrote:
>
> Something else:
>
> 1.2.1 and 4.1: What is the actual capability? 
> |urn:ietf:params:jmap:core:backendinfo or 
> ||urn:ietf:params:jmap:backendinfo |?
>
I went with |||urn:ietf:params:jmap:backendinfo| now. The additional 
"core:" was there to signal that this JMAP extension only extends the 
session object. However, I think its unnecessary on second glance.


> On 3/20/24 12:23 AM, Ken Murchison wrote:
>
>> This doc looks fairly complete.  A few nits:
>>
>> Section 1.2.1: The environment property is defined as being a String, 
>> but the example shows is as a SoftwareInfo object.

Fixed.


>> Section 1.2.1: As SoftwareInfo is also used for product, the 
>> definition of  the name property should probably just say something 
>> like "The name of the software" and not specify "JMAP API backend".

Agreed and fixed.


>> Section 1.2.2: I would think that the client would have to opt-in to 
>> receiving logs, but I don't see that in the text.  As a server 
>> developer, I don't want to take the time to generating these logs for 
>> a client that either doesn't want them and/or doesn't support them.

Yes. Added a text and extended the example.


Regards,


Joris


> -- 
> Kenneth Murchison
> Senior Software Developer
> Fastmail US LLC
>
> _______________________________________________
> Jmap mailing list
> Jmap@ietf.org
> https://www.ietf.org/mailman/listinfo/jmap

-- 
Joris Baum
Tel: +49 721 170293 16
Fax: +49 721 170293 179

http://www.audriga.com  |http://www.twitter.com/audriga

--------------------------------------------------------------------------
audriga GmbH |  Alter Schlachthof 57  | 76137 Karlsruhe
Sitz der Gesellschaft: Karlsruhe - Amtsgericht Mannheim - HRB 713034
Geschäftsführer: Dr. Frank Dengler, Dr. Hans-Jörg Happel
--------------------------------------------------------------------------

v2.23.0 | Report a Bug | By Email