Re: [OAUTH-WG] Second OAuth 2.0 Mix-Up Mitigation Draft

David Waite <david@alkaline-solutions.com> Fri, 22 January 2016 17:54 UTC

Return-Path: <david@alkaline-solutions.com>
X-Original-To: oauth@ietfa.amsl.com
Delivered-To: oauth@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B519B1B2B17 for <oauth@ietfa.amsl.com>; Fri, 22 Jan 2016 09:54:11 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.898
X-Spam-Level:
X-Spam-Status: No, score=-1.898 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RP_MATCHES_RCVD=-0.001, SPF_FAIL=0.001, SPF_HELO_FAIL=0.001] autolearn=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id xfuBojco2BKO for <oauth@ietfa.amsl.com>; Fri, 22 Jan 2016 09:54:09 -0800 (PST)
Received: from alkaline-solutions.com (lithium5.alkaline-solutions.com [IPv6:2600:3c00::f03c:91ff:fe93:6974]) by ietfa.amsl.com (Postfix) with ESMTP id 7DB941B2B10 for <oauth@ietf.org>; Fri, 22 Jan 2016 09:54:09 -0800 (PST)
Received: from [192.168.16.27] (c-50-155-144-64.hsd1.co.comcast.net [50.155.144.64]) by alkaline-solutions.com (Postfix) with ESMTPSA id 800C7315B1; Fri, 22 Jan 2016 17:54:08 +0000 (UTC)
Content-Type: multipart/alternative; boundary="Apple-Mail=_3C380A9B-6C21-4D72-8ACE-AB43E8E07788"
Mime-Version: 1.0 (Mac OS X Mail 9.2 \(3112\))
From: David Waite <david@alkaline-solutions.com>
In-Reply-To: <56A260B6.4060302@aol.com>
Date: Fri, 22 Jan 2016 10:54:07 -0700
Message-Id: <478C8683-0E47-43E7-A7F3-75F89AB1DCDA@alkaline-solutions.com>
References: <BY2PR03MB442662C73E3904E73D9B9EFF5C30@BY2PR03MB442.namprd03.prod.outlook.com> <2EB9855D-BAB2-4B90-B649-F1B24B8834EC@alkaline-solutions.com> <DC542E43-8D0B-4728-AB46-AC70F190D8AD@ve7jtb.com> <80218A07-9D37-42A3-B90A-56AEDBC3A86C@alkaline-solutions.com> <CAANoGhJT8j0ZO09kBJHpg+QVkFi-RuoA42Rb+zd698aC9AmLUw@mail.gmail.com> <CAAP42hCzVtXiECAZzaU-YEQe01BPbOc6OTL-u=PLQkxdsYeSfQ@mail.gmail.com> <56A260B6.4060302@aol.com>
To: George Fletcher <gffletch@aol.com>
X-Mailer: Apple Mail (2.3112)
Archived-At: <http://mailarchive.ietf.org/arch/msg/oauth/2xo9A3XNlaO6kq3Vu8KRyIZDKwU>
Cc: IETF oauth WG <oauth@ietf.org>
Subject: Re: [OAUTH-WG] Second OAuth 2.0 Mix-Up Mitigation Draft
X-BeenThere: oauth@ietf.org
X-Mailman-Version: 2.1.15
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: <https://mailarchive.ietf.org/arch/browse/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: Fri, 22 Jan 2016 17:54:11 -0000

It’s pronounced FronkenSTEEN-ian.

-DW

