[apps-discuss] Apps Area Review for draft-ietf-geopriv-held-measurements-06.txt

Eliot Lear <lear@cisco.com> Tue, 26 March 2013 14:29 UTC

Return-Path: <lear@cisco.com>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D0E3321F8BCF; Tue, 26 Mar 2013 07:29:36 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -109.812
X-Spam-Level:
X-Spam-Status: No, score=-109.812 tagged_above=-999 required=5 tests=[AWL=0.786, BAYES_00=-2.599, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-8, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id CF0TtUXqvLJz; Tue, 26 Mar 2013 07:29:35 -0700 (PDT)
Received: from ams-iport-4.cisco.com (ams-iport-4.cisco.com [144.254.224.147]) by ietfa.amsl.com (Postfix) with ESMTP id B450521F8BBC; Tue, 26 Mar 2013 07:29:34 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=10969; q=dns/txt; s=iport; t=1364308175; x=1365517775; h=message-id:date:from:mime-version:to:cc:subject; bh=lV1b6jV4N+OpQ6x7wESAVEkS6oc98eLazQUDmfXNjjQ=; b=eiM11IspriUj6gqaX/1bGr+TX5qdGthsbJlyB0sWWqhk7goUde9NYl0k OSWaeCYyq9eC08hYfcIv9hRRPkp5YM2054BviGO+4EiWcFa0yZkUGKb0I WRAx3jmgjRGPNct5qcFcOl6qR3jvaqB4I1pI0xOXuMS8J2GGL5B8bnj8E I=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: AgEFAHmwUVGQ/khR/2dsb2JhbABDhle9K4EFFoEqgkkEQBEBLw0WCwILAwIBAgFYAQcBAYgQDK5ygkCQBI8BEYI0gRMDlmeBH49ogws7gTAk
X-IronPort-AV: E=Sophos; i="4.84,911,1355097600"; d="scan'208,217"; a="12905311"
Received: from ams-core-1.cisco.com ([144.254.72.81]) by ams-iport-4.cisco.com with ESMTP; 26 Mar 2013 14:29:26 +0000
Received: from ams3-vpn-dhcp7552.cisco.com (ams3-vpn-dhcp7552.cisco.com [10.61.93.127]) by ams-core-1.cisco.com (8.14.5/8.14.5) with ESMTP id r2QETCQj019938 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Tue, 26 Mar 2013 14:29:19 GMT
Message-ID: <5151B0B5.2090407@cisco.com>
Date: Tue, 26 Mar 2013 15:29:09 +0100
From: Eliot Lear <lear@cisco.com>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20130307 Thunderbird/17.0.4
MIME-Version: 1.0
To: draft-ietf-geopriv-held-measurements.all@tools.ietf.org
X-Enigmail-Version: 1.5.1
Content-Type: multipart/alternative; boundary="------------080609080905010409090802"
Cc: 'IESG' <iesg@ietf.org>, "apps-discuss@ietf.org" <apps-discuss@ietf.org>
Subject: [apps-discuss] Apps Area Review for draft-ietf-geopriv-held-measurements-06.txt
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 26 Mar 2013 14:29:37 -0000

Gents,

I have been selected as the Applications Area Directorate reviewer for
this draft (for background on appsdir, please see ​
http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate ).


Please resolve these comments along with any other Last Call comments
you may receive. Please wait for direction from your document shepherd
or AD before posting a new version of the draft.

Document: draft-ietf-geopriv-held-measurements-06.txt
Title: Using Device-provided Location-Related Measurements in Location
Configuration Protocols
Reviewer: Eliot Lear
Review Date: 26 Mar 2013

Congratulations.  Really nice draft.    This draft describes a container
that can be used to request and report measurements relating to
location.  It is almost ready for publication.

I'd suggest that this draft receive additional review regarding its
privacy considerations section.  I will comment outside this review
regarding that issue, as this is an apps area review.  Also, I've not
done schema validation.

Major issues:

§ 4.3, page 10: I realize it'd be silly to write a six line RFC for
this, but the semantics of the HELD examples seem normative.  I think
it's appropriate for them to be so, but then you should make it more
explicit.  Same with your XMPP example. 

Minor issues:

§4.1, 2nd para on page 7.  While surely this is applicable for HELD,
there are protocols that have size considerations, and so your assertion
is a bit strong, and I would suggest that you scope it to be applicable
where XML can be carried.(*)

§4.4 Page 11, last paragraph.  It's not quite clear how the example
combines information in the LIS.  Maybe it's because I'm not steeped in
location lore, so perhaps you might point this out to me, if not others?


§8.3, Page 44: there has got to be an existing schema one can borrow IP
addresses from, no?  If not, is this worth splitting off?

Finally, see RFC 2026 for proper use of normative references.  You're
referencing 802.11v as an informative reference when in fact flightTime
depends on TOD and TOA from that spec.  And you're normatively
referencing RFC 20 (you were just aching to get that one in there) when
the ASCII reference is more than sufficient (btw, if you're using
xml2rfc you're looking for ANSI.X3-4.1986).

I get the following normative:

[ASCII], [RFC3629],  [RFC5985], [RFC2119], [RFC4119], [IEEE.8021AB],
[RFC3046], [RFC4649], [RFC3993], [RFC4580], [IANA.enterprise],
[RFC3986], [RFC4291], [IEEE.80211], [RFC5491], [TS.3GPP.23.003],
[TIA-2000.5], [GPS.ICD], [GALLILEO.ICD], [RFC2865], [IEEE.80211V]

The following are Informative:

[RFC3693], [RFC6155], [ANSI-TIA-1057], [RFC2661], [HARPER], [GPS.SPOOF],
[RFC5226], [RFC3688], [DSL.TR025], [DSL.TR101]

N.B.: regarding TRs 25 and 101, if you meant that other parameters could
somehow be pulled in as responses, then more work needs to be done in
the draft on this point to specify them.

Nits:

*Please save the RFC Editor some time and run the thing through a spell
checker*.  I've only caught a few errors, but I'll bet there are more.

  * In §2, plain language would dictate that an estimate cannot depend
    on a determination.  A location estimate can be based on reported
    observations. 
  * HELD should be defined on first use (§ 3, page 5).
  * Page 5, first paragraph, has some extraneous characters (\u002D).
  * Page 6, PIDF-LO should be defined on first use.
  * §4.2.1, Page 9, "multiple samples of a measurement is taken" ->
    "multiple samples of a measurement are taken"
  * §4.3 Page 9, last paragraph first sentence, last word: change "a
    error" to "an error"
  * Page 12, label the XMPP example.
  * §5.2, Page 14, remote identifier -> Remote ID or remote-id
    (depending on whether you refer to RFC 3046 or RFC4649,
    respectively).  Similarly, subscriber-id or Subscriber-ID for RFCs
    3993 and 4580.
  * Page 16, (type specification) introducted -> introduced
  * idnits indicates that there are 7 instances of lines that are too
    long, and that RFC 20 is listed as normative without being used in
    the body.

Eliot

(*) I thought about whether or not you've chosen the correct format for
this sort of information exchange (as opposed to ASN.1 or JSON), and
whether or not you should abstract away from it.  I think it would be a
good idea for somebody (like a grad student) to do that, but in the end
your primary application for this is HELD, and the draft is just fine
with its use of XML.  But it wouldn't kill you to indicate that as
possible future work, if someone is interested in this information in
very compressed form.