Re: [Gen-art] [OAUTH-WG] Gen-ART Telechat review of draft-ietf-oauth-v2-bearer-22.txt

Alexey Melnikov <alexey.melnikov@isode.com> Wed, 18 July 2012 11:37 UTC

Return-Path: <alexey.melnikov@isode.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2372421F86B7; Wed, 18 Jul 2012 04:37:04 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.996
X-Spam-Level:
X-Spam-Status: No, score=-102.996 tagged_above=-999 required=5 tests=[AWL=-0.397, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
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 UpN8SmL9icwr; Wed, 18 Jul 2012 04:37:03 -0700 (PDT)
Received: from statler.isode.com (statler.isode.com [62.3.217.254]) by ietfa.amsl.com (Postfix) with ESMTP id 3631A21F869D; Wed, 18 Jul 2012 04:37:03 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1342611471; d=isode.com; s=selector; i=@isode.com; bh=CXrghOeFzyerB+sc1r3pz2TSl+uOF5/2rE3b8GzbZCo=; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version: In-Reply-To:References:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description; b=Gd9QP7M5IfM4PTpUgw4rxZBV9kmfjFHuvdfQtJW3oNxIBgjAukFuImZoeFKctsNwHPqArh lpMhhbZZ/DBZ3QH0+ZocI4vt9BiYI/pXY3c6hcwoNYHtYL1QG2SzwrdLTYgppWb/MkA9ZP uH/VS32pXHE/fRIqvhQR3BJaT7ULcOY=;
Received: from [172.16.1.29] (shiny.isode.com [62.3.217.250]) by statler.isode.com (submission channel) via TCP with ESMTPSA id <UAagDAAwUxMD@statler.isode.com>; Wed, 18 Jul 2012 12:37:51 +0100
X-SMTP-Protocol-Errors: PIPELINING
Message-ID: <5006A010.60408@isode.com>
Date: Wed, 18 Jul 2012 12:37:52 +0100
From: Alexey Melnikov <alexey.melnikov@isode.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:13.0) Gecko/20120614 Thunderbird/13.0.1
To: Mike Jones <Michael.Jones@microsoft.com>
References: <4E1F6AAD24975D4BA5B16804296739436673769B@TK5EX14MBXC285.redmond.corp.microsoft.com>
In-Reply-To: <4E1F6AAD24975D4BA5B16804296739436673769B@TK5EX14MBXC285.redmond.corp.microsoft.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: "draft-ietf-oauth-v2-bearer.all@tools.ietf.org" <draft-ietf-oauth-v2-bearer.all@tools.ietf.org>, Julian Reschke <julian.reschke@gmx.de>, General Area Review Team <gen-art@ietf.org>, The IESG <iesg@ietf.org>, "oauth@ietf.org" <oauth@ietf.org>, Stephen Farrell <stephen.farrell@cs.tcd.ie>
Subject: Re: [Gen-art] [OAUTH-WG] Gen-ART Telechat review of draft-ietf-oauth-v2-bearer-22.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Jul 2012 11:37:04 -0000

On 17/07/2012 19:01, Mike Jones wrote:
> You should actually probably make that name change request to the HTTPbis working group.  I suspect that if they decide to change the name, that we could direct the RFC editor to make the same name change as HTTPbis does.
It looks like the discussion of changing this in HTTPBIS is in progress 
now.
> 				-- Mike
>
> -----Original Message-----
> From: Alexey Melnikov [mailto:alexey.melnikov@isode.com]
> Sent: Tuesday, July 17, 2012 10:58 AM
> To: Mike Jones
> Cc: Julian Reschke; The IESG; General Area Review Team; oauth@ietf.org; draft-ietf-oauth-v2-bearer.all@tools.ietf.org; Stephen Farrell
> Subject: Re: [Gen-art] [OAUTH-WG] Gen-ART Telechat review of draft-ietf-oauth-v2-bearer-22.txt
>
> On 17/07/2012 18:15, Mike Jones wrote:
>> For clarity of discussion, the definition in question is:
>>        b64token    = 1*( ALPHA / DIGIT /
>>                          "-" / "." / "_" / "~" / "+" / "/" ) *"="
>>
>> Note that b64token is a liberal syntax intended to permit base64 encoded content (hence the inclusion of the "+" and "/" characters and the optional trailing "=" characters), base64url encoded content (hence the inclusion of the "-" and "_" characters) and other URL-safe productions (hence the inclusion of the "." and "~" characters).
>>
>> Its use is definitely not intended to be restricted to base64 encoded content, per RFC 4648. If it were so restricted (by not allowing ".", for instance), this would exclude the use of JWTs as bearer tokens, for instance, which is something we *definitely* want to allow.
>>
>> As a result, I don't think adding a reference to RFC 4648 is either necessary or appropriate.
> In this case, can you please rename the production to something which is clearly not a base64 string.
>
>> Julian may be able to provide more background.
>>
>> 				Best wishes,
>> 				-- Mike
>>
>> -----Original Message-----
>> From: Alexey Melnikov [mailto:alexey.melnikov@isode.com]
>> Sent: Tuesday, July 17, 2012 10:02 AM
>> To: Julian Reschke; Mike Jones
>> Cc: The IESG; General Area Review Team; oauth@ietf.org;
>> draft-ietf-oauth-v2-bearer.all@tools.ietf.org; Stephen Farrell
>> Subject: Re: [Gen-art] [OAUTH-WG] Gen-ART Telechat review of
>> draft-ietf-oauth-v2-bearer-22.txt
>>
>> On 17/07/2012 17:40, Julian Reschke wrote:
>>> On 2012-07-17 18:10, Mike Jones wrote:
>>>> FYI, the b64 token definition is identical to the one in
>>>> draft-ietf-httpbis-p7-auth-20.  If it works there, it should work
>>>> for OAuth Bearer.
>>>> ...
>>> +1; not every constraint needs to be expressed in the ABNF. "b64token"
>>> is here so recipients can parse the header field; it's up to the auth
>>> scheme to state what the addition constraints are; and that can
>>> happen in prose.
>> I didn't say that it has to be expressed in ABNF (although I obviously wouldn't mind). I would like an ABNF comment pointing to the document which defines base64.