[Ietf-carddav] minor comments on -01 draft

Arnaud Quillaud <Arnaud.Quillaud@Sun.COM> Wed, 09 May 2007 15:45 UTC

Return-Path: <Arnaud.Quillaud@Sun.COM>
X-Original-To: ietf-carddav@osafoundation.org
Delivered-To: ietf-carddav@osafoundation.org
Received: from laweleka.osafoundation.org (laweleka.osafoundation.org [204.152.186.98]) by leilani.osafoundation.org (Postfix) with ESMTP id 1FA147F6E6 for <ietf-carddav@osafoundation.org>; Wed, 9 May 2007 08:45:19 -0700 (PDT)
Received: from localhost (laweleka.osafoundation.org [127.0.0.1]) by laweleka.osafoundation.org (Postfix) with ESMTP id DB44514227F for <ietf-carddav@osafoundation.org>; Wed, 9 May 2007 08:44:23 -0700 (PDT)
X-Virus-Scanned: by amavisd-new and clamav at osafoundation.org
X-Spam-Score: -1.189
X-Spam-Level:
X-Spam-Status: No, score=-1.189 tagged_above=-50 required=4 tests=[AWL=1.332, BAYES_00=-2.599, TW_BD=0.077, UNPARSEABLE_RELAY=0.001]
Received: from laweleka.osafoundation.org ([127.0.0.1]) by localhost (laweleka.osafoundation.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id vhBv92xiNu1N for <ietf-carddav@osafoundation.org>; Wed, 9 May 2007 08:44:21 -0700 (PDT)
Received: from gmp-ea-fw-1.sun.com (gmp-ea-fw-1.sun.com [192.18.1.36]) by laweleka.osafoundation.org (Postfix) with ESMTP id 47D6D142276 for <ietf-carddav@osafoundation.org>; Wed, 9 May 2007 08:44:21 -0700 (PDT)
Received: from d1-emea-09.sun.com ([192.18.2.119]) by gmp-ea-fw-1.sun.com (8.13.6+Sun/8.12.9) with ESMTP id l49FiKaL011388 for <ietf-carddav@osafoundation.org>; Wed, 9 May 2007 15:44:20 GMT
Received: from conversion-daemon.d1-emea-09.sun.com by d1-emea-09.sun.com (Sun Java System Messaging Server 6.2-6.01 (built Apr 3 2006)) id <0JHS00F0169HNW00@d1-emea-09.sun.com> (original mail from Arnaud.Quillaud@Sun.COM) for ietf-carddav@osafoundation.org; Wed, 09 May 2007 16:44:20 +0100 (BST)
Received: from KONE-JHY8LIXZ2A.Sun.COM ([129.150.116.246]) by d1-emea-09.sun.com (Sun Java System Messaging Server 6.2-6.01 (built Apr 3 2006)) with ESMTPSA id <0JHS007RL6DVQT70@d1-emea-09.sun.com> for ietf-carddav@osafoundation.org; Wed, 09 May 2007 16:44:20 +0100 (BST)
Content-return: prohibited
Date: Wed, 09 May 2007 17:44:34 +0200
From: Arnaud Quillaud <Arnaud.Quillaud@Sun.COM>
Sender: Arnaud.Quillaud@Sun.COM
To: ietf-carddav@osafoundation.org
Message-id: <0JHS007RM6DVQT70@d1-emea-09.sun.com>
MIME-version: 1.0
X-Mailer: Sun Outlook Connector 7.2.310.1
Content-type: TEXT/PLAIN; CHARSET=ISO-8859-1
Content-transfer-encoding: 7BIT
Subject: [Ietf-carddav] minor comments on -01 draft
X-BeenThere: ietf-carddav@osafoundation.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: ietf-carddav.osafoundation.org
List-Unsubscribe: <http://lists.osafoundation.org/cgi-bin/mailman/listinfo/ietf-carddav>, <mailto:ietf-carddav-request@osafoundation.org?subject=unsubscribe>
List-Archive: <http://lists.osafoundation.org/pipermail/ietf-carddav>
List-Post: <mailto:ietf-carddav@osafoundation.org>
List-Help: <mailto:ietf-carddav-request@osafoundation.org?subject=help>
List-Subscribe: <http://lists.osafoundation.org/cgi-bin/mailman/listinfo/ietf-carddav>, <mailto:ietf-carddav-request@osafoundation.org?subject=subscribe>
X-List-Received-Date: Wed, 09 May 2007 15:45:20 -0000

Some (rather minor) comments on the CardDAV Draft.

First of all, I find the name ("adbk") a little bit cryptic. "addressbook", "contactbook" would be more clear although a bit long since the name is used as a prefix in most xml elements.

 
1. Introduction and Overview
 
ACAP, LDAP and WebDAV are mentioned but SyncML is missing. Shouldn't we mention it and explain how it does not address the need ?


 
3. Requirements Overview:
 
Why is the support for MKADDRB a MUST when it is a SHOULD for CalDAV ? Not that I care that much but why the difference ?

 

5.1 vCard Object Resources
 
"UID must be unique within a collection".

Nevertheless, according to the UID definition in vCard (http://tools.ietf.org/html/rfc2426#section-3.6.7), the UID identifies the individual or resource. So potentially, you could have a personal vCard and a business vCard corresponding to the same individual. I doubt that it will be the case in practice but who knows...
 
 

6.2.1 adbk-description
 
Although my native language is just "fr" (or is it fr-FR ?) and not "fr-CA", I suspect that "Address de Oliver Daboo" is not really correct. "Adresses de Oliver Daboo" would be more appropriate.
 


6.3.2.  Creating vCard Object Resources
 
The UID property is missing from the example although it is a mandatory property.


8.3. Searching Text: Collations
(and other places)

draft-newman-i18n-comparator is now RFC 4790 ( http://tools.ietf.org/html/rfc4790 ).



8.8.  CARDDAV:adbk-sync Report:

It is mentioned that 
<<
The "adbk-sync" reports allows the client to specify whether it
   should receive vCard data for those objects that are new or have
   changed, and it uses the "adbk-data" element (also used in the "adbk-
   query" and "adbk-multiget" reports) for that purpose.
>>
Nevertheless, in the XML definition for abbk-sync (10.9. CARDDAV:adbk-sync XML Element) lacks "(DAV:allprop | DAV:propname | DAV:prop)?" as potential children.

BTW, wouldn't it be possible to move the whole abdk-sync report definition into another draft so that a common mechanism could be used against other types of collections (WebDAV, CalDAV,...) ?

Still in 8.8, it says "The response body for a successful CARDDAV:adbk-multiget REPORT" instead of "The response body for a successful CARDDAV:adbk-sync REPORT"

Arnaud Q