On Wed, May 30, 2018 at 6:06 PM, William Denniss <wdenniss@xxxxxxxxxx> wrote:
On Wed, May 30, 2018 at 3:48 PM, Brian Campbell <bcampbell@xxxxxxxxxxxxxxxx> wrote:I realize this is somewhat pedantic but I don't think referencing 4.1.2.1
works given how RFC 6749 set things up. Rather I believe that the device
flow needs to define and register "access_denied" as a valid token endpoint
response error code (it's not a token endpoint response error per RFC 6749
sec 5.2 nor has it been registered https://www.iana.org/assignmen
ts/oauth-parameters/oauth-parameters.xhtml#extensions-error ). Also <https://tools.ietf.org/html/r
invalid_grant is a a token endpoint response error from RFC 6749 sec 5.2 so
that reference is needed and appropriate. RFC 6749 Sec 4.1.2.1fc6749#section-4.1.2 > defines errors returned
from the authorization endpoint. But the device flow errors are from the
token endpoint.Yes, that's true. It's still the token endpoint, so 5.2 does in fact apply, it's just we're mixing in authorization-style actions which were not previously considered/used for that endpoint.Do you have any proposed text to resolve this?
Sure, here's a crack at some text/changes:
Add this to the list of error codes in section 3.5.:
"access_denied
The end-user denied the authorization request."
The end-user denied the authorization request."
And add this to section 7.2..1.:
"o Error name: access_denied
o Error usage location: Token endpoint response
o Related protocol extension: [[ this specification ]]
o Change controller: IETF
o Specification Document: Section 3.5 of [[ this specification ]]"
o Error usage location: Token endpoint response
o Related protocol extension: [[ this specification ]]
o Change controller: IETF
o Specification Document: Section 3.5 of [[ this specification ]]"
I might also slightly change this text in section 3.5:
"In addition to the error codes defined in Section 5.2 of [RFC6749],
the following error codes are specific for the device flow:"
the following error codes are specific for the device flow:"
to
"In addition to the error codes defined in Section 5.2 of [RFC6749],
the following error codes are specified by the device flow:"
the following error codes are specified by the device flow:"
so that the wording doesn't read as prohibiting the error codes from being used outside the device flow (access_denied from the token endpoint might well be useful for other grant types).
And add "Andrew Sciberras" to the Acknowledgements.
CONFIDENTIALITY NOTICE: This email may contain confidential and privileged material for the sole use of the intended recipient(s). Any review, use, distribution or disclosure by others is strictly prohibited.. If you have received this communication in error, please notify the sender immediately by e-mail and delete the message and any file attachments from your computer. Thank you.
