Re: [Captive-portals] AD review of draft-ietf-capport-architecture-07

Kyle Larose <kyle@agilicus.com> Mon, 11 May 2020 15:10 UTC

Return-Path: <kyle@agilicus.com>
X-Original-To: captive-portals@ietfa.amsl.com
Delivered-To: captive-portals@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AB1663A0044 for <captive-portals@ietfa.amsl.com>; Mon, 11 May 2020 08:10:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level:
X-Spam-Status: No, score=-2.098 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=agilicus.com
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 55fD_y58J3Kj for <captive-portals@ietfa.amsl.com>; Mon, 11 May 2020 08:10:38 -0700 (PDT)
Received: from mail-io1-xd34.google.com (mail-io1-xd34.google.com [IPv6:2607:f8b0:4864:20::d34]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 239613A0062 for <captive-portals@ietf.org>; Mon, 11 May 2020 08:10:28 -0700 (PDT)
Received: by mail-io1-xd34.google.com with SMTP id x5so967009ioh.6 for <captive-portals@ietf.org>; Mon, 11 May 2020 08:10:28 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=agilicus.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=0i23rfUIA87xpFHuQ70xAcXIk0Iwy4a+VXcjcAVn6u8=; b=HLZ/qUpVKEKPdCyXqXJwVhvtpznJ7ghPeHptvWD8W+BjLnBYUIezMa+EgK/L3iPSJo rT9phv80fkBrWUgkOcIyYRsuJivhaoBDLaDkl8oWd9e3Tlu5AXR5JW2yIufn5CD0Py9i XtYB+ybwd3OVCMSJ66xA1mHE8V+LrnlWNe5hXG0GkonHvnQlXkFBkvuPNBIyd8AZLrF3 7by1qfOO6gb2A9obxaHlVWaffY6vKMPbbcWKJKgIk8XhFLuvzKlyX8DLIffyeIlA7hfp bUZ3bvNYtffUoKfZlU6/4gZrNuxN/3QTxEoU7oHbpEp3MxIDRp2Jo00lwrkgnE1+MDmF /DJw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=0i23rfUIA87xpFHuQ70xAcXIk0Iwy4a+VXcjcAVn6u8=; b=kQhTzsO0m9YwA0+y6v49AgInQmIOEp846573KtfZU1rHjIece1wRMNBX9s+h9VR1+O QhMxBCvT0lCprMvCpGBaoiqltNlPELC7JYJYOEBjsV2tq4yvW4/27houtm9ZwTufaQF5 sByB9/1YAwtq+gBUkJbi7ZVrWsevKn9o6oPIVyTMKM9U5WmmxOznGNpDHnMSR5bnCbCr 7Bd2ApM3DHZH0C+ntRV+PQYt3LX4xg09wY7hcPUUpLmTSi0UPyT7IE989qRtiw/enZyg nT+UaqRi4ELUBzlnZSJiWunS5ub0DDveK4OfEzKWn9AlttOFxVfIJKoW7WR/kAzkZCcD 6Wlw==
X-Gm-Message-State: AGi0PuZF98zK0y9NZjV2LpI5NCLH3qfpoONh7qNNJmyLsBzr4hpuz1Xr gYihpOx+6bZ7BuWz5fTNbXvGBlkd/GY6L3Jq7YI0wxjY9w==
X-Google-Smtp-Source: APiQypL2LHPyF4Njn/WRNguSmx5nnJAs2WQXXdMxsyTmLQ9dqE18xUSe+2Egq9WvsECeKWsksAHZfkYJjS40n0YdhY0=
X-Received: by 2002:a5d:8ad9:: with SMTP id e25mr16120346iot.37.1589209828065; Mon, 11 May 2020 08:10:28 -0700 (PDT)
MIME-Version: 1.0
References: <CALaySJ+K1FZu5POa=TLz2-ON8=jvyE03gqdi+5+cNj8X4E9RoA@mail.gmail.com> <a4a3747ef330d8b2ac94a178ab691ca8@golden.net> <CALaySJLxqtqGAytEK3X083zg3hyXSOQ6FdnaTz5t1tDNS8tKeA@mail.gmail.com> <CALaySJJ5Z2QpnVQ19ax+ChVNLSYk0yKhpk7uX4P5NiU-xcwJuA@mail.gmail.com>
In-Reply-To: <CALaySJJ5Z2QpnVQ19ax+ChVNLSYk0yKhpk7uX4P5NiU-xcwJuA@mail.gmail.com>
From: Kyle Larose <kyle@agilicus.com>
Date: Mon, 11 May 2020 11:10:17 -0400
Message-ID: <CACuvLgwOUdQbjLk+m3NO2OSOWxexsp+iM2-Vyd15cmJ=hEoUdw@mail.gmail.com>
To: Barry Leiba <barryleiba@computer.org>
Cc: David Dolson <ddolson@acm.org>, draft-ietf-capport-architecture.all@ietf.org, captive-portals@ietf.org
Content-Type: multipart/alternative; boundary="0000000000004696a705a560bf3d"
Archived-At: <https://mailarchive.ietf.org/arch/msg/captive-portals/6YSdn4t93aitXr_dVcVhkuJKgWM>
Subject: Re: [Captive-portals] AD review of draft-ietf-capport-architecture-07
X-BeenThere: captive-portals@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussion of issues related to captive portals <captive-portals.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/captive-portals>, <mailto:captive-portals-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/captive-portals/>
List-Post: <mailto:captive-portals@ietf.org>
List-Help: <mailto:captive-portals-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/captive-portals>, <mailto:captive-portals-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 11 May 2020 15:10:44 -0000

Hey Barry,

I'll upload one shortly. Sorry about the delay.

On Mon, 11 May 2020 at 09:18, Barry Leiba <barryleiba@computer.org> wrote:

> Can we get a revised I-D up on this now, so I can get the last call
> started?  The last calls on the other two documents are ending this
> week.
>
> Martin, should I hold the telechat scheduling of the other two
> documents to make sure that architecture is on the same telechat as
> they are, so the IESG is reviewing them all together?  I think that's
> best, but can be convinced not to have the others wait.
>
> Barry
>
> On Mon, Apr 27, 2020 at 10:21 AM Barry Leiba <barryleiba@computer.org>
> wrote:
> >
> > Thanks for the reply, Dave, and I think we're OK to start last call on
> > the document after you post a revised I-D with the changes so far --
> > most unresolved things are not at a blocking level.  Just one thing
> > I'd like to push on before you revise the I-D, though:
> >
> > > > Please be consistent about using “URI”, and not “URL”.
> > >
> > > Changed all URI to URL in commit
> > >
> https://github.com/capport-wg/architecture/commit/a9c87ba48aa64564bd9d0e1f21bd82906a2714f4
> >
> > But that's backward: the IETF formally defines "URI" [RFC3986], not
> > "URL", and that document says [Section 1.1.3]:
> >
> >    Future specifications and related documentation should
> >    use the general term "URI" rather than the more restrictive terms
> >    "URL" and "URN"
> >
> > (Additionally, we should probably include an informative reference to
> > 3986 on first use of the term.)
> >
> > If the working group really wants "URL" here I won't block it further,
> > but I would strongly prefer that we use "URI", consistent with IETF
> > usage.
> >
> > ---
> >
> > Collecting the other points that aren't resolved, but that need not
> > block last call:
> >
> > > > General:
> > > > Expect comments during IESG Evaluation about the extensive use of BCP
> > > > 14 key words in an Informational document.  I don’t object to it
> here,
> > > > though I do find all the “MAY”s odd (example: “A device MAY query
> this
> > > > API at any time”, rather than “A device may query this API at any
> > > > time”), but I do expect some ADs to comment on it.
> > >
> > > I've reviewed all upper-case "MAY", and I believe they are used as
> > > intended.  We've allowed the user equipment to participate or not in
> various ways.
> >
> > OK, then no more to do here.  This was mostly just a note that I
> > expect to see comments, and not an expectation of any changes to the
> > document.  Thanks for checking.
> >
> > > > — Section 2.3 —
> > > >
> > > >    The API SHOULD provide evidence
> > > >    to the caller that it supports the present architecture.
> > > >
> > > > I don’t understand what this means; can you explain?
> > >
> > > To me, this means that User Equipment can determine from the
> > > interaction that the API is implementing this architecture vs.
> > > being some random API. I imagine that a version number or content-
> > > type could achieve this.
> > >
> > > I've opened https://github.com/capport-wg/architecture/issues/61
> > > to track the issue.
> >
> > OK.  A small wording tweak will be good at some point, but let's see
> > if anyone else raises this point in reviews.
> >
> > > > — Section 3.3 —
> > > > Are we really talking about evaluating individual identifiers here?
> > > > Or does this really mean to discuss *methods* of generating or
> > > > choosing identifiers?
> > >
> > > I believe this is about choice of identifier in a solution/standard.
> > >
> > > Opened issue https://github.com/capport-wg/architecture/issues/62
> >
> > Great; again, a small wording tweak will do it.
> >
> > > > — Section 3.4.3 —
> > > > Is this section talking about using a context-free URI as a UEe
> > > > identifier?  It should be clearer about that, if so (and if that’s
> not
> > > > what it’s about, the section is misplaced).  There’s nothing in here
> > > > that discusses how such identifiers would meet the specified criteria