Re: [sipcore] Comments to draft-sparks-sipcore-refer-clarifications-05.txt

Ivo Sedlacek <ivo.sedlacek@ericsson.com> Wed, 12 November 2014 15:25 UTC

Return-Path: <ivo.sedlacek@ericsson.com>
X-Original-To: sipcore@ietfa.amsl.com
Delivered-To: sipcore@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0EB421A1AC6 for <sipcore@ietfa.amsl.com>; Wed, 12 Nov 2014 07:25:44 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] 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 LSpWN6YJzIuB for <sipcore@ietfa.amsl.com>; Wed, 12 Nov 2014 07:25:40 -0800 (PST)
Received: from sesbmg23.ericsson.net (sesbmg23.ericsson.net [193.180.251.37]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C440E1A1B18 for <sipcore@ietf.org>; Wed, 12 Nov 2014 07:25:38 -0800 (PST)
X-AuditID: c1b4fb25-f791c6d00000617b-8b-54637bf06015
Received: from ESESSHC001.ericsson.se (Unknown_Domain [153.88.253.124]) by sesbmg23.ericsson.net (Symantec Mail Security) with SMTP id 2A.90.24955.0FB73645; Wed, 12 Nov 2014 16:25:36 +0100 (CET)
Received: from ESESSMB301.ericsson.se ([169.254.1.4]) by ESESSHC001.ericsson.se ([153.88.183.21]) with mapi id 14.03.0174.001; Wed, 12 Nov 2014 16:25:36 +0100
From: Ivo Sedlacek <ivo.sedlacek@ericsson.com>
To: Robert Sparks <rjsparks@nostrum.com>, "sipcore@ietf.org" <sipcore@ietf.org>
Thread-Topic: Comments to draft-sparks-sipcore-refer-clarifications-05.txt
Thread-Index: AQHP+f7mGM+M7RbfTpKpZBAgcDmyYJxdIHtQ
Date: Wed, 12 Nov 2014 15:25:35 +0000
Message-ID: <39B5E4D390E9BD4890E2B310790061011278D7C3@ESESSMB301.ericsson.se>
References: <39B5E4D390E9BD4890E2B31079006101127852C3@ESESSMB301.ericsson.se> <545BD7B4.1090509@nostrum.com>
In-Reply-To: <545BD7B4.1090509@nostrum.com>
Accept-Language: en-US, cs-CZ
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.16]
Content-Type: multipart/alternative; boundary="_000_39B5E4D390E9BD4890E2B310790061011278D7C3ESESSMB301erics_"
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrJLMWRmVeSWpSXmKPExsUyM+Jvje6H6uQQg8Xv9SyuzWlks/j6YxOb A5PHkiU/mTxm7XzCEsAUxWWTkpqTWZZapG+XwJWxadYZxoJnS5gqHty8y9rAOGEuUxcjJ4eE gInEm/6nULaYxIV769m6GLk4hASOMEo86PjPDuEsYpRYeWgmM0gVm4CexMQtR1hBbBGBQImF k5awgNjCAp4Sh18cYIeIe0nsP7WDBcI2kjiyayEjiM0ioCrxcuUCNhCbV8BX4smBb2BxIYE8 iY4LEPM5BbQlVkx9AjaHUUBW4uqfXrAaZgFxiVtP5kNdKiCxZM95ZghbVOLl43+sELaixM6z 7UBxDqD6fIkPv5QgVglKnJz5hGUCo8gsJJNmIVTNQlIFEdaUWL9LH6JaUWJK90N2CFtDonXO XHZk8QWM7KsYRYtTi5Ny042M9VKLMpOLi/Pz9PJSSzYxAuPq4JbfqjsYL79xPMQowMGoxMNr cDYpRIg1say4MvcQozQHi5I478Jz84KFBNITS1KzU1MLUovii0pzUosPMTJxcEo1MHqf/ta9 YcEHEVHWtw6fbttv+7W5WF3df5mEjLpxwA2R7UVJi3561cvWLJ4iz7uQJf3LNGXXVzIL9HIj pouzrJny8f+nbfcPZXTmFzXa+7967jwtc/5q9aXOmwr+fszdZ/D+mF9arc5GY63nAjIitz5s Yi74yPWt/VHClKJZjTum+2jdf5v0W1aJpTgj0VCLuag4EQBrTeh2jAIAAA==
Archived-At: http://mailarchive.ietf.org/arch/msg/sipcore/jL3FY-AXh_AFuJ8Xq8bNn_osBH4
Subject: Re: [sipcore] Comments to draft-sparks-sipcore-refer-clarifications-05.txt
X-BeenThere: sipcore@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: SIP Core Working Group <sipcore.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sipcore>, <mailto:sipcore-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/sipcore/>
List-Post: <mailto:sipcore@ietf.org>
List-Help: <mailto:sipcore-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sipcore>, <mailto:sipcore-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 12 Nov 2014 15:25:44 -0000

Hello Robert,

please see below.



COMMENT 1:

------------------

Abstract states "An accepted SIP REFER method creates an implicit subscription using the SIP-Specific Event Notification Framework."



This is only partly true since creation of implicit subscription do not necessarily happen when draft-sparks-sipcore-refer-explicit-subscription is used or when RFC4488 is used (assuming application specific logic ensures that the implicit subscription is not created).

