Re: [calsify] Barry Leiba's Discuss on draft-ietf-calext-eventpub-extensions-17: (with DISCUSS)

Michael Douglass <mikeadouglass@gmail.com> Wed, 13 January 2021 20:54 UTC

Return-Path: <mikeadouglass@gmail.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 8327C3A1377; Wed, 13 Jan 2021 12:54:00 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.359
X-Spam-Level:
X-Spam-Status: No, score=-2.359 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, NICE_REPLY_A=-0.262, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.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 bne9cqLTnVTo; Wed, 13 Jan 2021 12:53:58 -0800 (PST)
Received: from mail-qv1-xf32.google.com (mail-qv1-xf32.google.com [IPv6:2607:f8b0:4864:20::f32]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8BF453A1378; Wed, 13 Jan 2021 12:53:58 -0800 (PST)
Received: by mail-qv1-xf32.google.com with SMTP id p5so1443121qvs.7; Wed, 13 Jan 2021 12:53:58 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:subject:to:cc:references:message-id:date:user-agent :mime-version:in-reply-to:content-language; bh=d9m2qqhV1//Dze8yRmUDotbCKEn2EIxdlJg+T5v2UUY=; b=h5ZQBn1exGiOikMrLMz0ERuoVN5JMl7eax9Z4Ijb8KU8Nc4KnplWXsUocJ4egt2XK7 jc103u8i0g6qbhMEf/qJhj+MR2Xgk4cFUtLLXsvIhDmORcdrNgjfESgt47f/4FnoanLc 8dPayZ331+zOGVUYV9hYKVyN4t3RFBo30BtFD4ULKoDPBdqLQGU4SVTyQJ67n7mDSJVZ V2cdC8dHoezCK7b5sSn7yY3ygKfsvoVZrdSuOh+8WwiqPqN/KHx0Y9bLzr83gyLV2Q3k OHUtnJdfDnfU3XJgTvd9WINWo1DK+MFtVMB9sH7fuM2tN74lL84R1xZINAhH1DN0dc4s yYYg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:subject:to:cc:references:message-id:date :user-agent:mime-version:in-reply-to:content-language; bh=d9m2qqhV1//Dze8yRmUDotbCKEn2EIxdlJg+T5v2UUY=; b=TnJNvD1cLn5QtetNgXmdsFzBZ7v8tnmw3eXaH8ENSpAWqwPaJX3pcpgQJ4cyWFIlLE vYhvHQbgl2WPtoA/jL1MrQOBiM2i2hCXzjsnnObohccw3HLYWDS+mkzTkEHSywtJNHG6 Q+U/OUYN+M6SvEaT6qxVuDzCEehyOUtbpN00uF6bndXV9OuCC54bzKDc9SQbO4+mj2M3 +wrAE6PgSOzNveoW9Xo/Qcpdfg2Gejhg2gDdbWI691jLjkzxg7h4Gs4K9a5Tk3x8Vaa8 Z+PhgNucObj0eV3M6c94TjEsG4gObyACGHLJQt/9DWUF4mwt28DZRbEImYGBgFQADxWL SF4g==
X-Gm-Message-State: AOAM530k3j9kx7Kyp/A3cUfuLFho1myjvKc18DD1ZYjrymQbTn7QEoUU OeTeIsgTS+n/+nRsiU+a33Kj4+ij4qabZA==
X-Google-Smtp-Source: ABdhPJziwRsmyztmyQ2C5cRw6HXFz7tPLcgSYYl0vlkPvUoU14TZGhvlfKNclFXe1LRW6f6iWYEPJQ==
X-Received: by 2002:a05:6214:80d:: with SMTP id df13mr4325699qvb.10.1610571237312; Wed, 13 Jan 2021 12:53:57 -0800 (PST)
Received: from [192.168.1.151] (cpe-74-70-70-237.nycap.res.rr.com. [74.70.70.237]) by smtp.googlemail.com with ESMTPSA id i17sm1725719qtg.77.2021.01.13.12.53.56 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 13 Jan 2021 12:53:56 -0800 (PST)
From: Michael Douglass <mikeadouglass@gmail.com>
To: Barry Leiba <barryleiba@computer.org>, The IESG <iesg@ietf.org>
Cc: draft-ietf-calext-eventpub-extensions@ietf.org, calext-chairs@ietf.org, calsify@ietf.org
References: <160979733556.18901.5326788661603760785@ietfa.amsl.com>
Message-ID: <31e49aaa-f781-505d-e41a-8c1fcd56ed2d@gmail.com>
Date: Wed, 13 Jan 2021 15:53:55 -0500
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.16; rv:78.0) Gecko/20100101 Thunderbird/78.6.0
MIME-Version: 1.0
In-Reply-To: <160979733556.18901.5326788661603760785@ietfa.amsl.com>
Content-Type: multipart/alternative; boundary="------------3AA00849D2D7D819B1A1A596"
Content-Language: en-US
Archived-At: <https://mailarchive.ietf.org/arch/msg/calsify/4sTchar6ksGNuR-PoNVn9jZgQLs>
Subject: Re: [calsify] Barry Leiba's Discuss on draft-ietf-calext-eventpub-extensions-17: (with DISCUSS)
X-BeenThere: calsify@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 13 Jan 2021 20:54:01 -0000

