[sacm] Alexey Melnikov's No Objection on draft-ietf-sacm-nea-swima-patnc-03: (with COMMENT)
Alexey Melnikov <aamelnikov@fastmail.fm> Wed, 28 February 2018 15:13 UTC
Return-Path: <aamelnikov@fastmail.fm>
X-Original-To: sacm@ietf.org
Delivered-To: sacm@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 5CDCD126E01; Wed, 28 Feb 2018 07:13:53 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Alexey Melnikov <aamelnikov@fastmail.fm>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-sacm-nea-swima-patnc@ietf.org, sacm@ietf.org, Karen O'Donoghue <odonoghue@isoc.org>, sacm-chairs@ietf.org, odonoghue@isoc.org, sacm@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.73.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <151983083337.5220.14097491000929319862.idtracker@ietfa.amsl.com>
Date: Wed, 28 Feb 2018 07:13:53 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/sacm/VVQJBT-zH-Jv_FTYZ2dKMdT3Py0>
Subject: [sacm] Alexey Melnikov's No Objection on draft-ietf-sacm-nea-swima-patnc-03: (with COMMENT)
X-BeenThere: sacm@ietf.org
X-Mailman-Version: 2.1.22
List-Id: SACM WG mail list <sacm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sacm>, <mailto:sacm-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/sacm/>
List-Post: <mailto:sacm@ietf.org>
List-Help: <mailto:sacm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sacm>, <mailto:sacm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 28 Feb 2018 15:13:53 -0000
Alexey Melnikov has entered the following ballot position for draft-ietf-sacm-nea-swima-patnc-03: No Objection 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-sacm-nea-swima-patnc/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Thank you for addressing my DISCUSS and many of my comments. Remaining comments listed below (plus see updated point #7) Minor: (Points not mentioned below were addressed). 2) When referencing RFC 5198 (Network Unicode format), I personally prefer a stricter version that disallows control characters (other than CR LF). In RFC 5198, use of control characters is only "SHOULD NOT". So you might want to make a stronger statement on this. 5) In Section 4.3. Required Exchanges All SWIMA-PVs and SWIMA-PCs MUST support both types of exchanges. I question the value in requiring SWIMA-PVs in supporting both types of exchanges. For example, if a SWIMA-PV never uses subscriptions, it doesn't seem to need to handle subscription responses. Similar text in 5.2: All SWIMA-PVs MUST be capable of sending SWIMA Request attributes and be capable of receiving and processing all SWIMA Response attributes as well as PA-TNC Error attributes. 6) Use of fields which can contain both human readable and possibly machine readable information - I think this is rather handwavy and I wish you would be more specific. Also consider issues raised in BCP 18 (RFC 2277), in particular language tagging of human readable text. 7) (downgraded from DISCUSS): SWID registry is using "http://invalid.unavailable" Tag Creator RegID value in Section 6.1.1. invalid.unavailable is not a valid domain name and "unavailable" is not registered in the special-use domain registry <https://www.iana.org/assignments/special-use-domain-names/special-use-domain-names.xhtml> I am not entirely sure how big of a problem this is, but use of something which can be interpreted as a URI in a non existing non special-use domain seems like a bad idea. Note, if you can change the value to "http://unavailable.invalid", that would address my concern.
- [sacm] Alexey Melnikov's No Objection on draft-ie… Alexey Melnikov