Re: [sip-http-events] Fwd: New Version Notification for draft-roach-sip-http-subscribe-03

"Elwell, John" <john.elwell@siemens-enterprise.com> Mon, 30 November 2009 15:43 UTC

Return-Path: <john.elwell@siemens-enterprise.com>
X-Original-To: sip-http-events@core3.amsl.com
Delivered-To: sip-http-events@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 5B9413A63EB for <sip-http-events@core3.amsl.com>; Mon, 30 Nov 2009 07:43:25 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[AWL=0.000, BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6mrqySTmK1Mw for <sip-http-events@core3.amsl.com>; Mon, 30 Nov 2009 07:43:24 -0800 (PST)
Received: from ms01.m0019.fra.mmp.de.bt.com (m0019.fra.mmp.de.bt.com [62.180.227.30]) by core3.amsl.com (Postfix) with ESMTP id 0EE1F3A67A5 for <sip-http-events@ietf.org>; Mon, 30 Nov 2009 07:43:23 -0800 (PST)
Received: from senmx12-mx ([62.134.46.10] [62.134.46.10]) by ms01.m0020.fra.mmp.de.bt.com with ESMTP id BT-MMP-158530; Mon, 30 Nov 2009 16:42:55 +0100
Received: from MCHP063A.global-ad.net (unknown [172.29.37.61]) by senmx12-mx (Server) with ESMTP id 066B423F01F6; Mon, 30 Nov 2009 16:42:55 +0100 (CET)
Received: from MCHP058A.global-ad.net ([172.29.37.55]) by MCHP063A.global-ad.net ([172.29.37.61]) with mapi; Mon, 30 Nov 2009 16:42:54 +0100
From: "Elwell, John" <john.elwell@siemens-enterprise.com>
To: Adam Roach <adam@nostrum.com>, SIP HTTP Subscription Package <sip-http-events@ietf.org>
Date: Mon, 30 Nov 2009 16:42:53 +0100
Thread-Topic: [sip-http-events] Fwd: New Version Notification for draft-roach-sip-http-subscribe-03
Thread-Index: AcpuFAiJBneQO6uASCmqDFmFFjkrggDmnR+Q
Message-ID: <A444A0F8084434499206E78C106220CA43A2E6C9@MCHP058A.global-ad.net>
References: <4B0D9DC3.8030202@nostrum.com>
In-Reply-To: <4B0D9DC3.8030202@nostrum.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Subject: Re: [sip-http-events] Fwd: New Version Notification for draft-roach-sip-http-subscribe-03
X-BeenThere: sip-http-events@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: SIP HTTP Events <sip-http-events.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/sip-http-events>, <mailto:sip-http-events-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/sip-http-events>
List-Post: <mailto:sip-http-events@ietf.org>
List-Help: <mailto:sip-http-events-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sip-http-events>, <mailto:sip-http-events-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 30 Nov 2009 15:43:25 -0000

Adam,

I reviewed this again and it looks almost ready to go. Just a few comments:

- In Abstract: "This document further proposes that the HTTP work necessary to make
   such a mechanism work be extensible to support protocols other than
   SIP for monitoring HTTP resources." 
The only further mention of this seems to be in section 3 "Handling for
   other URI schemes is out of scope for the current document, although
   we expect future specifications to define procedures for monitoring
   via other protocols."
To justify the wording in Abstract I would expect more than this. I would propose deletion of the words in Abstract.

- In section 1: "Such subscriptions do not carry the content associated with
   the resource -- the HTTP protocol is still used to transfer the
   contents of HTTP resources."
With the addition of the body= parameter, this isn't always true.

- In section 4..2: "this parameter
   indicates to the server that the client wishes to receive a message-
   body component in the message/http bodies sent in NOTIFY messages."
The terminology used here ("message body component", "bodies") seems to be inconsistent with terminology used in SIP in general and RFC 5621 in particular, i.e., a "message body" (singular) and "body parts". In fact this inconsistency in terminology occurs in other sections, which I haven't pulled out specifically. In the particular case of the sentence above, shouldn't it say something like "wishes to receive message/http body part(s) in NOTIFY messages"?

- In section 4.7 "In the case that the NOTIFIER has insufficient information to return
   any useful information about the underlying HTTP resource, it may
   return a body that is zero bytes long."
What motivated this? Would termination of the subscription be an alternative possibility?


Some nits:
- In section 3: "the the"
- In 4.4: "other can change" - change to "others can change "
- There is one instance each of "NOTIFIER" and "SUBSCRIBER" (in 4.7 and 4.9) - these should not be capitalized.
- In 4.8: "subscriber should" - changed to "the subscriber should"

John


> -----Original Message-----
> From: sip-http-events-bounces@ietf.org 
> [mailto:sip-http-events-bounces@ietf.org] On Behalf Of Adam Roach
> Sent: 25 November 2009 21:13
> To: SIP HTTP Subscription Package
> Subject: [sip-http-events] Fwd: New Version Notification for 
> draft-roach-sip-http-subscribe-03
> 
> I was able to add in all the relevant information (including 
> a security section) to cover the previously unfinished 
> portions of the SIP HTTP Subscription draft. This is the 
> version that I will be asking for final comments on in the 
> upcoming weeks:
> 
>   http://www.ietf.org/id/draft-roach-sip-http-subscribe-03.txt
> 
> Please provide any comments you may have on this version. Thanks.
> 
> /a
> 
> -------- Original Message -------- 
> Subject: 	New Version Notification for 
> draft-roach-sip-http-subscribe-03	
> Date: 	Wed, 25 Nov 2009 13:08:08 -0800 (PST)	
> From: 	IETF I-D Submission Tool 
> <idsubmission@ietf.org>; <mailto:idsubmission@ietf.org> 	
> To: 	adam@nostrum.com	
> 
> 
> A new version of I-D, draft-roach-sip-http-subscribe-03.txt 
> has been successfuly submitted by Adam Roach and posted to 
> the IETF repository.
> 
> Filename:	 draft-roach-sip-http-subscribe
> Revision:	 03
> Title:		 A SIP Event Package for Subscribing to 
> Changes to an HTTP Resource
> Creation_date:	 2009-11-25
> WG ID:		 Independent Submission
> Number_of_pages: 18
> 
> Abstract:
> The Session Initiation Protocol (SIP) is increasingly being used in
> systems that are tightly coupled with Hypertext Transport Protocol
> (HTTP) servers for a variety of reasons.  In many of these cases,
> applications can benefit from being able to discover, in near-real-
> time, when a specific HTTP resource is created, changed, or deleted.
> This document proposes a mechanism, based on the SIP events
> framework, for doing so.
> 
> This document further proposes that the HTTP work necessary to make
> such a mechanism work be extensible to support protocols other than
> SIP for monitoring HTTP resources.
>                                                               
>                     
> 
> 
> The IETF Secretariat.
> 
> 
> 
>