[Captive-portals] Genart last call review of draft-ietf-capport-api-07

Brian Carpenter via Datatracker <noreply@ietf.org> Mon, 04 May 2020 04:29 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: captive-portals@ietf.org
Delivered-To: captive-portals@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id AE9763A0BED; Sun, 3 May 2020 21:29:55 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Brian Carpenter via Datatracker <noreply@ietf.org>
To: <gen-art@ietf.org>
Cc: captive-portals@ietf.org, last-call@ietf.org, draft-ietf-capport-api.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.129.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <158856659566.16634.2637013886257923503@ietfa.amsl.com>
Reply-To: Brian Carpenter <brian.e.carpenter@gmail.com>
Date: Sun, 03 May 2020 21:29:55 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/captive-portals/wPUlTAUpBq0aj2ikTTjYtNtUGZk>
Subject: [Captive-portals] Genart last call review of draft-ietf-capport-api-07
X-BeenThere: captive-portals@ietf.org
X-Mailman-Version: 2.1.29
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, 04 May 2020 04:29:56 -0000

Reviewer: Brian Carpenter
Review result: Ready with Nits

https://www.ietf.org/id/draft-ietf-capport-api-07.html

Gen-ART Last Call review of draft-ietf-capport-api-07

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-capport-api-07.html
Reviewer: Brian Carpenter
Review Date: 2020-05-04
IETF LC End Date: 2020-05-11
IESG Telechat date:  

Summary: Ready (almost...)
--------

Minor Issue:
------------

>      If the client is captive (i.e. captive=true),
>      it can still be allowed enough access for it to perform server
>      authentication Section 4.1.

What does "can" mean? MAY or perhaps SHOULD? Is this a local policy decision?

Nit:
----

>      If the client is captive (i.e. captive=true),
>      it can still be allowed enough access for it to perform server
>      authentication Section 4.1.

Missing parentheses around "Section 4.1"?