Re: [Gen-art] Gen-ART LC review for draft-ietf-calext-extensions-03

"Romascanu, Dan (Dan)" <dromasca@avaya.com> Tue, 21 June 2016 13:08 UTC

Return-Path: <dromasca@avaya.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4E5F212D590 for <gen-art@ietfa.amsl.com>; Tue, 21 Jun 2016 06:08:24 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.346
X-Spam-Level:
X-Spam-Status: No, score=-6.346 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, HTTPS_HTTP_MISMATCH=1.989, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.426, T_KAM_HTML_FONT_INVALID=0.01] autolearn=ham autolearn_force=no
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 tMsS3mfIZCoZ for <gen-art@ietfa.amsl.com>; Tue, 21 Jun 2016 06:08:22 -0700 (PDT)
Received: from co300216-co-outbound.net.avaya.com (co300216-co-outbound.net.avaya.com [198.152.13.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id ED4A712D0B2 for <gen-art@ietf.org>; Tue, 21 Jun 2016 06:08:21 -0700 (PDT)
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A2HTAAAaO2lX/yYyC4ddGgEBAQGCUiEtVn0GjSStTYE4QiGFdgKBNDgUAQEBAQEBAQNiJ4JAOQYHLwEBAQEBAQEiAg8vEgEBGQEBAQEDEhtMEAIBCA0EAwEBAQsdBzIUCQgBAQQBDQUIGogOAQ2mB5snAQEBAQEBAQEBAQEBAQEBAQEBAQEBHIYohEyEQh4WCIJugi8Fk0iFMQGGB4oNToQFgyGFRo94HjaCCByBTG4BAQGJRgF+AQEB
X-IPAS-Result: A2HTAAAaO2lX/yYyC4ddGgEBAQGCUiEtVn0GjSStTYE4QiGFdgKBNDgUAQEBAQEBAQNiJ4JAOQYHLwEBAQEBAQEiAg8vEgEBGQEBAQEDEhtMEAIBCA0EAwEBAQsdBzIUCQgBAQQBDQUIGogOAQ2mB5snAQEBAQEBAQEBAQEBAQEBAQEBAQEBHIYohEyEQh4WCIJugi8Fk0iFMQGGB4oNToQFgyGFRo94HjaCCByBTG4BAQGJRgF+AQEB
X-IronPort-AV: E=Sophos;i="5.26,504,1459828800"; d="scan'208,217";a="180155878"
Received: from unknown (HELO p-us1-erheast-smtpauth.us1.avaya.com) ([135.11.50.38]) by co300216-co-outbound.net.avaya.com with ESMTP; 21 Jun 2016 09:08:20 -0400
X-OutboundMail_SMTP: 1
Received: from unknown (HELO AZ-FFEXHC03.global.avaya.com) ([135.64.58.13]) by p-us1-erheast-out.us1.avaya.com with ESMTP/TLS/AES256-SHA; 21 Jun 2016 09:08:19 -0400
Received: from AZ-FFEXMB04.global.avaya.com ([fe80::6db7:b0af:8480:c126]) by AZ-FFEXHC03.global.avaya.com ([135.64.58.13]) with mapi id 14.03.0174.001; Tue, 21 Jun 2016 09:08:18 -0400
From: "Romascanu, Dan (Dan)" <dromasca@avaya.com>
To: Alexey Melnikov <alexey.melnikov@isode.com>, General Area Review Team <gen-art@ietf.org>
Thread-Topic: [Gen-art] Gen-ART LC review for draft-ietf-calext-extensions-03
Thread-Index: AdHLssZ6Ixxrt8UVTTO5Re1nCGvhDAAK6SaAAAgoPhA=
Date: Tue, 21 Jun 2016 13:08:17 +0000
Message-ID: <9904FB1B0159DA42B0B887B7FA8119CA75217EA8@AZ-FFEXMB04.global.avaya.com>
References: <9904FB1B0159DA42B0B887B7FA8119CA75217DAB@AZ-FFEXMB04.global.avaya.com> <19a6ce1b-956a-d74d-fa23-a595bb914af6@isode.com>
In-Reply-To: <19a6ce1b-956a-d74d-fa23-a595bb914af6@isode.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [135.64.58.48]
Content-Type: multipart/alternative; boundary="_000_9904FB1B0159DA42B0B887B7FA8119CA75217EA8AZFFEXMB04globa_"
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/i1Z-gxmMKBsw5pLSK87wjhNgiFg>
Cc: "draft-ietf-calext-extensions.all@tools.ietf.org" <draft-ietf-calext-extensions.all@tools.ietf.org>
Subject: Re: [Gen-art] Gen-ART LC review for draft-ietf-calext-extensions-03
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 21 Jun 2016 13:08:24 -0000

Hi Alexey - thanks for the quick response. I found answers to issues 2-3 in RFC 5545 indeed.

Issue 1 would still deserve clarification, although I do not consider it a show-stopper.

Regards,

Dan


From: Alexey Melnikov [mailto:alexey.melnikov@isode.com]
Sent: Tuesday, June 21, 2016 4:01 PM
To: Romascanu, Dan (Dan); General Area Review Team
Cc: draft-ietf-calext-extensions.all@tools.ietf.org
Subject: Re: [Gen-art] Gen-ART LC review for draft-ietf-calext-extensions-03


Hi Dan,

Thank you for your review.

On 21/06/2016 12:48, Romascanu, Dan (Dan) wrote:

I am the assigned Gen-ART reviewer for this draft. The General Area Review Team (Gen-ART) reviews all IETF documents being processed by the IESG for the IETF Chair.  Please treat these comments just like any other last call comments.



For more information, please see the FAQ at



< https://trac.tools.ietf.org/area/gen/trac/wiki/GenArtfaq <https://urldefense.proofpoint.com/v2/url?u=http-3A__wiki.tools.ietf.org_area_gen_trac_wiki_GenArtfaq&d=CwICAg&c=BFpWQw8bsuKpl1SgiZH64Q&r=I4dzGxR31OcNXCJfQzvlsiLQfucBXRucPvdrphpBsFA&m=azY-8O9zKU_qyQsIatC9huyRk-QxiYF2J3tY-yt8SWE&s=Lejj6aBY4lSWw3Rk7wGB75sQItp6gprPZgz3nBBp4-c&e=%20> >.



Document: draft-ietf-calext-extensions-03

Reviewer: Dan Romascanu

Review Date: 6/21/16

IETF LC End Date: 6/22/16

IESG Telechat date: 7/7/16



Summary: A clear document, almost ready from the Gen-ART point of view. I recommend to clarify the two minor issues below before approval.



Major issues:



Minor issues:



1.       Should not this document be marked as 'Updates RFC 5545'? The document recommends that all properties not defined in 5545 always include a "VALUE" parameter if the type is other than "TEXT" It also modifies some existing properties and defines new properties and elements that bring into the standard extension elements added by vendors into their specification.
This is a good question. Cyrus?


2.       I had a hard time understanding some of the details in 5.7 (REFRESH-INTERVAL Property). These may be however due to my lack of familiarity with the style of iCal definitions and notations - so only explanations may be sufficient. First I do not understand what is covered under Property Parameters: what means 'IANA and non-standard property parameters'
These are defined in RFC 5545. There is some text and ABNF covering these. Basically these property parameters are for extensibility.


3.       that can be defined here - and why here? Maybe an example would help. Second in the format definition: dur-value ... consisting of a positive duration of time - should not units be specified here as well? Last, I did not understand VALUE=DURATION:P1W - maybe it's just me missing the notation
I think both of these are defined in RFC 5545.

Best Regards,
Alexey