[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Simple] MSRP URLs
Hi:
A few comments about section 5 of the MSRP draft:
http://www.ietf.org/internet-drafts/draft-ietf-simple-message-sessions-08.txt
1) The draft should describe the type of resources that are identified
by an MSRP URL. So I would suggest to add an introductory paragraph at
the beginning of section 5 that says something on the following lines:
The MSRP and MSRPS URL schemes are used to designate a session of
instant messages at a particular host computer.
Question: is this definition also applicable to relays?
2) RFC 2396 is under revision, nowadays
draft-fielding-uri-rfc2396bis-06.txt. I am not sure if you still want to
refer to RFC 2396 or the revision of it. I guess this depends of which
draft makes it first to the IESG.
Among other things, in case you want to refer to rfc2396bis, this draft
has removed the definition of "hostport". Now it is just "host [: port]"
3) The current definition of an MSRP URL includes a "resource" part,
when actually it is a "path" or "session-id", because a "resource" is
the the whole MSRP URL, including the URI scheme and the hostname.
So I would suggest to replace the current definition:
MSRP_urls = msrp-scheme "://" [userinfo "@"] hostport ["/"
resource] ";" transport
resource = 1*unreserved
with this other one:
MSRP_urls = msrp-scheme "://" [userinfo "@"] hostport ["/"
path] ";" transport
resource = 1*unreserved
4) According to rfc2396bis, "unrerved" is defined as:
unreserved = ALPHA / DIGIT / "-" / "." / "_" / "~"
On purpose "unreserved" does not include "sub-delims", which is defined as:
sub-delims = "!" / "$" / "&" / "'" / "(" / ")"
/ "*" / "+" / "," / ";" / "="
I guess it is OK for MSRP that the path is formed of unreserved
characters, not including sub-delims. But I would like to point this out
in case someone feels the path needs to include sub-delims.
5) I find a contradiction that we, on one hand, allow to have a
"userinfo" part in the MSRP URL including a reference to RFC 2396, and
on the other hand we restrict the "userinfo" definition in RFC 23964 to
unreserved characters.
RFC 2396 (and 2396bis) allow to have escaped characters in the userinfo.
However, MSRP does not, but still refers to 2396 for the definition of
userinfo.
So, if the "userinfo" defined in the MSRP URL is different from (or a
subset of) the "userinfo" defined in RFC 2396, then I would suggest to
change the name (perhaps "unescaped-userinfo"), and avoid the reference
to RFC 2396.
6) Section 5.1 reads on the second bullet point:
2. If the hostpart contains an eplicit IP address, and/or port,
these are compared numerically. Otherwise, hostpart is compared
as a case insensitive character string.
I was wondering in the how to treat zeros in either IPv4 or IPv6 (IPv6
uses this quite oftenly). The missleading word is "numerically", since I
am not sure if the "::" should be normalized by adding trailing and
leading zeros or not.
Is it the intention that the following two URLs are equivalent and MSRP
endpoints should detect it?
msrp://[1080:0:0:0:8:800:200C:417A]/aodisfa;tcp
msrp://[1080::8:800:200C:417A]/aodisfa;tcp
A clarification should be written.
Regards,
Miguel
--
Miguel A. Garcia tel:+358-50-4804586
Nokia Research Center Helsinki, Finland
_______________________________________________
Simple mailing list
Simple at ietf.org
https://www1.ietf.org/mailman/listinfo/simple