Hi Alexey, Thanks for your comments. Also thanks to Qin for your help. I will split the media-type registration into two separated subsections as well as address all your other comments and get back to you soon. For some of your comments, I have some further questions inline.
Thanks, Jensen On Wed, Dec 15, 2021 at 9:23 AM Qin Wu <[email protected]> wrote: > Thanks Alex for valuable review and suggestion, I will ask leading author > of draft-alto-request-routing-alto to implement the changes you suggested. > > > > -Qin > > *发件人:* Alexey Melnikov [mailto:[email protected]] > *发送时间:* 2021年12月15日 4:22 > *收件人:* Qin Wu <[email protected]>; [email protected] > *抄送:* [email protected]; Francesca Palombini <[email protected]>; > [email protected] > *主题:* Re: [media-types] application/alto-* Media Types Registration > Request > > > > Hi Qin, > > Overall this looks fine to me. A few comments below: > > > > On 03/12/2021 11:17, Qin Wu wrote: > > Hello, > > Here is the registration request of application/alto-* Media Types defined > in section 7.1 of draft-ietf-alto-cdni-request-routing-alto: > > https://datatracker.ietf.org/doc/draft-ietf-alto-cdni-request-routing-alto/ > > > https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html > > > > I’ve included the details below. > > > > Note that this document has already entered into IESG review phase. > Francesca Palombini kindly reminds us and the authors of this draft to > > send the media type registrations to the media-type mailing list for > review before the document moves forward. Thanks Francesca. > > > > -Qin (on behalf of chairs) > 7.1. > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#section-7.1>application/alto-* > Media Types > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#name-application-alto-media-type> > > This document updates the IANA Media Types Registry by registering two > additional ALTO media types, listed in Table 1 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#TableMediaTypes> > . > > *Table 1 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#table-1>: > Additional > ALTO Media Types. > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#name-additional-alto-media-types>* > > *Type* > > *Subtype* > > *Specification* > > application > > alto-cdni+json > > Section 3 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#cdnifci> > of > RFCthis > > application > > alto-cdnifilter+json > > Section 5 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#filteredcdnifci> > of > RFCthis > > Type name: > > application > > Subtype name: > > This document registers multiple subtypes, as listed in Table 1 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#TableMediaTypes> > . > > Existing media type registration template doesn’t allow registering > multiple media types using the same template. So you will need to create > multiple registrations. > > > > Having said that, a few minor comments below: > > Required parameters: > > n/a > > Optional parameters: > > n/a > > Encoding considerations: > > Encoding considerations are identical to those specified for the > "application/json" media type. See [RFC8259 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#RFC8259> > ]. > > Security considerations: > > Security considerations related to the generation and consumption of ALTO > Protocol messages are discussed in Section 15 of [RFC7285 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#RFC7285> > ]. > > Interoperability considerations: > > This document specifies formats of conforming messages and the > interpretation thereof. > > This section is for known interoperability concerns. If you don’t know of > any, putting N/A here is probably the best. > Thanks for the suggestion. It has been done in version -18. > Published specification: > > This document is the specification for these media types; see Table 1 > <https://www.ietf.org/archive/id/draft-ietf-alto-cdni-request-routing-alto-17.html#TableMediaTypes> > for > the section documenting each media type. > > Applications that use this media type: > > ALTO servers and ALTO clients either stand alone or are embedded within > other applications. > > Please add a reference to a document where ALTO is defined. Registration > templates can be read by people who are not necessarily familiar with this > technology. > Got it. Will add the reference to [RFC7285]. > > > Also when you split the registration template into 2 it would be good to > have a sentence here explaining how the two formats differ. > Thanks for the suggestion. Could you kindly give us some further examples about what should be explained? Do we need to explain the different cases where the two subtypes should be used, or just explain the difference between the two registration forms? > Additional information: > > Magic number(s): > > n/a > > File extension(s): > > This document uses the mime type to refer to protocol messages and thus > does not require a file extension. > > > > I don’t think I agree with this statement, but not having a file extension > is Ok. So just put “N/A” here. > OK. We will just put n/a here. > > Macintosh file type code(s): > > n/a > > Person & email address to contact for further information: > > See Authors' Addresses section. > > Intended usage: > > COMMON > > Restrictions on usage: > > n/a > > Author: > > See Authors' Addresses section. > > Change controller: > > Internet Engineering Task Force (mailto:[email protected] <[email protected]>). > > > > > > _______________________________________________ > > media-types mailing list > > [email protected] > > https://www.ietf.org/mailman/listinfo/media-types > > _______________________________________________ > alto mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/alto >
_______________________________________________ alto mailing list [email protected] https://www.ietf.org/mailman/listinfo/alto
