Re: [Gen-art] Genart last call review of draft-ietf-sipcore-sip-push-21

Alissa Cooper <alissa@cooperw.in> Wed, 09 January 2019 15:27 UTC

Return-Path: <alissa@cooperw.in>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 22EF8130DF6; Wed, 9 Jan 2019 07:27:00 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.801
X-Spam-Level:
X-Spam-Status: No, score=-0.801 tagged_above=-999 required=5 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=cooperw.in header.b=dIMTTdKo; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=t4sRmmy2
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 aeST5vDl41kP; Wed, 9 Jan 2019 07:26:58 -0800 (PST)
Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 6684D130DE9; Wed, 9 Jan 2019 07:26:58 -0800 (PST)
Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.west.internal (Postfix) with ESMTP id 400BD1559; Wed, 9 Jan 2019 10:26:56 -0500 (EST)
Received: from mailfrontend1 ([10.202.2.162]) by compute7.internal (MEProxy); Wed, 09 Jan 2019 10:26:56 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cooperw.in; h= content-type:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; s=fm2; bh=7 zwT9AaBqx40VdOU5/j0XFenhkHqblHe99+C5tTfOV8=; b=dIMTTdKoKX4dkuODX 4vjZwpJ8ZbsApxRAudhgY8m4xqg9lZgycY9r7tZ10EqPfrTdn2fpti+6dDYwpDOp mAR0qUPQ+aPR31qUYM/u649oG5ayewdqopLURjlTDRgvILZdv35FN+yAn6ZJH6fV oMwEdI777diCSX3wNJNg5qrHJvQJimDeJVoG9goWTOuhKZZtcP/Y5t0iW3MOReze P3fHj1L/FKpyeZPPm9nURPY5dmIM1ypQbAmtqpWKFzJU5sEi1ZU6aVeyGWvhdxBZ mkCPu0cMyuKpDATovMjTI8+6BJMxDGMuPsB6L6HQot1oARPvtLL0/FFgQwF8+Eq5 cFtsw==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=7zwT9AaBqx40VdOU5/j0XFenhkHqblHe99+C5tTfO V8=; b=t4sRmmy2P0/PZSk6DQVocjDcLUaBTSDWNCCndWmWTg6BZLEINAYNFG+8O wLB2OP9QElaMhXpiQF37m3jna/69vwrRCUeaRuNw1YHc5Oc4JtMtmdUx7RE497DC UDhA2WZrWqPtrMC3d23oAEVkKnCPDanNaU6gicbDctvb7JahTJ+7mzL2oeD5dtvS e9YRaJZzFnY0dp+jBzBAwdpSHldMqNX/pjaEx6I9jGiWBHzkqPqybbTKT/lT0S59 ojCVXo9TA95ZG4tzXX9g54mRS330HXoCFfuapMNpYr0dlTW6TUZnUGJt2vpHJVBN Cypkk+IYVf50/VTHXX9aHm2xqoamA==
X-ME-Sender: <xms:vxI2XM7dROeyaUiS-VItSr8joChkwBiWz0tazHMkXye-dY5uIDmUbA>
X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedtledrfedugdejjeculddtuddrgedtkedrtddtmd cutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfhuthen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurheptggguffhjgffgffkfhfvofesthhqmhdthhdtjeenucfhrhhomheptehlihhs shgrucevohhophgvrhcuoegrlhhishhsrgestghoohhpvghrfidrihhnqeenucffohhmrg hinheprghpphhlvgdrtghomhdpihgvthhfrdhorhhgpdgvgigrmhhplhgvrdgtohhmnecu kfhppedujeefrdefkedruddujedrkeegnecurfgrrhgrmhepmhgrihhlfhhrohhmpegrlh hishhsrgestghoohhpvghrfidrihhnnecuvehluhhsthgvrhfuihiivgeptd
X-ME-Proxy: <xmx:vxI2XFyPLJDQ497JxDmyOcX1ggrgsYgmzS6trtUwpQS_bTVIQmArQg> <xmx:vxI2XK0RSRAtVwCwrSZ40nl07dIhafrMa4FlsbW6FojywV_blaDVmg> <xmx:vxI2XI58y_oF_OIECLCUes--mf7CIal0CovpVqqw7Ye2wJcB3u4nnQ> <xmx:vxI2XBzLd-ILevYutei4Fz5QXwMPN3H0FMkAMG5WkmW4nG0kwazXtQ>
Received: from rtp-alcoop-nitro5.cisco.com (unknown [173.38.117.84]) by mail.messagingengine.com (Postfix) with ESMTPA id 01D98E435E; Wed, 9 Jan 2019 10:26:54 -0500 (EST)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\))
From: Alissa Cooper <alissa@cooperw.in>
In-Reply-To: <154523455825.2864.6694264562010996848@ietfa.amsl.com>
Date: Wed, 09 Jan 2019 10:26:53 -0500
Cc: IETF Gen-ART <gen-art@ietf.org>, sipcore@ietf.org, draft-ietf-sipcore-sip-push.all@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <7A1D6973-83FB-4F14-9296-3D6428DB13B0@cooperw.in>
References: <154523455825.2864.6694264562010996848@ietfa.amsl.com>
To: Stewart Bryant <stewart.bryant@gmail.com>
X-Mailer: Apple Mail (2.3445.9.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/SlrLm8pKzIdJw3e5h27zcT5NczQ>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-sipcore-sip-push-21
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 09 Jan 2019 15:27:00 -0000

Stewart, thanks for your review. Christer, thanks for addressing Stewart’s comments. I entered a No Objection ballot.

Alissa

> On Dec 19, 2018, at 10:49 AM, Stewart Bryant <stewart.bryant@gmail.com> wrote:
> 
> Reviewer: Stewart Bryant
> Review result: Ready with Nits
> 
> I am the assigned Gen-ART reviewer for this draft. The General Area
> Review Team (Gen-ART) reviews all IETF documents being processed
> by the IESG for the IETF Chair.  Please treat these comments just
> like any other last call comments.
> 
> For more information, please see the FAQ at
> 
> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
> 
> Document: draft-ietf-sipcore-sip-push-21
> Reviewer: Stewart Bryant
> Review Date: 2018-12-19
> IETF LC End Date: 2018-12-21
> IESG Telechat date: Not scheduled for a telechat
> 
> Summary: A well written document with some minor points that could use a little
> attention.
> 
> Major issues: None
> 
> Minor issues:
> 
> In Figure 1 the following is included:
> 
>     REGISTER sip:alice@example.com SIP/2.0
>     Via: SIP/2.0/TCP alicemobile.example.com:5060;branch=z9hG4bKnashds7
>     Max-Forwards: 70
>     To: Alice <sip:alice@example.com>
>     From: Alice <sip:alice@example.com>;tag=456248
>     Call-ID: 843817637684230@998sdasdh09
>     CSeq: 1826 REGISTER
>     Contact: <sip:alice@alicemobile.example.com;
>       pn-provider=acme;
>       pn-param=acme-param;
>       pn-prid=ZTY4ZDJlMzODE1NmUgKi0K>
>     Expires: 7200
>     Content-Length: 0
> 
> SB> However I don't at this stage of the text see the relationship between the
> packet flow digram and the text that follows.
> 
> =========
>   Contact: IESG (iesg@ietf.org)
> 
> SB> Is the whole IESG the most appropriate first point of contact?
> 
> =========
> 
> Nits/editorial comments:
> Presumably the references to RFC XXXX will be replaced by RFC <this RFC> but
> that does not seem to be noted in the text
> 
> ========
> 
>   As dicussed in [RFC4320] and [RFC4321], non-INVITE transactions must
> SB> Typo s/dicussed/discussed/
> 
> ========
> 
>   Example: pn-prid = 00fc13adff78512
> 
>   For more information about the APNs Topic and device token:
> 
> SB> Is the following part of the example? If so it could usefully be delimited
> as SB>  such, otherwise, I don't understand why it is not a normal document
> SB> reference.
> 
>   https://developer.apple.com/library/archive/documentation/NetworkingI
>   nternet/Conceptual/RemoteNotificationsPG/CommunicatingwithAPNs.html
> 
> SB> Similarly in the following section
> =========
> 
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art