[RAI] RAI-ART review of draft-ietf-sipping-sip-offeranswer-13

Ben Campbell <ben@nostrum.com> Fri, 21 May 2010 19:24 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: rai@core3.amsl.com
Delivered-To: rai@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 23BB03A67B6 for <rai@core3.amsl.com>; Fri, 21 May 2010 12:24:02 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.907
X-Spam-Level:
X-Spam-Status: No, score=-0.907 tagged_above=-999 required=5 tests=[AWL=-0.907, BAYES_50=0.001, SPF_PASS=-0.001]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id oMONFaE6NVqD for <rai@core3.amsl.com>; Fri, 21 May 2010 12:24:00 -0700 (PDT)
Received: from nostrum.com (nostrum-pt.tunnel.tserv2.fmt.ipv6.he.net [IPv6:2001:470:1f03:267::2]) by core3.amsl.com (Postfix) with ESMTP id 8F3483A6977 for <rai@ietf.org>; Fri, 21 May 2010 08:58:22 -0700 (PDT)
Received: from dn3-174.estacado.net (vicuna-alt.estacado.net [75.53.54.121]) (authenticated bits=0) by nostrum.com (8.14.3/8.14.3) with ESMTP id o4LFvqLt024675 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Fri, 21 May 2010 10:57:53 -0500 (CDT) (envelope-from ben@nostrum.com)
From: Ben Campbell <ben@nostrum.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Date: Fri, 21 May 2010 10:57:53 -0500
Message-Id: <59DEC6E7-96E1-47DB-99C6-414CF29F1FCA@nostrum.com>
To: shinji.okumura@softfront.jp, Paul Kyzivat <pkyzivat@cisco.com>
Mime-Version: 1.0 (Apple Message framework v1078)
X-Mailer: Apple Mail (2.1078)
Received-SPF: pass (nostrum.com: 75.53.54.121 is authenticated by a trusted mechanism)
Cc: rai@ietf.org
Subject: [RAI] RAI-ART review of draft-ietf-sipping-sip-offeranswer-13
X-BeenThere: rai@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "Real-time Applications and Infrastructure \(RAI\)" <rai.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/rai>, <mailto:rai-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/rai>
List-Post: <mailto:rai@ietf.org>
List-Help: <mailto:rai-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rai>, <mailto:rai-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 21 May 2010 19:24:02 -0000

[Looks like my first attempt got caught in moderation. Sorry for the repeat]

I am the assigned RAI-ART reviewer for draft-ietf-sipping-sip-offeranswer-13.

For background on RAI-ART, please see the FAQ at
<http://www.softarmor.com/rai/art/FAQ.html>.

Please resolve these comments along with any other comments you may receive.

Summary:

This draft claims an intended status of "informational". I believe this needs to be standards track, or a BCP. For that purpose, I think it's on the right track, but I have some other minor comments and questions that I think should be addressed first.

And BTW, I think this is a great document to have, as it is correct in pointing out our current offer/answer rules are scattered all over the place.

Major Comments:


-- The draft is listed as informational, and goes to great lengths to pretend to make no normative changes. I am not convinced. There are a number of places where it has the practical effect of  updating or adding normative constraints to existing proposed standards. I think it would be best to make this standards or BCP track and drop the pretense of being non-normative.


Minor Comments:

-- section 1.1: 

The need for normative language in a draft that states it is not making any normative changes is telling.

-- section 2.2, 5th paragraph, last sentence: "The 488 response is another proposed solution, UAS may respond with a 488 response and then UAC should send again a PRACK request without an offer."

What do you mean by "proposed solution"? Is this solution allowed, recommended, etc?

-- 2.2, Paragraph 8 (in NOTE):

Please elaborate on "401 response cannot be avoided". Perhaps something to the effect of "For example, the UAS may need to send an authentication challenge in a 401 response."

--2.2, paragraph 9:

Please elaborate on "...a UA should take care not to cause an infinite offer/answer loop."

-- 2.2, Table 2, (**)

Please elaborate -- how can a UA know the mind of the receiver?

-- 3.1.1

Are you certain there are no normative changes here? Can all the assertions be supported unambiguously by existing specs? 

-- UAC Behavior, item 1: "... not a real answer...

Does that make it a fake answer? I don't think "real" is the property you are looking for. Perhaps " not a final answer" or "preliminary answer" would be. Ore descriptive?

-- 3.1.1, paragraph before figure 1: "Therefore, a UAS should send a SDP answer reliably (if possible) before it starts sending media."

This  really sounds like either normative or BCP material.

-- 3.1.2, last paragraph, last sentence: "The UAC should take care to avoid this situation when it sends the INVITE request without SDP."

How? Are you suggesting that you shouldn't send an empty invite unless you can accept any answer without user intteraction?

-- section 4.1, paragraph after figure 4:

Can you elaborate on "should not send...at this point"? For example, should not send the reINVITE until...

Comment repeats for most of the following scenarios.

-- 5.1, first paragraph: "...without regard for what the other party in the call may have indicated previously."

This is not universally true. For example, the sender may have access to the recipient's presence information, which it generally shouldn't ignore.

5.2.3, first paragraph: "The media lines that are accepted should typically be those with types and formats the UAS would have included if it were the offerer"

I'm not sure how to apply that advice to make any useful decision. For example, if I might not have offered video, should I reject a media stream in someone else's offer?




Nits and Editorial Comments:

-- Please proofread for the following recurring errors: comma-spliced run-on sentences, missing articles, and incorrect selection of "a/an" vs "the".

-- s/"can not"/"cannot" throughout.

-- 3.1.1, figure 1:

The discussion talked about endpoints that do not support 100rel, but the example uses it.

-- 3.1.2, 2nd paragraph (Note): 

Please reference the figure number when you first start talking about messages from that figure.

-- 3.3, last paragraph, last sentence: "Otherwise, some 3pcc flows will break."

Expand 3pcc on first mention. Also, please provide a reference.

-- 3.4, first paragraph: "...newer specifications..."

Please specify which specs you are talking about.

-- 4, 3rd paragraph:

What do you mean by "...rules are guaranteed"?

No need for quotes around "exceptional".

I'm not sure what you mean by " carefully restricted in the RFCs" Are you suggesting a change in existing or future RFCs?

-- 4.2, paragraph after figure 11: "At best it..."

What is the antecedent for "it"?

-- 5.1, general:

I find it curious to ascribe wants and interests to the UA. This leads to some awkward constructions when discussing how a UA may not know what it wants.