Re: [salud] Christer's review of draft-ietf-salud-alert-info-urns-09

"DRAGE, Keith (Keith)" <keith.drage@alcatel-lucent.com> Tue, 21 January 2014 14:13 UTC

Return-Path: <keith.drage@alcatel-lucent.com>
X-Original-To: salud@ietfa.amsl.com
Delivered-To: salud@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 31FB81A00D2 for <salud@ietfa.amsl.com>; Tue, 21 Jan 2014 06:13:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.918
X-Spam-Level:
X-Spam-Status: No, score=-5.918 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_FONT_FACE_BAD=0.981, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id XMLw6scCq7ha for <salud@ietfa.amsl.com>; Tue, 21 Jan 2014 06:13:46 -0800 (PST)
Received: from hoemail2.alcatel.com (hoemail2.alcatel.com [192.160.6.149]) by ietfa.amsl.com (Postfix) with ESMTP id DF1541A00D3 for <salud@ietf.org>; Tue, 21 Jan 2014 06:13:45 -0800 (PST)
Received: from fr712usmtp2.zeu.alcatel-lucent.com (h135-239-2-42.lucent.com [135.239.2.42]) by hoemail2.alcatel.com (8.13.8/IER-o) with ESMTP id s0LEDeIU018574 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=FAIL); Tue, 21 Jan 2014 08:13:42 -0600 (CST)
Received: from FR711WXCHHUB02.zeu.alcatel-lucent.com (fr711wxchhub02.zeu.alcatel-lucent.com [135.239.2.112]) by fr712usmtp2.zeu.alcatel-lucent.com (GMO) with ESMTP id s0LEDdkl000611 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Tue, 21 Jan 2014 15:13:39 +0100
Received: from FR712WXCHMBA11.zeu.alcatel-lucent.com ([169.254.7.26]) by FR711WXCHHUB02.zeu.alcatel-lucent.com ([135.239.2.112]) with mapi id 14.02.0247.003; Tue, 21 Jan 2014 15:13:39 +0100
From: "DRAGE, Keith (Keith)" <keith.drage@alcatel-lucent.com>
To: Christer Holmberg <christer.holmberg@ericsson.com>, "laura.liess.dt@googlemail.com" <laura.liess.dt@googlemail.com>
Thread-Topic: [salud] Christer's review of draft-ietf-salud-alert-info-urns-09
Thread-Index: Ac8LoI2GwVZiq6LbSqq5Xri0pXR59AATzaeXABbZ38ABrQwL2wCr/kcgADfc3gAAA05iWgAAlX1w
Date: Tue, 21 Jan 2014 14:13:39 +0000
Message-ID: <949EF20990823C4C85C18D59AA11AD8B11F58E@FR712WXCHMBA11.zeu.alcatel-lucent.com>
References: <7594FB04B1934943A5C02806D1A2204B1C5EF0E7@ESESSMB209.ericsson.se> <201401072131.s07LVQZs2347719@shell01.TheWorld.com> <7594FB04B1934943A5C02806D1A2204B1C5F00D5@ESESSMB209.ericsson.se> <201401162107.s0GL7AKI2944531@shell01.TheWorld.com> <7594FB04B1934943A5C02806D1A2204B1D108F92@ESESSMB209.ericsson.se>, <CACWXZj033ssRvpJPQ=V8LACYUW9SpySbf+Pi86XR6eu5YWmdcw@mail.gmail.com> <7594FB04B1934943A5C02806D1A2204B1D11179F@ESESSMB209.ericsson.se>
In-Reply-To: <7594FB04B1934943A5C02806D1A2204B1D11179F@ESESSMB209.ericsson.se>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [135.239.27.39]
Content-Type: multipart/alternative; boundary="_000_949EF20990823C4C85C18D59AA11AD8B11F58EFR712WXCHMBA11zeu_"
MIME-Version: 1.0
Cc: "salud@ietf.org" <salud@ietf.org>
Subject: Re: [salud] Christer's review of draft-ietf-salud-alert-info-urns-09
X-BeenThere: salud@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Sip ALerting for User Devices working group discussion list <salud.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/salud>, <mailto:salud-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/salud/>
List-Post: <mailto:salud@ietf.org>
List-Help: <mailto:salud-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/salud>, <mailto:salud-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 21 Jan 2014 14:13:49 -0000

The agreement we made is that we should not use the tables, but also that the main text should accurately specify the requirements for which methods are allowed to include header fields or not, preferably in a generic way that accommodates new methods. So text that states "all requests", "all requests initiating a new dialog" and so on.

