Review of draft-murchison-webdav-prefer-11

Stewart Bryant <stewart@g3ysx.org.uk> Wed, 21 December 2016 19:31 UTC

Return-Path: <stewart@g3ysx.org.uk>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 2EB3F129532; Wed, 21 Dec 2016 11:31:57 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Stewart Bryant <stewart@g3ysx.org.uk>
To: <gen-art@ietf.org>
Subject: Review of draft-murchison-webdav-prefer-11
X-Test-IDTracker: no
X-IETF-IDTracker: 6.40.2
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <148234871714.12720.9762267582738671683.idtracker@ietfa.amsl.com>
Date: Wed, 21 Dec 2016 11:31:57 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/xL3AFgKtV5OfFEl5t_QRTBAvIl0>
Cc: draft-murchison-webdav-prefer.all@ietf.org, ietf@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 21 Dec 2016 19:31:57 -0000

Reviewer: Stewart Bryant
Review result: Ready with Issues

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

<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-murchison-webdav-prefer-11
Reviewer: Stewart Bryant
Review Date: 21 Dec 2016
IETF LC End Date: 16 January 2017
IESG Telechat date: Unknown


Summary: Ready with issues

This is a well written document with some minor editorial issues that
need
to be looked at before it is sent to the RFC Editor.


Issues:

>From ID-nits:

  -- The draft header indicates that this document updates RFC7240,
but the
     abstract doesn't seem to mention this, which it should.

=======

SB> The following suggests an open issue, which needs to be
SB> closed, or if closed already, the issue warning needs to be
removed.

Open Issues

   o  Should we add any text regarding caching responses in Section
3?

========



3.1.  Successful State-Changing Requests


   representating the current state resource in the resulting 201
SB> representating - do you mean representing?

==========

3.2.  Unsuccessful Conditional State-Changing Requests

   Frequently, clients using a state-changing method such as those
   listed above will make them conditional by including either an If-
   Match or If-None-Match [RFC7232] header field in the request. 
This
   is done to prevent the client from accidentially overwriting a
SB> s/accidentially/accidentally./

=========


9.3.  URIs

SB> I think that this section needs a "remove on publishing
instruction"
SB> since I think you have given instructions to remove all the
SB> text that calls its entries.

<end>