Re: Review of draft-yu-enumservice-sms-smpp WG ITEM?
"Richard Shockey" <[email protected]>
| Newsgroups | gmane.ietf.enum |
|---|---|
| Message-ID | <147a01c8a6fe$e4def4a0$ae9cdde0$@us> |
I'm going to assume silence as consent ..... > Subject: Re: [Enum] Review of draft-yu-enumservice-sms-smpp > > So I'm assuming for purposes of argument that there is no objection to > making this a WG document. > > > -----Original Message----- > > From: [email protected] [mailto:[email protected]] On > Behalf > > Of Alexander Mayrhofer > > Sent: Wednesday, April 23, 2008 11:40 AM > > To: [email protected]; [email protected] > > Subject: [Enum] Review of draft-yu-enumservice-sms-smpp > > > > > > Hello Yu, > > > > i spent some time on reviewing your sms:smpp draft - i have a > couple > > of > > comments (i probably missed some details, but given the early stage > of > > the draft, i guess we'll go through more reviewing anyway..) > > > > I like the idea, i understand that it makes sense. However, i don't > > think this is neccessarily limited to private use... For example, > > we're > > running our own SMPP server for the number range +4359966, and i > don't > > see why i wouldn't publish a corresponding ENUM record even in User > > ENUM > > ... (we're limiting access on the SMPP layer, of course). > > > > There are some minor things, but also some major content issues, > i've > > listed them in the order i noticed them... > > > > > > - Title: I'd recommend moving the "RFC 4355" part to the block on > top > > of > > the draft, so that it says "Updates: RFC 4355 (if approved)". I'd > also > > change the title into "IANA registration of the 'smpp' URI scheme > and > > the 'smpp' subtype for the 'sms' Enumservice" (or the other way > round, > > mentioning the Enumservice first) > > > > - The draft does not have proper page breaks. > > > > - Maybe you could merge "Conventions" and "Abbreviations" into a > > "Terminology" section, and put it in front of the Introdcution, > which > > would save you expanding all the terms in the intro. > > > > - There should be an informative reference to X.25 > > > > - The ABNF reference is outdated in the text (2234), but ok in the > > references (RFC 5234) > > > > - Section 5 ("use of..") is not just about the URI scheme, also > about > > the Enumservice. Maybe just rename that to "Use Cases" or "Use Case > > Examples". Disclaimer: I didn't go thorugh the use cases themselves > > because i'm not that deep into SMS delivery :-) > > > > - I'm missing a description of the dereference process of an "smpp" > > URI. > > For example: What is the exact process of determining the final (IP > > level) destination (and port) from an "hostpart": > > > > - Does it make use of NAPTR lookups, or SRV lookups? > > - What is the default port, if not defined? > > - How can one specify a port? in the "hostpart"? > > > > The "hostpart" that you say is "imported from RFC 3261" is not > > specified > > in 3261... it occurs once in the text, but is never specified... > > > > In addition, the "telephone-subscriber" that you import into the > > "user" > > part already allows parameters (namely, all "tel" URI parameters, > so > > you > > do essentially allow _two_ sets of parameters, like > > > > smpp:+4359966;cic=+1-6789;[email protected];parameterXY=bla > > > > Does that make sense, and is this intended? If it is intended, what > > are > > the semantics of the "user" parameters... I'm unsure whether this > is > > actually allowed by the URI ABNF itself... > > > > - Would it be possible to make note of the "Application Class" > subtype > > in the Enumservice Registration itself? i think it would be a > "Common > > Application" Enumservice as in 4.2.4 of the Enumservice guide > draft.. > > > > - The column (":") is not part of the URI scheme name. Please > remove > > it > > from the URI scheme part of the registration template. > > > > - I'm not happy with the last sentence of the "security > > considerations" > > section of the Enumservice template (limiting access to the DNS). I > > think that might also trigger a lot of headwind from the DNS guys. > > It's > > actually an implementation veriant in certain scenarios, but i > don't > > see > > why that would need to be included in the registration template. > > > > - The URI scheme needs improvement. As mentioned above, i can't > find > > the > > "hostpart" definition, the "telephone-subscriber" import adds > > parameter > > space with unclear semantics, and i'm missing clear dereferencing > > instructions. > > > > - If you define parameters for the SMPP URI, you will probably need > to > > define a registry for them (remember what happened to the tel: URI > - > > they needed to add a registry in a seperate document > > http://www.ietf.org/internet-drafts/draft-ietf-iptel-tel-reg- > 05.txt) > > > > - References: I think a couple of references can be moved to > > informative > > status. > > > > hope that helps! > > > > cheers > > > > Alex > > _______________________________________________ > > enum mailing list > > [email protected] > > https://www.ietf.org/mailman/listinfo/enum > > _______________________________________________ > enum mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/enum