Torsten, thanks for the review. Comments inline.

On 05/27/2013 03:03 PM, Torsten Lodderstedt wrote:
Hi Justin,

the drafts looks very good.

Just some questions/comments from my side:

section 1.4

How is the client supposed to identify/distinguish authorization servers? Based on the Client Registration Endpoint URI? Authorization server identification is necessary in order to map client_ids to authorization servers for clients, which are connected to multiple authorization servers.
That's a great question -- but I think it's entirely dependent on how discovery and configuration is set up for a client, which is ultimately orthogonal to registration. The way that I've implemented it in our client is based on the OpenID Connect discovery process, which bases everything in the server's configuration off of an "issuer" URL. It would be easy enough to point out here that discovery and differentiation of different servers is out of scope.


section 1.4.1 f

Why does the client secret expire while the access token ist still valid? Secret and token are stored at the same
locations so an attacker may obtain both at once.
Secrets are used at the token endpoint, so the attack surface is slightly different. Since you can only use the registration access token at the registration endpoint, you can use it to rotate your other credentials.


"token_endpoint_auth_method"
What is the use case for dynamic registration of public clients? In my opinion, public clients exist because OAuth 2.0 core does not provided a mechanism to provision secrets to the different instances of an installed/native app. Dynamic registration closes this gap, so any installed app may retrieve a distinct secret.

This gap-closing is true for some classes of client that used to have to be public, like many native apps. But there are still clients that have no use for a client secret, like in-browser clients that use the implicit flow. Now if these clients are also talking to multiple auth servers, they'll each need a client_id at the auth server (because there's no practical way to publish a "public" client ID to *all* auth servers). A discussion in the security considerations about the limitations and usefulness of this use case is probably worthwhile, so I'll make a note to do that.


"client_secret_post vs client_secret_basic"
BASIC and POST are essentially the same just different ways to send the client secret. If an authorization server supports both, both should work for any client. So are both methods treated differently?
I agree, and this was one of my original arguments for making this field plural (or plural-able), but there hasn't been WG support for that so far.


"jwks_uri"
What is this data used for? the OAuth JWT Bearer Token Profiles?

That's one, but it's really for any higher-level protocol that uses signing and encryption. There are several out there that are using JOSE on top of OAuth to do things, so we felt it was worthwhile to have one standard place to have the client say "here's my public key".

 -- Justin


best regards,
Torsten.

Am 24.05.2013 23:10, schrieb Richer, Justin P.:
New Dynamic Registration draft is published, incorporating much of the discussion from the group this week.

Some normative changes that should have minimal impact:
  - "expires_at" is now "client_secret_expires_at"
  - "issued_at" is now "client_id_issued_at"
  - creation of an IANA registry for token_endpoint_auth_method
- removal of two underdefined values from token_endpoint_auth_method (client_secret_jwt and private_key_jwt), these are now defined in an extension (OpenID Connect Registration)

And several editorial changes:

- new "client lifecycle" section that describes how different kinds of clients can use the dynamic registration protocol, how a client's credentials get refreshed, and the relationship between the Client Identifier and the Client software - new "registration tokens and credentials" section describing the different kinds of tokens and credentials used in the registration process, what they're for, and why they're all separate - clarified the definitions of several fields like policy_uri and tos_uri

Thanks for all the great feedback, and please keep the constructive commentary coming!
  -- Justin

On May 24, 2013, at 4:36 PM, <[email protected]>
  wrote:

A New Internet-Draft is available from the on-line Internet-Drafts directories. This draft is a work item of the Web Authorization Protocol Working Group of the IETF.

    Title           : OAuth 2.0 Dynamic Client Registration Protocol
    Author(s)       : Justin Richer
                          John Bradley
                          Michael B. Jones
                          Maciej Machulak
    Filename        : draft-ietf-oauth-dyn-reg-11.txt
    Pages           : 34
    Date            : 2013-05-24

Abstract:
   This specification defines an endpoint and protocol for dynamic
   registration of OAuth 2.0 Clients at an Authorization Server and
   methods for the dynamically registered client to manage its
   registration.


The IETF datatracker status page for this draft is:
https://datatracker.ietf.org/doc/draft-ietf-oauth-dyn-reg

There's also a htmlized version available at:
http://tools.ietf.org/html/draft-ietf-oauth-dyn-reg-11

A diff from the previous version is available at:
http://www.ietf.org/rfcdiff?url2=draft-ietf-oauth-dyn-reg-11


Internet-Drafts are also available by anonymous FTP at:
ftp://ftp.ietf.org/internet-drafts/

_______________________________________________
OAuth mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/oauth
_______________________________________________
OAuth mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/oauth


_______________________________________________
OAuth mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/oauth

Reply via email to