Re: [OAUTH-WG] question about the b64token syntax in draft-ietf-oauth-v2-bearer

Justin Richer <jricher@mitre.org> Wed, 07 March 2012 19:02 UTC

Return-Path: <jricher@mitre.org>
X-Original-To: oauth@ietfa.amsl.com
Delivered-To: oauth@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 8CF0811E8086 for <oauth@ietfa.amsl.com>; Wed, 7 Mar 2012 11:02:10 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.583
X-Spam-Level:
X-Spam-Status: No, score=-6.583 tagged_above=-999 required=5 tests=[AWL=0.016, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id RWO6RwCS1vYs for <oauth@ietfa.amsl.com>; Wed, 7 Mar 2012 11:02:08 -0800 (PST)
Received: from smtpksrv1.mitre.org (smtpksrv1.mitre.org [198.49.146.77]) by ietfa.amsl.com (Postfix) with ESMTP id 74C4311E8087 for <oauth@ietf.org>; Wed, 7 Mar 2012 11:02:05 -0800 (PST)
Received: from smtpksrv1.mitre.org (localhost.localdomain [127.0.0.1]) by localhost (Postfix) with SMTP id 44E0C21B1987; Wed, 7 Mar 2012 14:02:01 -0500 (EST)
Received: from IMCCAS04.MITRE.ORG (imccas04.mitre.org [129.83.29.81]) by smtpksrv1.mitre.org (Postfix) with ESMTP id 8D0AE21B0F6B; Wed, 7 Mar 2012 14:02:00 -0500 (EST)
Received: from [129.83.50.12] (129.83.31.51) by IMCCAS04.MITRE.ORG (129.83.29.81) with Microsoft SMTP Server (TLS) id 14.1.339.1; Wed, 7 Mar 2012 14:02:00 -0500
Message-ID: <4F57B04A.9070509@mitre.org>
Date: Wed, 07 Mar 2012 14:00:26 -0500
From: Justin Richer <jricher@mitre.org>
User-Agent: Mozilla/5.0 (X11; Linux i686; rv:10.0.2) Gecko/20120216 Thunderbird/10.0.2
MIME-Version: 1.0
To: Brian Campbell <bcampbell@pingidentity.com>
References: <CA+k3eCTTsqJZ7XzjA1qgxEJcyU0uio5EN2=yvs+h6ja1JEymiQ@mail.gmail.com> <255B9BB34FB7D647A506DC292726F6E114EDF66EE8@WSMSG3153V.srv.dir.telstra.com> <4E1F6AAD24975D4BA5B1680429673943663DB078@TK5EX14MBXC283.redmond.corp.microsoft.com> <CA+k3eCS71Lhfffu-D_mZ=emk_rR7FASdSjpu+j1KnJWytSEXLw@mail.gmail.com> <1331059528.92345.YahooMailNeo@web31806.mail.mud.yahoo.com> <CA+k3eCQ-PJvOYMB-fg6PtShy2=Wiit01bFvEVE-1_VpEABg5Eg@mail.gmail.com>
In-Reply-To: <CA+k3eCQ-PJvOYMB-fg6PtShy2=Wiit01bFvEVE-1_VpEABg5Eg@mail.gmail.com>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 8bit
X-Originating-IP: [129.83.31.51]
Cc: oauth <oauth@ietf.org>
Subject: Re: [OAUTH-WG] question about the b64token syntax in draft-ietf-oauth-v2-bearer
X-BeenThere: oauth@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: OAUTH WG <oauth.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/oauth>, <mailto:oauth-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/oauth>
List-Post: <mailto:oauth@ietf.org>
List-Help: <mailto:oauth-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/oauth>, <mailto:oauth-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 07 Mar 2012 19:02:10 -0000

Makes sense to me, except that I think the token_type value is typically 
lowercase "bearer", though it's defined to be case insensitive in 
Oauth-v2-23 section 5.1. Come to think of it, I'm not sure that the 
value of this field for the Bearer token type ever got defined anywhere. 
Section 7.1 references the bearer spec as defining the value of the 
"token_type" parameter, and passes its example as if by reference. Would 
be worthwhile giving an example of a token endpoint response, such as 
what's found in OAuth-v2-23 section 5.1.

  -- Justin

On 03/07/2012 12:16 PM, Brian Campbell wrote:
> I'd like to propose the following changes and additions, derived
> largely from Bill and James suggestions, to
> draft-ietf-oauth-v2-bearer-17.  These changes have no normative impact
> and only aim to add additional clarity and explanation the workings
> and implications of the current spec.  I'm definitely open to changes
> or improvements in the wording here (not my strong suit by any means)
> but I think it's important that these general ideas be conveyed in the
> draft.
>
>
> ==>  Insert the following text at the beginning of §2:
>
> To make a protected resource request, the client must be in possession
> of a valid bearer token. Typically a bearer token is returned to the
> client as part of an access token response as defined in OAuth 2.0
> [I-D.ietf-oauth-v2]. When the "token_type" parameter of the access
> token response is "Bearer", the string value of the "access_token"
> parameter becomes the bearer access token used to make protected
> resource requests.
>
> ==>  Change the value of the token in the Authorization header example to this:
>
> Authorization: Bearer vF_9.5dCf-t4.qbcmT_k1b
>
> ==>  Insert the following text before the last paragraph of §2.1:
>
> Note that the name b64token does not imply base64 encoding but rather
> is the name for an ABNF syntax definition from HTTP/1.1, Part 7
> [I-D.ietf-httpbis-p7-auth]. Because of its use, the "access_token"
> string value from an access token response MUST match the b64token
> ABNF so it can be used with the Bearer HTTP scheme.
>
>
> Thanks,
> Brian
>
>
>
>
> On Tue, Mar 6, 2012 at 11:45 AM, William Mills<wmills@yahoo-inc.com>  wrote:
>> Yeah, something as simple as, "Note that the name 'b64token' does not imply
>> base64 encoding, see the definition in [[INSERT REFERENCE HERE]]." would do
>> it.
>>
>> -bill
>>
>> ________________________________
>> From: Brian Campbell<bcampbell@pingidentity.com>
>> To: Mike Jones<Michael.Jones@microsoft.com>
>> Cc: oauth<oauth@ietf.org>
>> Sent: Tuesday, March 6, 2012 8:23 AM
>>
>> Subject: Re: [OAUTH-WG] question about the b64token syntax in
>> draft-ietf-oauth-v2-bearer
>>
>> Thanks Mike, I think changing the example would be helpful.
>>
>> However I think that including some text along the lines of what James
>> suggested would also be very valuable. I agree that the connection
>> between OAuth and Bearer could and should be made more explicit. And
>> that the implications of the b64token syntax, particularly on what AS
>> can use to construct ATs, could/should be made more clear.
>>
>> I can propose some specific text (building on James') if others in the WG
>> agree?
>>
>>
>> On Mon, Mar 5, 2012 at 5:32 PM, Mike Jones<Michael.Jones@microsoft.com>
>> wrote:
>>> I'm fine with changing the example to make it clearer that b64token allows
>>> a wider range of characters than just those legal for base64 and base64url
>>> encodings of data values.
>>>
>>> I'll add it to my to-do list for any additional edits for the Bearer spec.
>>>
>>>                                 -- Mike
>>>
>>> -----Original Message-----
>>> From: oauth-bounces@ietf.org [mailto:oauth-bounces@ietf.org] On Behalf Of
>>> Manger, James H
>>> Sent: Monday, March 05, 2012 3:33 PM
>>> To: Brian Campbell; oauth
>>> Subject: Re: [OAUTH-WG] question about the b64token syntax in
>>> draft-ietf-oauth-v2-bearer
>>>
>>> Brian,
>>>
>>>> On casual reading of "The OAuth 2.0 Authorization Protocol: Bearer
>>>> Tokens"* I've encountered several people (including myself) who have
>>>> made the assumption that the name b64token implies that some kind of
>>>> base64 encoding/decoding on the access token is taking place between
>>>> the client and RS.
>>>>
>>>> Digging a bit deeper in to "HTTP/1.1, part 7: Authentication"**,
>>>> however, I see that b64token is just an ABNF syntax definition
>>>> allowing for characters typically used in base64, base64url, etc.. So
>>>> the b64token doesn't define any encoding or decoding but rather just
>>>> defines what characters can be used in the part of the Authorization
>>>> header that will contain the access token.
>>>>
>>>> Do I read this correctly?
>>> Yes.
>>>
>>>> If so, I feel like some additional clarifying text in the Bearer
>>>> Tokens draft might help avoid what is (based on my small sample) a
>>>> common point of misunderstanding.
>>> Changing the example bearer token should be a simple way to avoid some
>>> confusion by showing that it does not have to be base64 encoding. How about
>>> changing:
>>>   Authorization: Bearer vF9dft4qmT
>>> to
>>>   Authorization: Bearer vF9.dft4.qmT
>>>
>>> The Bearer spec has lots of (unnecessary) text about OAuth, but doesn't
>>> quite manage to be precise about how OAuth and Bearer connect. It could
>>> explicitly state that the string value of the "access_token" member of an
>>> access token response is the bearer token. The "access_token" string value
>>> (after unescaping any JSON-escapes) MUST match the b64token ABNF so it can
>>> be used with the Bearer HTTP scheme. Such text could be put in §5.1.1 where
>>> the "Bearer" OAuth access token type is defined.
>>>
>>>
>>>> Also, does the use of b64token implicitly limit the allowed characters
>>>> that an AS can use to construct a bearer access token?
>>> Yes.
>>>
>>>
>>>> * http://tools.ietf.org/html/draft-ietf-oauth-v2-bearer-17#section-2.1
>>>> **
>>>> http://tools.ietf.org/html/draft-ietf-httpbis-p7-auth-18#section-2.1
>>> --
>>> James Manger
>>> _______________________________________________
>>> 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
>>
>>
> _______________________________________________
> OAuth mailing list
> OAuth@ietf.org
> https://www.ietf.org/mailman/listinfo/oauth