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

[Christer Holmberg <[e-mail]>]

Hi,

In my opinion, the normative update (no matter how is done), should not be in section 1, which is introduction.

It shall of course be mentioned there, but I still think there should be a dedicated “Update to RFC 3261” section.

Regards,

Christer

Sent from Windows Mail

From: [e-mail]<mailto:[e-mail]>
Sent: ‎Tuesday‎, ‎January‎ ‎21‎, ‎2014 ‎3‎:‎30‎ ‎PM
To: Hans-Christer Holmberg<mailto:[e-mail]>
Cc: [e-mail]<mailto:[e-mail]>

Hi Christer,

thank you. What about moving the third paragraph to subsection 1.1?  Could this work for you?

Laura


2014/1/21 Christer Holmberg <[e-mail]<mailto:[e-mail]>>
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: [e-mail]<mailto:[e-mail]>
Sent: ‎Tuesday‎, ‎January‎ ‎21‎, ‎2014 ‎12‎:‎54‎ ‎PM
To: Hans-Christer Holmberg<mailto:[e-mail]>
Cc: Dale R. Worley<mailto:[e-mail]>, [e-mail]<mailto:[e-mail]>

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 <[e-mail]<mailto:[e-mail]>>
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
[e-mail]<mailto:[e-mail]>
https://www.ietf.org/mailman/listinfo/salud


_______________________________________________
salud mailing list
[e-mail]<mailto:[e-mail]>
https://www.ietf.org/mailman/listinfo/salud