Thanks for this,

Comments below - just posted a version 18

On 1/4/21 16:55, Barry Leiba via Datatracker wrote:
> Barry Leiba has entered the following ballot position for
> draft-ietf-calext-eventpub-extensions-17: Discuss
>
> When responding, please keep the subject line intact and reply to all
> email addresses included in the To and CC lines. (Feel free to cut this
> introductory paragraph, however.)
>
>
> Please refer tohttps://www.ietf.org/iesg/statement/discuss-criteria.html
> for more information about IESG DISCUSS and COMMENT positions.
>
>
> The document, along with other ballot positions, can be found here:
> https://datatracker.ietf.org/doc/draft-ietf-calext-eventpub-extensions/
>
>
>
> ----------------------------------------------------------------------
> DISCUSS:
> ----------------------------------------------------------------------
>
> Thanks very much for the changes in Section 9.1, and I think we're now at the
> best place we can reasonable be here.  Well done.
>
> Thanks also for the changes to clarify the ABNF.  They're mostly good, and we
> should be able to clean these last bits up pretty easily:
>
> — Section 6.6 —
>
> The new ABNF doesn’t correctly specify what the old was trying to say.  I think
> this is correct and concise, but please check it over:
>
> NEW
>
> sdataprop    = "STRUCTURED-DATA" sdataparam ":"
>                       sdataval CRLF
>
>        sdataparam     = ; all parameter elements may appear in any order,
>                         ; and the order is not significant.
>                         (sdataparamtext / sdataparambin / sdataparamuri)
>                         *(";" other-param)
>
>        sdataparamtext = ";VALUE=TEXT"
>                         ";" fmttypeparam
>                         ";" schemaparam
>
>        sdataparambin  = ";VALUE=BINARY"
>                         ";ENCODING=BASE64"
>                         ";" fmttypeparam
>                         ";" schemaparam
>
>        sdataparamuri  = ";VALUE=URI"
>                         [";" fmttypeparam]
>                         [";" schemaparam]
>
>        sdataval       = ( binary / text /uri )
>                         ; value MUST match value type
>
> END
Thanks - that seems much better.
>
> — Section 7.1 —
>
>        participantc = "BEGIN" ":" "PARTICIPANT" CRLF
>                       *( partprop / locationc / resourcec )
>                       "END" ":" "PARTICIPANT" CRLF
>
> This allows multiple instances of partprop (or none), which is not what you
> mean.  The “*” isn’t right.  Also, do you really mean to have locationc and
> resourcec here?  Those are blocks that are peers of participantc within eventc,
> todoc, journalc, and freebusyc… are they also meant to be nested within
> participantc?  If so, it would be good to have an example or two that shows
> that.  In any case, that bit of ABNF still needs some work.

Replaced with

participantc = "BEGIN" ":" "PARTICIPANT" CRLF
                partprop *locationc *resourcec
                "END" ":" "PARTICIPANT" CRLF

location and resources are allowed as properties, locationc and 
resourcec allow the eventpub style components. I'll add an example

>
>                       (calendaraddress)
>                       (created)
>                       (description)
>                       (dtstamp)
>                       (geo)
>                       (last-mod)
>                       (priority)
>                       (seq)
>                       (status)
>                       (summary)
>                       (url)
>
> All of these are meant to be optional, so they should be in square brackets,
> rather than in parentheses.  The same is true in Sections 7.2 and 7.3.
All fixed.
>
>
>
>
>
> _______________________________________________
> calsify mailing list
> calsify@ietf.org
> https://www.ietf.org/mailman/listinfo/calsify