Last Call Review of draft-ietf-oauth-discovery-07
review-ietf-oauth-discovery-07-genart-lc-carpenter-2017-10-01-00

Request Review of draft-ietf-oauth-discovery
Requested rev. no specific revision (document currently at 08)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2017-10-09
Requested 2017-09-25
Other Reviews Opsdir Last Call review of -07 by Shwetha Bhandari (diff)
Secdir Last Call review of -07 by Donald Eastlake (diff)
Genart Telechat review of -08 by Brian Carpenter
Review State Completed
Reviewer Brian Carpenter
Review review-ietf-oauth-discovery-07-genart-lc-carpenter-2017-10-01
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/bFBXIpSB2BrhZJr0AzcJ2DL011w
Reviewed rev. 07 (document currently at 08)
Review result Ready
Draft last updated 2017-10-01
Review completed: 2017-10-01

Review
review-ietf-oauth-discovery-07-genart-lc-carpenter-2017-10-01

Gen-ART Last Call review of draft-ietf-oauth-discovery-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-oauth-discovery-07.txt
Reviewer: Brian Carpenter
Review Date: 2017-10-02
IETF LC End Date: 2017-10-09
IESG Telechat date: 

Summary: Ready
--------

Comment:
--------

As far as my competence goes, I have no issues with this draft.
Just for fun, I checked that the JSON example works as the
value of a GRASP objective (draft-ietf-anima-grasp) with the
GRASP prototype code. And yes, of course it does, so we could
map OAuth over the ANIMA discover/synchronize model if we wanted.

FWIW there are a couple of errors in the shepherd's writeup:

> This document does not request any actions by IANA.
>
> 18) List any new IANA registries that require Expert Review for future
> allocations. Provide any public guidance that the IESG would find
> useful in selecting the IANA Experts for these new registries.
>
> None.

Wrong, there are extensive IANA considerations and a requirement
for multiple Designated Experts.

> There is no text in formal languages in the document. 

Maybe not, but there is a JSON example (which as noted
above seems to be fine).

--