> On Jan 22, 2016, at 10:02 AM, George Fletcher <gffletch@aol.com> wrote:
> 
> "Frankensteinian Amalgamation" -- David Waite
> 
> I like it! :)
> 
> On 1/22/16 8:11 AM, William Denniss wrote:
>> +1 ;)
>> On Fri, Jan 22, 2016 at 8:45 PM John Bradley < <mailto:ve7jtb@ve7jtb.com>ve7jtb@ve7jtb.com <mailto:ve7jtb@ve7jtb.com>> wrote:
>> Perhaps Frankenstein response is a better name than cut and paste attack.
>> 
>> John B.  
>> 
>> On Jan 22, 2016 1:22 AM, "David Waite" <david@alkaline-solutions.com <mailto:david@alkaline-solutions.com>> wrote:
>>> On Jan 21, 2016, at 2:50 PM, John Bradley < <mailto:ve7jtb@ve7jtb.com>ve7jtb@ve7jtb.com <mailto:ve7jtb@ve7jtb.com>> wrote:
>>> 
>>> In that case you probably would put a hash of the state in the code to manage size.  The alg would be up to the AS, as long as it used the same hash both places it would work.
>> Yes, true. 
>>> 
>>> Sending the state to the token endpoint is like having nonce and c_hash in the id_token, it binds the issued code to the browser instance.
>> I think I understand what you are saying. Someone won’t be able to frankenstein up a state and a token from two different responses from an AS, and have a client successfully fetch an access token based on the amalgamation.
>>  
>>> This protects against codes that leak via redirect uri pattern matching. failures etc.  It prevents an attacker from being able to replay a code from a different browser.
>> Yes, if a party intercepts the redirect_url, or the AS fails to enforce one time use (which even for a compliant implementation could just mean the attacker was faster than the state propagated within the AS)
>> 
>> Makes sense. Thanks John.
>> 
>> -DW
>> 
>>> If the client implements the other mitigations on the authorization endpoint, then it wouldn't be leaking the code via the token endpoint. 
>>> 
>>> The two mitigations are for different attacks, however some of the attacks combined both vulnerabilities.
>>> 
>>> Sending the iss and client_id is enough to stop the confused client attacks, but sending state on its own would not have stopped all of them.
>>> 
>>> We discussed having them in separate drafts, and may still do that.   However for discussion having them in one document is I think better in the short run.
>>> 
>>> John B.
>>> 
>>>> On Jan 21, 2016, at 4:48 PM, David Waite <david@alkaline-solutions.com <mailto:david@alkaline-solutions.com>> wrote:
>>>> 
>>>> Question: 
>>>> 
>>>> I understand how “iss" helps mitigate this attack (client knows response was from the appropriate issuer and not an attack where the request was answered by another issuer). 
>>>> 
>>>> However, how does passing “state” on the authorization_code grant token request help once you have the above in place? Is this against some alternate flow of this attack I don’t see, or is it meant to mitigate some entirely separate attack?
>>>> 
>>>> If one is attempting to work statelessly (e.g. your “state” parameter is actual state and not just a randomly generated value), a client would have always needed some way to differentiate which issuer the authorization_code grant token request would be sent to.
>>>> 
>>>> However, if an AS was treating “code” as a token (for instance, encoding: client, user, consent time and approved scopes), the AS now has to include the client’s state as well. This would effectively double (likely more with encoding) the state sent in the authorization response back to the client redirect URL, adding more pressure against maximum URL sizes.
>>>> 
>>>> -DW
>> _______________________________________________
>> OAuth mailing list
>> OAuth@ietf.org <mailto:OAuth@ietf.org>
>> https://www.ietf.org/mailman/listinfo/oauth <https://www.ietf.org/mailman/listinfo/oauth>
>> 
>> 
>> _______________________________________________
>> OAuth mailing list
>> OAuth@ietf.org <mailto:OAuth@ietf.org>
>> https://www.ietf.org/mailman/listinfo/oauth <https://www.ietf.org/mailman/listinfo/oauth>
> 
> -- 
> Chief Architect                   
> Identity Services Engineering     Work: george.fletcher@teamaol.com <mailto:george.fletcher@teamaol.com>
> AOL Inc.                          AIM:  gffletch
> Mobile: +1-703-462-3494           Twitter: http://twitter.com/gffletch <http://twitter.com/gffletch>
> Office: +1-703-265-2544           Photos: http://georgefletcher.photography <http://georgefletcher.photography/>