[salud] AD review of draft-ietf-salud-alert-info-urns-12

Richard Barnes <rlb@ipv.sx> Fri, 11 April 2014 17:33 UTC

Return-Path: <rlb@ipv.sx>
X-Original-To: salud@ietfa.amsl.com
Delivered-To: salud@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3F89E1A0727 for <salud@ietfa.amsl.com>; Fri, 11 Apr 2014 10:33:16 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.078
X-Spam-Level:
X-Spam-Status: No, score=-0.078 tagged_above=-999 required=5 tests=[BAYES_20=-0.001, FM_FORGED_GMAIL=0.622, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=ham
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 NZVqRyas8sDx for <salud@ietfa.amsl.com>; Fri, 11 Apr 2014 10:33:14 -0700 (PDT)
Received: from mail-ob0-f171.google.com (mail-ob0-f171.google.com [209.85.214.171]) by ietfa.amsl.com (Postfix) with ESMTP id 1018E1A070D for <salud@ietf.org>; Fri, 11 Apr 2014 10:33:14 -0700 (PDT)
Received: by mail-ob0-f171.google.com with SMTP id wn1so6391876obc.2 for <salud@ietf.org>; Fri, 11 Apr 2014 10:33:12 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:date:message-id:subject:from:to :content-type; bh=czC+furOSJjZWgcWEAMACq5jhhSevp0M1PBGfkKIIdQ=; b=mj0S14NgukAQC+KcqROEZHEEwMzrb7jMOBMQEf6Ygbb6lrKz5lOIslLyV8zqDO0HQk tM8GOQCTOR0dR999KbzjBle9bHXFGU5109eVNyPhnOe5uduz1E33ezt1X/eN4Ll23hXy Qm1miyO9HRZ7kXp4rP22dNa8IX6RCii5OmL1YVKt1Y7YN17FWg3fFQe5+07yifbFpBSE KUMyYrPA1gWm7y4zeKOyBW0V3hK9ikojrQt0YBlZEN5RCbQhoRrNEuTW6RhttDfsTaVd GxTpiGs6xwVruOFrYRJ4lRjTrnbOBEMkcJhI69FrGnvLWyTEjcgobmgXDR0HR3sll+3W R7QQ==
X-Gm-Message-State: ALoCoQneLie6a9PoqQLEzmjsKPv3V5unZEkfLjWuGgWa2uv8dmiw0B4F+dmaVfzIPaIkeTX99P9i
MIME-Version: 1.0
X-Received: by 10.182.135.228 with SMTP id pv4mr2559250obb.62.1397237592585; Fri, 11 Apr 2014 10:33:12 -0700 (PDT)
Received: by 10.60.136.231 with HTTP; Fri, 11 Apr 2014 10:33:12 -0700 (PDT)
Date: Fri, 11 Apr 2014 13:33:12 -0400
Message-ID: <CAL02cgR9SAtq3hxck1zkJmUYOSqNfPDQzZPYX5ztpWNTOnYfYQ@mail.gmail.com>
From: Richard Barnes <rlb@ipv.sx>
To: draft-ietf-salud-alert-info-urns@tools.ietf.org, salud@ietf.org
Content-Type: multipart/alternative; boundary=089e0112c5d25fa4f504f6c7ba6c
Archived-At: http://mailarchive.ietf.org/arch/msg/salud/hT1dDM_VPHPauFSWj5ktHuXNyNw
Subject: [salud] AD review of draft-ietf-salud-alert-info-urns-12
X-BeenThere: salud@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Sip ALerting for User Devices working group discussion list <salud.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/salud>, <mailto:salud-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/salud/>
List-Post: <mailto:salud@ietf.org>
List-Help: <mailto:salud-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/salud>, <mailto:salud-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 11 Apr 2014 17:33:16 -0000

I have reviewed this document in preparation for IETF LC.  Thanks for a
clearly written document!  I especially liked the style of the Terminology
section.  I've requested LC, and a few comments to address along with any
LC comments are below.

Thanks,
--Richard

Some of the values in Section 8.2 have internal colons, which seems like
you're actually defining two things.  But it's not meaningful to remove the
final label.  Should these use a different separator?

In 8.2.6, do you want a language tag instead of a country?

It seems like the tables in 9.2.* could be collapsed into one.

It's not clear to me why you need all the date machinery in provider IDs.
 This things are ephemeral -- why not just use the org that has the domain
now?

Section 13 says that the UA MUST provide a "reasonable rendering".   How do
I tell if a UA meets that criterion?  Might be better to add in Section 11
a requirement of the form, "if a UA cannot process the set of Alert URNs in
a message, then it MUST still provide some alert (chosen by local policy).
 Failure to parse Alert URNs MUST NOT cause a UA not to alert at a all."