Even on an update, it may be preferable to specify it this way.

regards

Keith

________________________________
From: salud [mailto:salud-bounces@ietf.org] On Behalf Of Christer Holmberg
Sent: 21 January 2014 11:29
To: laura.liess.dt@googlemail.com
Cc: salud@ietf.org
Subject: Re: [salud] Christer's review of draft-ietf-salud-alert-info-urns-09

Hi Laura,

SIPCORE some time ago decided not to use/maintain the header field tables anymore, so that is why I didn't include it in my change suggestion.

But, if you want to also change the table, I won’t object 😊

Regards,

Christer

Sent from Windows Mail

From: laura.liess.dt@googlemail.com<mailto:laura.liess.dt@googlemail.com>
Sent: ‎Tuesday‎, ‎January‎ ‎21‎, ‎2014 ‎12‎:‎54‎ ‎PM
To: Hans-Christer Holmberg<mailto:christer.holmberg@ericsson.com>
Cc: Dale R. Worley<mailto:worley@ariadne.com>, salud@ietf.org<mailto:salud@ietf.org>

Christer, Dale,

IMO, the  draft updates the RFC 3261 because it allows the usage of the Alert-Info header field in all provisional responses 1xx except the 100 response. This is described in the first two paragraphs of Section 1.2.  This paragraphs are OK in this section.
The third  paragraph is usefull (it explains what the draft does) but, as Christer remarked,  it does not describe the update of the RFC3261 and so it does not belong to this section.  I would move this paragraph at the end of Section 1.1. What do you think?

Concerning the structure if the Section 1.2,   I will put the text from section 1.2 into a new main Section 2 and structure it as Christer proposes in http://www.ietf.org/mail-archive/web/salud/current/msg00454.html.
But  I think there are two places where we should update the 3261 text:
1) Section 20.4 as proposed by Christer and

2) Table 2: Summary of header fields, A--O on page 161, the " 180" in the row


 Alert-Info             180     ar    -   -   -   o   -   -

   should be changed in "1xx (excepting 100)"

so we probably need two subsections "New text repalcing the text.........of RFC 3261" .
Opinions?

Thank you
Laura



2014/1/20 Christer Holmberg <christer.holmberg@ericsson.com<mailto:christer.holmberg@ericsson.com>>
Hi,

>> BTW, the following sentence is unclear to me:
>>
>>    "In practice, this specification extends Alert-Info in that it will
>>    cause the use of a new class of URIs and the use of multiple URIs."
>>
>> What does it mean?
>
> The idea is that, while 3261 syntactically and semantically *allows* non-HTTP URIs in Alert-Info, nobody ever used non-HTTP URIs in Alert-Info, and nobody implemented an interpretation of those URIs.
> Thus, even though we are not changing what is officially permitted, implementers will have to change their code because the new *practice* will be an extension of the old practice.  (And both the new practice and the old practice are a small subset of what is permitted by 3261.)

Well, then you should use different wording. Because, the draft does not extend Alert-Info, it simply describes the usage of a new URN in the header field.


>>>> Q9:
>>>> -----
>>>>
>>>> Section 4 defines the ABNF for the URN, but [there] is no text on
>>>> how to "map" it into the Alert-Info header field ABNF:
>>>>
>>>> For example, I assume that the URN is encoded using the opaque-part
>>>> format of the absoluteURI, and that the scheme value is "urn". I
>>>> think it would be good to indicate that.
>>>
>>> Actually, we're depending on the fact (unstated in RFC 3261) that
>>> any "absolute" URI (per RFC 3986) is allowed as a header field value
>>> in Alert-Info, and that the URNs we are defining conform to the
>>> absolute URI syntax.  If the syntax for <absoluteURI> in RFC 3261
>>> didn't allow all absolute URIs, we'd have to amend RFC 3261.
>>
>> There are two different "structures" for absolute URI, and the URNs
>> only fit into the opaque-part structure.
>>
>> In addition, as the URI requires a scheme value, I think we should
>> explicitly say what it is.
>
> I guess my point is that the 3261 ABNF already allows all alert URNs to appear in Alert-Info, and there is no need to specify exactly how alert URNs are compatible with the 3261 ABNF.  Anyone who cares exactly how this draft is directly compatible with the ABNF can see that by looking at the ABNF.
>
> Have there been other situations where this sort of syntax explication has been provided?

I don't know - I am just commenting on how I think the draft could from my perspective be improved :)

Regards,

Christer
_______________________________________________
salud mailing list
salud@ietf.org<mailto:salud@ietf.org>
https://www.ietf.org/mailman/listinfo/salud