[Gen-art] review of draft-ietf-scim-api-16.txt

Francis Dupont <Francis.Dupont@fdupont.fr> Wed, 22 April 2015 15:07 UTC

Return-Path: <Francis.Dupont@fdupont.fr>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8D6BE1A1A30 for <gen-art@ietfa.amsl.com>; Wed, 22 Apr 2015 08:07:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.337
X-Spam-Level:
X-Spam-Status: No, score=0.337 tagged_above=-999 required=5 tests=[BAYES_40=-0.001, HELO_EQ_FR=0.35, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01] autolearn=no
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 QUyy6Hoekx7H for <gen-art@ietfa.amsl.com>; Wed, 22 Apr 2015 08:07:57 -0700 (PDT)
Received: from givry.fdupont.fr (givry.fdupont.fr [IPv6:2001:41d0:1:6d55:211:5bff:fe98:d51e]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D25C61A039F for <gen-art@ietf.org>; Wed, 22 Apr 2015 08:07:52 -0700 (PDT)
Received: from givry.fdupont.fr (localhost [127.0.0.1]) by givry.fdupont.fr (8.14.3/8.14.3) with ESMTP id t3MF67xd089983; Wed, 22 Apr 2015 17:06:07 +0200 (CEST) (envelope-from dupont@givry.fdupont.fr)
Message-Id: <201504221506.t3MF67xd089983@givry.fdupont.fr>
From: Francis Dupont <Francis.Dupont@fdupont.fr>
To: gen-art@ietf.org
Date: Wed, 22 Apr 2015 17:06:07 +0200
Sender: Francis.Dupont@fdupont.fr
Archived-At: <http://mailarchive.ietf.org/arch/msg/gen-art/b0I0Bvz6PuRbSddT_jtk94f43_Q>
Cc: draft-ietf-scim-api.all@tools.ietf.org
Subject: [Gen-art] review of draft-ietf-scim-api-16.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
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: <http://www.ietf.org/mail-archive/web/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, 22 Apr 2015 15:07:59 -0000

I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at

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

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-scim-api-16.txt
Reviewer: Francis Dupont
Review Date: 20150418
IETF LC End Date: 20150420
IESG Telechat date: unknown

Summary: Ready with nits

Major issues: None

Minor issues: None

Nits/editorial comments: many!
 - Abstract page 1: a standardized services:
  either 'a standardized service' or 'standardized services'

 - Abstract page 1: add at least a comma in:
   a common user
   schema and extension model and a service protocol
                             ^ e.g., here

 - ToC page 2:
   3.4.1.  Retrieving a known Resource
                        ^ Known

 - about keywords: IMHO it is far better, less ambigous and BTW
  compliant to avoid lower case keywords.

 - 1.1 page 4: some examples of (not very ambigous) lower case "may"s.

 - 3.2 page 6: ask the RFC Editor to check the page break is not
  as badly placed as in my paper copy (PATCH alone at last line).

 - 3.2 page 7 table 1 and a lot of other places: e.g. -> e.g.,

 - 3.5.2 page 30: long uri cut issue (there is no perfect solution:
  either cut it into two lines, or insert a line break. But you
  should be consistent in this choice).

 - 3.5.2 page 31: misplaced comma?
    a patch operation that sets a value's
   "primary" attribute to "true", SHALL cause the server to
                                ^

 - 3.5.2 page 31: no closing parenthesis:
    resource (subject to
             ^

 - 3.5.2.2 page 35: missing required SP:
    "path":"members[value eq\"2819c223...919d-413861904646\"]"
                            ^

 - 3.5.2.3 page 38: selction -> selection

 - 3.6 page 42: from my long list a debatable lower case "should not":
   the previously deleted resource should not fail

 - 3.9 page 60: why an upper case "OR" in:
   "attributes" OR
   "excludedAtributes"

 - 3.10 page 61: "A" in plurals?
   A Complex
   attributes' Sub-Attributes are referenced

 - 5 page 70: bad wording:
   To increase the likelihood that the input and comparison of unicode
   usernames and passwords will work in ways that make sense for typical
   users throughout the world there are special string preparation and
   comparison methods (PRECIS) that MUST be followed for usernames and
   passwords.

 - 7.2 page 73: spurious comma:
   As mentioned in ,Section
                   ^

 - 7.4 page 74: i.e. -> i.e., (the only one I found :-)

 - 9.2 page 78: strange ', .'s (missing parameter in a macro?)
   [OpenSearch]
              Clinton, D., "OpenSearch Protocol 1.1, Draft 5", .

   [Order-Operations]
              Wikipedia, "Order of Operations: Programming Languages", .

Regards

Francis.Dupont@fdupont.fr