Re: [MMUSIC] AD review: draft-ietf-mmusic-sdp-cs-16

"Miguel A. Garcia" <Miguel.A.Garcia@ericsson.com> Mon, 14 January 2013 09:20 UTC

Return-Path: <miguel.a.garcia@ericsson.com>
X-Original-To: mmusic@ietfa.amsl.com
Delivered-To: mmusic@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 96FD721F8607 for <mmusic@ietfa.amsl.com>; Mon, 14 Jan 2013 01:20:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.249
X-Spam-Level:
X-Spam-Status: No, score=-6.249 tagged_above=-999 required=5 tests=[AWL=-0.000, BAYES_00=-2.599, HELO_EQ_SE=0.35, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ILaPIo2aqwnv for <mmusic@ietfa.amsl.com>; Mon, 14 Jan 2013 01:20:35 -0800 (PST)
Received: from mailgw1.ericsson.se (mailgw1.ericsson.se [193.180.251.45]) by ietfa.amsl.com (Postfix) with ESMTP id DE87521F855D for <mmusic@ietf.org>; Mon, 14 Jan 2013 01:20:33 -0800 (PST)
X-AuditID: c1b4fb2d-b7f316d0000028db-27-50f3cde0cce8
Received: from esessmw0256.eemea.ericsson.se (Unknown_Domain [153.88.253.124]) by mailgw1.ericsson.se (Symantec Mail Security) with SMTP id 3C.51.10459.0EDC3F05; Mon, 14 Jan 2013 10:20:32 +0100 (CET)
Received: from [159.107.26.63] (153.88.115.8) by esessmw0256.eemea.ericsson.se (153.88.115.97) with Microsoft SMTP Server id 8.3.279.1; Mon, 14 Jan 2013 10:20:32 +0100
Message-ID: <50F3CDDF.5070601@ericsson.com>
Date: Mon, 14 Jan 2013 10:20:31 +0100
From: "Miguel A. Garcia" <Miguel.A.Garcia@ericsson.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130107 Thunderbird/17.0.2
MIME-Version: 1.0
To: Gonzalo Camarillo <Gonzalo.Camarillo@ericsson.com>
References: <50EFFC93.70406@ericsson.com> <50F3CBC6.90901@ericsson.com> <50F3CDAC.8090702@ericsson.com>
In-Reply-To: <50F3CDAC.8090702@ericsson.com>
Content-Type: text/plain; charset="ISO-8859-1"; format=flowed
Content-Transfer-Encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmpmluLIzCtJLcpLzFFi42KZGfG3RvfB2c8BBl0vtC2mLn/MYnHu010W ByaPJUt+MnncvXWJKYApissmJTUnsyy1SN8ugSvj1YrprAWL1Cue/9vN1MB4Wq6LkYNDQsBE YsvcvC5GTiBTTOLCvfVsXYxcHEICJxklNvcsZoJwVjNK9P58xA5SxSugLbFj1kU2kGYWAVWJ rRcqQcJsAuYSrRs3gpWICkRJvL/axAxRLihxcuYTFhBbRMBM4v2/VUwgNrNAgMTan9PBbGEB G4n161eA1QsJpEvMbW5nA7E5BXQkzs/byw5RbytxYc51FghbXmL72zlQ9ZoSk28uZZ7AKDgL ybpZSFpmIWlZwMi8ipE9NzEzJ73ccBMjMCAPbvmtu4Px1DmRQ4zSHCxK4rxhrhcCgK5ILEnN Tk0tSC2KLyrNSS0+xMjEwSnVwDjRJGbvxcUlP2zm2E3U7ZiV0HOufopKbeSthrRr0ybUfc4M 2aB/w+111/UbU44dnJ224e2Hn0vN5r6XZpq5wviyof0HI60Jv25+S9n31vzXsrIJk/iiczKu HC3aEHm/+LfPyjixsAdcS/If/vHX7L4RWG5qGiI/W6F39585UquKpfLOOdzLNV2qxFKckWio xVxUnAgAR2nmPxYCAAA=
Cc: mmusic <mmusic@ietf.org>
Subject: Re: [MMUSIC] AD review: draft-ietf-mmusic-sdp-cs-16
X-BeenThere: mmusic@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mmusic>, <mailto:mmusic-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/mmusic>
List-Post: <mailto:mmusic@ietf.org>
List-Help: <mailto:mmusic-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mmusic>, <mailto:mmusic-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 14 Jan 2013 09:20:37 -0000

I have just submitted version -17.

/Miguel

On 14/01/2013 10:19, Gonzalo Camarillo wrote:
> Hi Miguel Angel,
>
> thanks for your quick response. I am OK with all your proposals below.
> When you implement the changes, please post a new revision of the draft.
>
> Thanks,
>
> Gonzalo
>
> On 14/01/2013 11:11 AM, Miguel A. Garcia wrote:
>> Hi Gonzalo:
>>
>> Here are some inline comments. I am about to post version -17 addressing
>> your comments:
>>
>>
>> On 11/01/2013 12:50, Gonzalo Camarillo wrote:
>>> Hi,
>>>
>>> I just got a publication request for the following draft:
>>>
>>> http://tools.ietf.org/html/draft-ietf-mmusic-sdp-cs-16
>>>
>>> Please, find below my AD review.
>>>
>>> Cheers,
>>>
>>> Gonzalo
>>>
>>>
>>> draft-ietf-mmusic-sdp-cs-16:
>>>
>>> Page 10: remove the indentation of the following paragraph. Indented
>>> paragraphs are typically used to provide clarifications while this one
>>> contains normative language:
>>>
>>>         Note that <addrtype> and/or <connection-address> MUST NOT be
>>>         omitted when unknown since this would violate basic syntax of SDP
>>>         [RFC4566].  In such cases, they MUST be set to a "-".
>>
>> Ok.
>>
>>>
>>> Section 5.2.3.1 provides a "format" for the cs-correlation attribute.
>>> However, its ABNF formal syntax is in Section 5.7. So, why does Section
>>> 5.2.3.1 also contain a syntax for the attribute... and what type of
>>> syntax is that?
>>
>> The sort-of-syntax in Section 5.2.3.1 is simply an introduction to the
>> three types of correlation mechanisms that can be used. So, the syntax
>> is the way to introduce to Sections 5.2.3.2, 5.2.3.3, and 5.2.3.4.
>>
>> But I understand your point. I suggest removing the syntax from there
>> and including a pointer to Section 5.7, where the formal syntax is
>> described.
>>
>>>
>>> Page 14: the text says:
>>>
>>>         To mitigate this problem implementations
>>>         should consider only some of the rightmost digits from the E.164
>>>
>>> Can we provide a more concrete advice (or at least some discussion) on
>>> the number of digits to consider?
>>
>> This is a hard problem even to provide guidance, because the PSTN E.164
>> numbers have a variable length. So, if you want to do it right, you
>> should become aware of the number plan in the network you are camping.
>> But this is outside the scope of the specification.
>>
>> We can add a note to refer to ITU-T E.164 specification in order for
>> implementations to consider what the right number of digits is.
>>
>>>
>>> Page 15: we define how to encode a binary value in hexadecimal format
>>> and represent it as a two-digit ASCII value. Can we point to an existing
>>> specification that defines how to do that instead of specifying it
>>> inline?
>>>
>>
>> Yeap, and I found that there is an RFC specifying it (there is always an
>> RFC specifying what you want :-). It is RFC 4648 and we are referring to
>> the base16 (or "hex") encoding. We will change the text accordingly.
>>
>>
>>> Page 17: the text says:
>>>
>>>         Implementations are advised to select a number of DTMF digits that
>>>         provide enough assurance that the call is related, but on the
>>>         other hand do not prolong the bearer setup time unnecessarily.
>>>
>>> Can we provide a more concrete recommendation?
>>
>> Ok, I guess we are talking of 5 to 10 digits.
>>
>>>
>>> Page 17: Section 5.2.3.4 specifies what to do when the correlation
>>> fails. However, that is described in general in Section 5.3.3. Why don't
>>> we move all the statements about how to decide whether the sessions are
>>> correlated to Section 5.3.3? Now, the following statement contradicts
>>> what Section 5.3.3 says, for example:
>>>
>>>      the passive side
>>>      SHOULD treat the circuit-switched bearer as not correlated to the
>>>      ongoing session.
>>
>> ok.
>>
>>>
>>> The indented text right after the statement above talks about
>>> suppressing alerting. Such a supression is applicable to any session,
>>> not only those that are correlated using DTMF tones. So, we could also
>>> move that discussion to Section 5.3.3.
>>
>> ok too.
>>
>>>
>>> Section 5.5: rephrase the following sentence (possibly splitting it into
>>> more than one sentence) so that it is clearer:
>>>
>>>         Note that this may result in the recipient of the initial Offer in
>>>         rejecting the Offer if the recipient of the Offer is not aware of
>>>         its own E.164 number, and thus concluding that it will not be
>>>         possible to establish a circuit-switched bearer since neither
>>>         party is aware of their E.164 number.
>>
>> Done
>>
>>
>>
>> Thanks,
>>
>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> mmusic mailing list
>>> mmusic@ietf.org
>>> https://www.ietf.org/mailman/listinfo/mmusic
>>>
>>
>

-- 
Miguel A. Garcia
+34-91-339-3608
Ericsson Spain