[core] Ben Campbell's Discuss on draft-ietf-core-senml-14: (with DISCUSS and COMMENT)

Ben Campbell <ben@nostrum.com> Mon, 16 April 2018 05:15 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: core@ietf.org
Delivered-To: core@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 25E1C12D893; Sun, 15 Apr 2018 22:15:13 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Ben Campbell <ben@nostrum.com>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-core-senml@ietf.org, Jaime Jimenez <jaime.jimenez@ericsson.com>, core-chairs@ietf.org, jaime.jimenez@ericsson.com, core@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.78.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <152385571314.20985.5160681583375127961.idtracker@ietfa.amsl.com>
Date: Sun, 15 Apr 2018 22:15:13 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/Gpx7oyKDbHS2OsPGC95dAfXBc9I>
Subject: [core] Ben Campbell's Discuss on draft-ietf-core-senml-14: (with DISCUSS and COMMENT)
X-BeenThere: core@ietf.org
X-Mailman-Version: 2.1.22
List-Id: "Constrained RESTful Environments \(CoRE\) Working Group list" <core.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/core>, <mailto:core-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/core/>
List-Post: <mailto:core@ietf.org>
List-Help: <mailto:core-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/core>, <mailto:core-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 16 Apr 2018 05:15:13 -0000

Ben Campbell has entered the following ballot position for
draft-ietf-core-senml-14: Discuss

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-core-senml/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

Hopefully this is easy to address:

§4.7  talks about how SenML can also be used to configure parameters and
controlling actuators. That capability has some rather significant security
implications, but I failed to find mention of it in the security
considerations. That needs to be explicitly discussed.


----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Substantive:

§4.4: "If this value is a version number larger than the version which the
   system understands, the system SHOULD NOT use this object."

Why not MUST NOT? Are there situations where an implementation might reasonably
use an object with a higher version number than the implementation understands?

Editorial/Nits:

The title is misleading. It makes it sound like the document is just
registering media types; in fact it defines SenML.

§1, first paragraph: "This format was defined...": The antecedent of "this" is
unclear, since the fact the document defines SenML has not yet been mentioned.

§4.3, table 1: What do the asterisks mean?

§5.1.2, paragraph starting with "Note that...": I'm surprised to find normative
requirements buried in a note in an example.

§10, first paragraph: " the an integrated sum...": competing articles.

§14: "Sensor data can range from information with almost no security
   considerations..."
s/security/privacy