Thanks for the implementation feedback Michael.

-- Dick

On Jul 17, 2012, at 3:46 PM, Michael Scalia wrote:

> Thanks for your response, Dick, and for pointing out that this is 
> RECOMMENDED.  I'll just say one more thing about this.  While implementing 
> the token endpoint of an authorization server, I naturally looked for the 
> things I needed to account for in the request under 4.1.3  Access Token 
> Request.  I missed this because it wasn't with the other information about 
> access token requests.  Others may miss it as well, for the same reason.  
> Just happened to notice this today under 4.1.2 while re-reading the spec.
> 
> Regards,
> Michael
> 
> On Tue, Jul 17, 2012 at 6:09 PM, Dick Hardt <dick.ha...@gmail.com> wrote:
> Thanks for the feedback Michael.
> 
> 4.1.2 is where the authorization code is first talked about, and it makes 
> sense to discuss how it is generated and used at that point. I can see how it 
> might also be useful to put it in 4.1.3. Note that this is this is 
> RECOMMENDED as opposed to MUST so it does not flow into "The authorization 
> server MUST" list of points.
> 
> Personally, I don't see a need to change. Anyone else have an opinion on this?
> 
> -- Dick
> 
> On Jul 17, 2012, at 2:22 PM, Michael Scalia wrote:
> 
> > Dear OAuth Authors,
> >
> > I'm not sure if this is the right way to suggest an edit to the current 
> > OAuth draft.  Please let me know if I should use a different route.
> >
> > Section 4.1.2 Authorization Response includes the text, "If an 
> > authorization code is used more than once, the authorization server MUST 
> > deny the request and SHOULD revoke (when possible) all tokens previously 
> > issued based on that authorization code.  The authorization code is bound 
> > to the client identifier and redirection URI."
> >
> > I believe this text is in the wrong place.  A client does not supply the 
> > authorization code to the authorization endpoint.  It supplies it to the 
> > token endpoint.  This should move to 4.1.3. Access Token Request, in the 
> > list of bulleted items under "The authorization server MUST".
> >
> > Thanks for all your work on this protocol.
> >
> > Regards,
> > Michael Scalia
> 
> 

_______________________________________________
OAuth mailing list
OAuth@ietf.org
https://www.ietf.org/mailman/listinfo/oauth

Reply via email to