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

Alexey Melnikov <alexey.melnikov@isode.com> Sun, 30 April 2017 12:04 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 E140B1293F3 for <calsify@ietfa.amsl.com>; Sun, 30 Apr 2017 05:04:23 -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 bGL8pMIjBgZt for <calsify@ietfa.amsl.com>; Sun, 30 Apr 2017 05:04:22 -0700 (PDT)
Received: from statler.isode.com (Statler.isode.com [62.232.206.189]) by ietfa.amsl.com (Postfix) with ESMTP id 31AE7127444 for <calsify@ietf.org>; Sun, 30 Apr 2017 05:03:15 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1493553794; d=isode.com; s=june2016; i=@isode.com; bh=+CRuLmhKEuC9PgES1PgYLgIW7FvF4pl5PwyzI/oc5OQ=; 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=pP4uw8tPMR6jJ2MlGqI69IX1+MX5DZBdP4Ee08l3+DEyqtZvxW1opX3Ye1574S6g124uUk 0R3a+KzxVK22lsGH4KCY9T1VWkeNEiTcPPSksvu6g3LOIj4uMlL0unjJCajiqYy8AE6vZS 3GAn189GuD2ahMEKVJhOvVUV05k8XXE=;
Received: from [192.168.0.3] (cpc121086-nmal24-2-0-cust54.19-2.cable.virginm.net [77.97.145.55]) by statler.isode.com (submission channel) via TCP with ESMTPSA id <WQXSgQAOgW7o@statler.isode.com>; Sun, 30 Apr 2017 13:03:14 +0100
From: Alexey Melnikov <alexey.melnikov@isode.com>
To: calsify@ietf.org
Message-ID: <5905D298.6060204@isode.com>
Date: Sun, 30 Apr 2017 13:03:36 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0
MIME-Version: 1.0
Content-Type: text/plain; charset="windows-1252"
Content-transfer-encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/calsify/KMWcoHu9UJQwCdLeaM5Pl8olqu4>
Subject: [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 12:04:24 -0000

Hi,
The document is in a good shape, but I have a few minor comments/questions:

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