KantaraInitiative / wg-uma

This is the repository of all specifications related to the User Managed Access Group
http://kantarainitiative.org/confluence/display/uma/
Other
27 stars 21 forks source link

Terminology and diagram comments #335

Closed xmlgrrl closed 6 years ago

xmlgrrl commented 6 years ago

The following comments were received during the Public Comment period [UPDATE: Edited to add letters to distinguish individual comments for reference in Disposition of Comments document]:

Terminology

Sequence diagrams

xmlgrrl commented 6 years ago

Editor response to sub-issue (a): In the UMA V1.0.n specifications, the prefix "UMA" was added to "authorization server", "resource server", "client", etc. when necessary for disambiguation in contexts where OAuth was also being discussed. However, in the UMA V2.0 specifications, an UMA authorization server, resource server, client, etc. is literally an OAuth one as well, since UMA is defined as an OAuth extension grant.

The word "authorization" is used carefully and is generally qualified in precise ways with "... process", "... assessment", and so on as defined in the specifications.

When identity (vs authorization) terms are used, and specifically OpenID Connect terms, the full phrases are indeed used (for example OpenID Provider).

The editor therefore recommends making no changes.

xmlgrrl commented 6 years ago

Editor response to sub-issue (b): The editor accepts the suggestion to use the full phrase "permission ticket" where possible, including throughout the figure(s) in Grant.

Not sure if the commenter intended to provide a concrete suggestion regarding "correlation handle"; this is just a phrase used as part of the definitions of "permission ticket" and "persisted claims token (PCT)". While Internet searches for this phrase mostly turn up definitions that are somewhat .NET-specific, its usage in software and protocol conversations is not uncommon.

Regarding the mention of "token", perhaps the suggestion is to always qualify the word "token" with "access ...", "claim ...", "persisted claim ...", "requesting party ...", "refresh ...", "bearer ...", "ID ...", etc. where possible. This would be reasonable. In this case, the editor recommends making the following edits (or similar):

xmlgrrl commented 6 years ago

Editor response to sub-issue (c): The original rationale for including the (out-of-scope) setting of policy by the resource owner was to have a single diagram with the five UMA entities present in it. However, the editor agrees it would indeed be beneficial to demonstrate the asynchronous nature of the grant (along with potentially some notion of the flexibility afforded authorization servers and resource owners in the timing of policy setting) by having two diagrams.

xmlgrrl commented 6 years ago

Editor response to sub-issue (d): This is a good suggestion. While there is a brief explanation, after a fashion, of separate rationales for claims pushing vs claims gathering in Grant Sec 1.3.1, it doesn't follow closely enough after Figure 1. The editor agrees the explanation would be helped by being moved up and by having the two flows separated out into different diagrams, since the hybrid flow in Figure 1 is (at least at this time) an unlikely scenario.

xmlgrrl commented 6 years ago

On second thought about sub-issues (c) and (d): Will need to seek the WG's feedback, as the existing diagram was created with some specific implementer purposes in mind.