While this can work, it
seems like it's mixing semantics a little as well. 'invalid_request'
normally mean that the request is malformed in some way. Even
'unsupported parameter value' is really addressing a malformed
request (e.g. a parameter only allows values of 'true' and 'false'
and the invocation uses the value 'maybe').
What about registering an error code of 'invalid_parameter_value'.
Then the description could explain which parameter value is
invalid and possibly why. We might even be able to shorten this to
'invalid_parameter' as 'invalid_request' handles the case of an
unsupported/invalid parameter name.
I did a quick check of the OpenID Connect specs and they also
don't define an 'invalid_parameter' error code.
Thanks,
George
On 1/9/13 8:34 AM, Peter Mauritius
wrote:
Ok,
now I understand your intention. In oauth-revocation the token is
just a parameter not an authorization token as in RFC6750. RFC6749
uses "invalid_request" for
includes an unsupported parameter value
Perhaps we should drop the error-code and use invalid-request with
a error-description explaining the token parameter is not usable.
Regards
Peter
On 09.01.2013 14:08, George Fletcher wrote:
Hi Peter,
I do agree that the meanings of 'invalid_token' between the
two specs (6750 and revocation) are different. After thinking
about this for a while, I've determined, at least for myself,
what the difference is between the 'invalid_token' error code
in RFC 6750 and the revocation spec.
In RFC 6750 'invalid_token' means that the authorization token
for the request is invalid.
In 'oauth-revocation', 'invalid_token' means that the
parameter containing the token to be revoked is invalid.
I am very concerned about using the same error string,
'invalid_token', to mean two different things. While the
semantic difference is not great in this case, I think it sets
a bad precedent for OAuth to have the same error string have
two different semantic meanings.
I do agree that the error code used in this spec should be
registered.
Thanks,
George
On 1/8/13 5:07 PM, Peter Mauritius
wrote:
Hi George,
RFC6750 defines "invalid-token" for access tokens which is
not the case for "invalid-token" in the revocation
specification. Here it is applicable for refresh tokens as
well. Therefore we should not simply reference the
"invalid-token" of RFC6750.
As far as I understand both, the reviewed specification and
RFC6750, reference RFC6749. RFC6750 includes in section
6.1 OAuth Extensions Error Registration sections
according to RFC6749
section 11.4. for the error codes defined throughout
the document including "invalid-token".
I am not very experienced in the formal process but
shouldn't we add such sections for the two error codes
defined in the revocation document? Especially for
"invalid-token" we should define an error registration
section that defines the error code for our error usage
location and protocol extension to distinguish it from
RFC6750 and to avoid confusion. Doing this I hope there is
no necessity to add a reference to RFC6750 or to define a
new error code.
What do the more experienced reviewers think?
Regards
Peter
Am 07.01.13 17:25, schrieb George Fletcher:
My concern with
leaving both specs separated is that over time the
semantics of the two error codes could diverge and that
would be confusing for developers. If we don't want to
create a dependency on RFC 6750, then I would recommend a
change to the error code name so that there is no name
collision or confusion.
Thanks,
George
On 1/7/13 11:18 AM, Torsten
Lodderstedt wrote:
Hi George,
thank you for pointing this out. Your proposal sounds
reasonable although the revocation spec does not build on
top of RFC 6750.
As refering to RFC 6750 would create a new dependency, one
could also argue it would be more robust to leave both
specs separated.
What do others think?
regards,
Torsten.
Am 07.01.2013 17:12, schrieb
George Fletcher:
One quick
comment...
Section 2.0: Both RFC 6750 and this specification
define the 'invalid_token' error code.
Should this spec reference the error code from RFC
6750?
Thanks,
George
On 1/7/13 7:08 AM, Torsten
Lodderstedt wrote:
Hi,
the new revision is based on the WGLC feedback and
incorporates the following changes:
- renamed "access grant" to "authorization" and
reworded parts of Abstract and Intro in order to
better align with core spec wording (feedback by
Amanda)
- improved formatting of section 2.1. (feedback by
Amanda)
- improved wording of last paragraph of section 6
(feedback by Amanda)
- relaxed the expected behavior regarding revocation
of related tokens and the authorization itself in
order to remove unintended constraints on
implementations (feedback by Mark)
- replaced description of error handling by pointer to
respective section of core spec (as proposed by Peter)
- adopted proposed text for implementation note (as
proposed by Hannes)
regards,
Torsten.
Am 07.01.2013 13:00, schrieb internet-dra...@ietf.org:
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 : Token Revocation
Author(s) : Torsten Lodderstedt
Stefanie Dronia
Marius Scurtescu
Filename :
draft-ietf-oauth-revocation-04.txt
Pages : 8
Date : 2013-01-07
Abstract:
This document proposes an additional endpoint
for OAuth authorization
servers, which allows clients to notify the
authorization server that
a previously obtained refresh or access token is
no longer needed.
This allows the authorization server to cleanup
security credentials.
A revocation request will invalidate the actual
token and, if
applicable, other tokens based on the same
authorization.
The IETF datatracker status page for this draft is:
https://datatracker.ietf.org/doc/draft-ietf-oauth-revocation
There's also a htmlized version available at:
http://tools.ietf.org/html/draft-ietf-oauth-revocation-04
A diff from the previous version is available at:
http://www.ietf.org/rfcdiff?url2=draft-ietf-oauth-revocation-04
Internet-Drafts are also available by anonymous FTP
at:
ftp://ftp.ietf.org/internet-drafts/
_______________________________________________
OAuth mailing list
OAuth@ietf.org
https://www.ietf.org/mailman/listinfo/oauth
_______________________________________________
OAuth mailing list
OAuth@ietf.org
https://www.ietf.org/mailman/listinfo/oauth
--
Peter Mauritius Chief Technical Director
Senior Consultant
Tel: +49 721 96448-0 Fax: +49 721 96448-299 peter.maurit...@fun.de
fun communications GmbH Lorenzstr. 29 D-76135 Karlsruhe
Geschaeftsfuehrer Johannes Feulner
Amtsgericht Mannheim HRB 106906
http://www.fun.de
http://blogs.fun.de
http://www.twitter.com/fun_de
http://www.facebook.com/funcommunications
--
|