Re: [apps-discuss] Apps Dir review for: draft-daboo-webdav-sync-06

Cyrus Daboo <cyrus@daboo.name> Mon, 02 January 2012 18:42 UTC

Return-Path: <cyrus@daboo.name>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1F7CB21F84FC; Mon, 2 Jan 2012 10:42:15 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.25
X-Spam-Level:
X-Spam-Status: No, score=-102.25 tagged_above=-999 required=5 tests=[AWL=0.349, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id SMVHhNnxRGYF; Mon, 2 Jan 2012 10:42:14 -0800 (PST)
Received: from daboo.name (daboo.name [173.13.55.49]) by ietfa.amsl.com (Postfix) with ESMTP id A060121F84C9; Mon, 2 Jan 2012 10:42:14 -0800 (PST)
Received: from localhost (localhost [127.0.0.1]) by daboo.name (Postfix) with ESMTP id CFD5F1F9346F; Mon, 2 Jan 2012 13:42:12 -0500 (EST)
X-Virus-Scanned: amavisd-new at daboo.name
Received: from daboo.name ([127.0.0.1]) by localhost (daboo.name [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id AnmqNuWXzxYu; Mon, 2 Jan 2012 13:42:11 -0500 (EST)
Received: from [10.0.1.8] (unknown [173.13.55.49]) by daboo.name (Postfix) with ESMTPSA id 8F0421F9345D; Mon, 2 Jan 2012 13:42:10 -0500 (EST)
Date: Mon, 02 Jan 2012 13:42:04 -0500
From: Cyrus Daboo <cyrus@daboo.name>
To: Salvatore Loreto <salvatore.loreto@ericsson.com>, apps-discuss@ietf.org, draft-daboo-webdav-sync.all@tools.ietf.org
Message-ID: <EFF5A5DC54804393DA899C85@cyrus.local>
In-Reply-To: <4EEE4026.7050108@ericsson.com>
References: <4EEE4026.7050108@ericsson.com>
X-Mailer: Mulberry/4.1.0a3 (Mac OS X)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline; size=1162
Cc: The IESG <iesg@ietf.org>
Subject: Re: [apps-discuss] Apps Dir review for: draft-daboo-webdav-sync-06
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 02 Jan 2012 18:42:15 -0000

Hi Salvatore,
Thank you for your review. Comments inline:

--On December 18, 2011 9:33:58 PM +0200 Salvatore Loreto 
<salvatore.loreto@ericsson.com> wrote:

> Minor Issues:
>
>
> Abstract:
>
>
>     expand the WebDAV acronym on the first use.

Fixed for -07.

>    Preconditions:
>
>       (DAV:valid-sync-token): The DAV:sync-token element value MUST be a
>       valid token previously returned by the server.
>
>
> please clarify that the token is collection specific, so it needs to be
> originated
>  from the same collection.

Fixed in -07. Now reads:

    The DAV:sync-token element value MUST be a valid token previously
    returned by the server for the collection targeted by the request-URI.

> 	   <!ELEMENT multistatus (DAV:response*, DAV:responsedescription?,
> 	                          sync-token?) >
>
>
>
>
> remove the DAV: prefix.

Fixed in -07.

>
>
> Nits:
>
>
> References:
>
> - draft-ietf-vcarddav-carddav has been published as RFC 6352

Fixed in -07.

>  - the nits tools also report the following error:
>            Downref: Normative reference to an Experimental RFC: RFC 5842

AD is aware that a Downref occurs.


-- 
Cyrus Daboo