Mercurial > hg > ietf

annotate draft-ietf-extra-processimip-07.txt @ 8:d4e5d6b079bc default tip

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
merge
author Henry S. Thompson <ht@inf.ed.ac.uk>
date Tue, 02 Jul 2024 13:51:47 +0100
parents 328798b2a7d1
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
7
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
1 Document: draft-ietf-extra-processimip-07
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
2 Intended RFC status: Proposed Standard
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
3 Review type: artart - Last Call review
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
4 Reviewer: Henry S. Thompson
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
5 Review Date: 2024-06-28
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
6 Result:
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
7
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
8 *Summary*
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
9
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
10 *Substantive points*
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
11
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
12 Section 4: "based on the iTIP method (see Section 1.4 of [RFC5546]) of
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
13 the message, calendar objects are created, updated or deleted..." --
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
14 but the introduction says "sometimes the enclosed iCalendar [RFC5545]
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
15 data does not include an iTIP method property".
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
16
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
17 I'm confused at this point: If there _is_, for example, an iTIP
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
18 "REQUEST" method specified, does the processcalendar script _replace_,
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
19 _modify_ or _supplement_ that method, or what?
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
20
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
21 Section 4: "contains calendar data in multiple MIME [RFC2045] parts"
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
22 -- This seems underspecified. It makes most sense for multiple
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
23 calendar parts all in a single multipart/alternative, but why
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
24 shouldn't multiple calendar parts with different semantics be allowed
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
25 (possibly nested at some depth) in a multipart/mixed?
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
26
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
27
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
28 *Minor points*
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
29
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
30 Section 3: I take it that by "capability identifier" is meant
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
31 "capability name" or "capability string" as used in RFC5228. Please
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
32 change the name of this section to one of these, and include a
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
33 reference to 5228.
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
34
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
35 Section 4: Similarly, in the opening sentence of this section a
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
36 reference to 5228 would be welcome.
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
37
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
38 Section 4: "If the calendar data is malformed in any way" -- this
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
39 hides a potential multitude of detail. I presume that in part this
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
40 means "malformed according to the media type specified for the
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
41 encapsulated calendar data", but also implies something along the
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
42 lines of "or if the specified media type is not supported". That is,
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
43 no sniffing or just scanning for strings that look like dates.
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
44
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
45
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
46
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
47 *Nits*
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
48
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
49 [iMIP] appears to be an RFC now: iCalendar Message-Based
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
50 Interoperability Protocol (iMIP) RFC 6047
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
51
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
52 ht
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
53 --
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
54
328798b2a7d1 partial
Henry S. Thompson <ht@inf.ed.ac.uk>
parents:
diff changeset
55 [1] https://datatracker.ietf.org/doc/html/draft-ietf-core-coap-pubsub-12