Re: [calsify] AD review of draft-ietf-calext-caldav-attachments-02

Alexey Melnikov <alexey.melnikov@isode.com> Sun, 30 April 2017 14:59 UTC

Return-Path: <alexey.melnikov@isode.com>
X-Original-To: calsify@ietfa.amsl.com
Delivered-To: calsify@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 46D4A126BF6 for <calsify@ietfa.amsl.com>; Sun, 30 Apr 2017 07:59:04 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.698
X-Spam-Level:
X-Spam-Status: No, score=0.698 tagged_above=-999 required=5 tests=[BAYES_50=0.8, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=isode.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id npa5dgmB4Tde for <calsify@ietfa.amsl.com>; Sun, 30 Apr 2017 07:59:02 -0700 (PDT)
Received: from waldorf.isode.com (waldorf.isode.com [62.232.206.188]) by ietfa.amsl.com (Postfix) with ESMTP id CD9BB127978 for <calsify@ietf.org>; Sun, 30 Apr 2017 07:56:53 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1493564212; d=isode.com; s=june2016; i=@isode.com; bh=uqV2AI5Dscb1k+Yx2xmsQLgFxqz9jDW5b6htDw7bdbM=; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version: In-Reply-To:References:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description; b=LDQoZE74jovlt24rh1gWWRsM1ffzoTNneZxsITyk4EpJJwtJghybdL2GGdrbD33ierArrk I+V47+bmXH2QK+hiBVGAoUWWOZ9Qpa7HFF9QZ/v6BfW8VzgPS39bV2Dr5x9XhKfazinPtZ Iqwuk5iBeXPUShdcgynb8VGg7JjaXt0=;
Received: from [192.168.0.3] (cpc121086-nmal24-2-0-cust54.19-2.cable.virginm.net [77.97.145.55]) by waldorf.isode.com (submission channel) via TCP with ESMTPSA id <WQX7MwBV0ncE@waldorf.isode.com>; Sun, 30 Apr 2017 15:56:52 +0100
To: calsify@ietf.org
References: <5905D298.6060204@isode.com>
From: Alexey Melnikov <alexey.melnikov@isode.com>
Message-ID: <5905FB4B.8060902@isode.com>
Date: Sun, 30 Apr 2017 15:57:15 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0
In-Reply-To: <5905D298.6060204@isode.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="windows-1252"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/calsify/AGe6FCYyGDcKBOXRCe5apU2ib-c>
Subject: Re: [calsify] AD review of draft-ietf-calext-caldav-attachments-02
X-BeenThere: calsify@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: <calsify.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/calsify>, <mailto:calsify-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/calsify/>
List-Post: <mailto:calsify@ietf.org>
List-Help: <mailto:calsify-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/calsify>, <mailto:calsify-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 30 Apr 2017 14:59:04 -0000

On 30/04/2017 13:03, Alexey Melnikov wrote:
> Hi,
> The document is in a good shape, but I have a few minor comments/questions:

I should add that none of my comments are blocking, so I will start IETF
LC on the document.

> In Section 3 HTTP PUT is not mentioned. I think it should be added.
> 
> All examples should use https: URI scheme.
> 
> In 3.7: if SIZE property is included, should its value be
> replaced/ignored/validated?
> 
> In 3.12.3: is support for Expect: 100-continue required in compliant
> servers?
> 
> In Section 8 (Security Considerations): is it worth mentioning
> Denial-of-Service attacks?
> 
> Appendix B, page 31: just to help me understand the example: after
> adding an attachment the RECURRENCE-ID added is the same as the "rid"
> request in the attachment-add request.
> 
> Thank you,
> Alexey