------------------



PROPOSED SOLUTION 1:

------------------

State: "An accepted SIP REFER method >>can<< create an implicit subscription using the SIP-Specific Event Notification Framework."

------------------
Hrmm - this is effectively the same feedback you sent earlier on sentences like this in the document (but thanks for the cleaner proposed text).

My response then (and now) is that accounting for the one case where you can ask (but not be guaranteed)
to suppress the subscription doesn't help with the clarity of this document.


[Ivo]
The document needs to be correct as well as clear.

Omitting the one case results to incorrect document.

To me, your statement ".... doesn't help with the clarity of this document." is comparable with a judge stating to an accused: "There is one possibility that you are innocent. However, describing it in judgement doesn't help with the clarity of the judgement. So, I declare you guilty instead.".


I'd like to hear from others in the group.
Do you want to find a way to call out this case, or leave the text as is?
I would prefer to leave the text alone.
If we change it, I don't think this proposal is the right one yet. "Can" understates the issue significantly.



COMMENT 2:

------------------

Section 2 states "An accepted SIP REFER method creates an implicit subscription using the SIP-Specific Event Notification Framework."



This is only partly true since creation of implicit subscription do not necessarily happen when draft-sparks-sipcore-refer-explicit-subscription is used or when RFC4488 is used (assuming application specific logic ensures that the implicit subscription is not created).

------------------



PROPOSED SOLUTION 2:

------------------

State: "An accepted SIP REFER method >>can<< create an implicit subscription using the SIP-Specific Event Notification Framework."

------------------
Same as 1:


[Ivo]

Same as 1:





COMMENT 3:

------------------

Section 3 states "

   Section 4.5.1 of [RFC6665] makes GRUU [RFC5627] mandatory to

   implement and use as the local target in the subscription created by

   the REFER request.

"



In RFC6665, usage of GRUU is mandatory for notifier (">>Notifiers<< MUST implement the Globally Routable User Agent URI (GRUU) extension defined in [RFC5627], and MUST use a GRUU as their local target."), while the statement above could be understood to refer to both subscriber and notifier.

------------------
I think Adam tried to address this in an earlier response. The new 6665 clarifications draft may address it more strongly.

[Ivo]
"The new 6665 clarifications draft may address it more strongly." seems to suggest that the comment above is actually correct, i.e. RFC665 does not contain such statement, and new 6665 clarifications draft  is necessary to make it.




PROPOSED SOLUTION 3:

------------------

State: "

   Section 4.5.1 of [RFC6665] makes GRUU [RFC5627] mandatory to

   implement and use as the local target >>by notifiers<< in the subscription created by

   the REFER request.

"

------------------



COMMENT 4:

------------------

Section 3 states "

   A UA that will accept a REFER request needs to include a GRUU in the

   Contact header field of all INVITE requests.  This ensures that out-

   of-dialog REFER requests corresponding to any resulting INVITE

   dialogs arrive at this UA.  Future extensions (such as

   [I-D.sparks-sipcore-refer-explicit-subscription]) might relax this

   requirement by defining a REFER request that cannot create an

   implicit subscription.

"



The UA might be globally reachable without indicating GRUU - this is common today with network conference servers. Requiring GRUU is unnecessary.

------------------
Again, see Adam's 6665 clarifications draft (and please take any argument with it to the list under that name as a subject, not this draft).

[Ivo]
Well, this draft states this requirement and therefore it can be questioned.

Yes, the same comments can be raised against the "6665 clarifications draft" but that does not make this document correct.




PROPOSED SOLUTION 4:

------------------

Make the statement general to state that the UA must provide URI which is a globally reachable in the Contact of INVITE requests.

------------------



COMMENT 5:

------------------

Section 4, 1st paragraph and 2nd paragraph provide overlapping information.



In 1st paragraph, there is a statement that if remote UA provided a GRUU as its local target then a user agent sending a REFER request (towards that target) that could result in an  implicit subscription within a dialog is prohibited.



In 2nd paragraph, the same is stated (in positive way) regardless whether the remote UA provided a GRUU as its local target or not.

------------------



PROPOSED SOLUTION 5:

------------------

Remove the 1st paragraph of section 4.

------------------
I can almost see the overlap you are talking about, but I don't see where that introduces any weakness into the text.
Removing the paragraph is not the right thing to do. The first paragraph describes the prohibition, the second talks
about what's left that you can do. The document is less accurate if you leave either point out.

[Ivo]
The text is inconsistent.

The reader will wonder why:
- the 1st paragraph prohibits dialog reuse if the remote UE provided GRUU; and
- the 2nd paragraph requires out-of-dialog REFER regardless of whether the remote UE provided GRUU.





COMMENT 6:

------------------

Section 3 states: "

   A user agent constructing any REFER that can result in an implicit

   subscription MUST populate its Contact header field with a GRUU.

"



In RFC665, there is no such requirement for subscriber. Why do we state such new requirement?

------------------
See Adam's earlier response and the 6665 clarifications draft.

[Ivo] Again, this document refers to RFC6665 and RFC6665 does not state it.



PROPOSED SOLUTION 6:

------------------

Clarify the need for this requirement or remove it.

------------------



Kind regards



Ivo Sedlacek