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

[Christer Holmberg <[e-mail]>]

Hi,

I have now read the draft. In general I am ok with it. There is text which is not wrong, but which I think could be simplified, but I will not focus on that.

Some comments below.


Q1:
-----

Remove the references from the Abstract.


Q2:
-----

There are a number of editorial inconsistences that needs to be fixed.

E.g. "Alert-Info header field" vs "SIP Alert-Info header field" vs "Alert-Info"

E.g. "180 (Ringing)" vs "180 Ringing"

Etc.


Q3:
-----

As the draft updates RFC 3261, I would like to have a section named "Update to RFC 3261". Now there are only a few places in the introduction talking about the update.


Q4:
-----

I am not sure the text in section 1.2 belongs to the Introduction. It is basically the "Update to RFC 3261" section I mentioned in Q3.


Q5:
-----

I am not sure the text in section 1.3 belongs to the Introduction. We normally have a separate section.


Q6:
-----

Section 1.1 says:

"In [RFC3261] this is done by including a URI in the Alert-Info header field, that specifies the tone."

Should the text say that the header field includes a REFERENCE to a tone? Later in the paragraph, you DO talk about the referenced tone.


Q7:
----

Section 1.1 says:

"The URI is most commonly the HTTP URL to the audio file."

s/"the audio file"/"an audio file"


Q8:
-----

Section 1.1. says:

"It is not possible to provide semantic indications or names for rendering characteristics
                that signals the intent and allows the recipient UA to decide how to render the received
information in an appropriate way."

The sentence is very confusing. Why not simply say something like:

                "It is not possible to define semantics associated with a given tone."


Q9:
-----

Section 4 defines the ABNF for the URN, but the 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 sceme value is "urn". I think it would be good to indicate that.


Regards,

Christer