Re: [OAUTH-WG] OAuth Security Topics -- Recommend authorization code instead of implicit

Neil Madden <neil.madden@forgerock.com> Fri, 30 November 2018 08:26 UTC

Return-Path: <neil.madden@forgerock.com>
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 DB3D1130934 for <oauth@ietfa.amsl.com>; Fri, 30 Nov 2018 00:26:58 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.999
X-Spam-Level:
X-Spam-Status: No, score=-1.999 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, MIME_QP_LONG_LINE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=forgerock.com
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 WBZ4qLkMWE_P for <oauth@ietfa.amsl.com>; Fri, 30 Nov 2018 00:26:56 -0800 (PST)
Received: from mail-wr1-x42c.google.com (mail-wr1-x42c.google.com [IPv6:2a00:1450:4864:20::42c]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id BFBF112F1A2 for <oauth@ietf.org>; Fri, 30 Nov 2018 00:26:55 -0800 (PST)
Received: by mail-wr1-x42c.google.com with SMTP id v6so4363896wrr.12 for <oauth@ietf.org>; Fri, 30 Nov 2018 00:26:55 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=forgerock.com; s=google; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=zAo8eSujJe0nAhPk62MrOu1TYEM7mr5uG4mZrCEoUcE=; b=LuPVxX4LnJ/07o2OZ1P8b9XGirngN1uFQgHWa7h8x5oXxd7iX/ajZLVK8BvROwthJL zFXyuMWCiRZPH4+EhA7nek1qPYQF2edJSEAWOWMAkUrCTyw6QJSo1bjchq8qIBuloYHg kQ98352aqGsOUTP51igDj6z8kxBWUstLYHCJM=
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=zAo8eSujJe0nAhPk62MrOu1TYEM7mr5uG4mZrCEoUcE=; b=oLAka/+s8A2aqcgB2hMSB0MiILL8Gf8NpMzV4FI7AWEz1llaKcRfQmRQ9d3fFE5Vmg Qb9XrIDL2Wd1ZJphAkJ29Q/jOtHKUulYTx52vfobKiRKteduFESqZvJLTQt5l2cBz3Kv J0cNwJqNsldIDqQlpfEI/NyyGiETUo+dqE60+p9yiz8QOCalSNgq90UtFOZqjmAaNswc ywlwDLpNRci9/DNpWAFHSf9mmftFEaf7m4M+pcSflSE2xLOht51xukQmxB7573lVJ7Er pZYfnSig16lFnQvM7BKWmEmXCia4DZE8sUgJ/EkVo87OSEqIJEfvaSNMnvNVbPpcubR+ 92FQ==
X-Gm-Message-State: AA+aEWaFgQXZgyFnrms4bQTlumrYgFBrCnbwGSon1nz9nowrxQYmrWoh B+ge6SYMtSlRmn11W56yNGlk8iPkhV47l9Q6gr6UVo8PbP8O0ACOlEPuouwzdFxKAkDjfUE3883 wXLdd4gM+MT+rAR5mF/T8g8WEncV1Wakgm4j2fsrHI5IqS4Uky/u/oDFxuUqwLPg=
X-Google-Smtp-Source: AFSGD/WjfCyTwpiJgvq8d6ZG9JeDkDK++3pBdi7eILThgLnBmM8MZ07Yno9VAHbDx2h3RHVTa7SB7Q==
X-Received: by 2002:adf:f691:: with SMTP id v17mr3928134wrp.114.1543566413678; Fri, 30 Nov 2018 00:26:53 -0800 (PST)
Received: from ?IPv6:2a01:4c8:c:5949:601e:f94b:ee1c:c567? ([2a01:4c8:c:5949:601e:f94b:ee1c:c567]) by smtp.gmail.com with ESMTPSA id x15sm3733749wrs.27.2018.11.30.00.26.52 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 30 Nov 2018 00:26:52 -0800 (PST)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (1.0)
From: Neil Madden <neil.madden@forgerock.com>
X-Mailer: iPhone Mail (16B92)
In-Reply-To: <FE51CE20-7A49-4A13-A180-6A7C481F3965@lodderstedt.net>
Date: Fri, 30 Nov 2018 08:26:51 +0000
Cc: IETF oauth WG <oauth@ietf.org>, Daniel Fett <fett@danielfett.de>
Content-Transfer-Encoding: quoted-printable
Message-Id: <B895EDA6-AC5A-4FC8-AC22-D1E420CC21C2@forgerock.com>
References: <VI1PR0801MB211266BA6F6E06FFB3081425FAD80@VI1PR0801MB2112.eurprd08.prod.outlook.com> <CAD9ie-v3onmKc498cg_-a0AD58ZV=aZANtz=UV+Q0f=9N3nSzQ@mail.gmail.com> <OSBPR01MB2869E83F37046C7FCD4463DDF9D10@OSBPR01MB2869.jpnprd01.prod.outlook.com> <9FF3F589-0423-4CBC-B323-481F771D097C@lodderstedt.net> <OSBPR01MB28690F77DFFB2A85BDB83FBAF9D10@OSBPR01MB2869.jpnprd01.prod.outlook.com> <D6C66E6A-687B-4997-B830-980BE25994C2@lodderstedt.net> <CAANoGhL9aD75AV9QQRdeGE1=4ynjTnULNVr0PXXvt20ipsb4Rw@mail.gmail.com> <FE51CE20-7A49-4A13-A180-6A7C481F3965@lodderstedt.net>
To: Torsten Lodderstedt <torsten@lodderstedt.net>
Archived-At: <https://mailarchive.ietf.org/arch/msg/oauth/h4OQ9WsGTMjBWe-TTgrUYNTWV2I>
Subject: Re: [OAUTH-WG] OAuth Security Topics -- Recommend authorization code instead of implicit
X-BeenThere: oauth@ietf.org
X-Mailman-Version: 2.1.29
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, 30 Nov 2018 08:26:59 -0000

+1 from me too. 

> On 29 Nov 2018, at 18:06, Torsten Lodderstedt <torsten@lodderstedt.net> wrote:
> 
> Hi all, 
> 
> based on your feedback on the list and off list, Daniel and I polished the text. That’s our proposal:
> 
> —
> In order to avoid these issues, clients MUST NOT use the implicit
> grant (response type "token") or any other response type issuing access 
> tokens in the authorization response, such as "token id_token" and "code token id_token“, 
> unless the issued access tokens are sender-constrained and access token injection in 
> the authorization response is prevented. 
> —
> 
> Explantation: 
> - we wanted to have the right balance between a generic definition of the response types we do not recommend/allow to be used and a concrete/actionable list of the affected response types. 
> - we changed from SHOULD NOT to MUST NOT as suggested by Nat and supported by William
> 
> We look forward to seeing your feedback.
> 
> kind regards,
> Torsten.  
> 
>> Am 29.11.2018 um 15:15 schrieb John Bradley <ve7jtb@ve7jtb.com>:
>> 
>> I am ok with that.  
>> 
>>>> On Wed, Nov 28, 2018, 8:03 PM Torsten Lodderstedt <torsten@lodderstedt.net wrote:
>>> 
>>> Am 28.11.2018 um 23:50 schrieb n-sakimura <n-sakimura@nri.co.jp>:
>>> 
>>> That works.
>> 
>> Good!
>> 
>> I just realized this text has an issue with „token“ (only). It would allow „token“ to be used if the token would sender constrained. This completely ignores the fact implicit also shall be abandoned because of its vulnerability for access token injection. 
>> 
>> I therefore propose a modified text: 
>> 
>>   In order to avoid these issues, Clients SHOULD NOT use the implicit
>>   grant. Furthermore, clients SHOULD only use other response types causing the authorization server to
>>   issue an access token in the authorization response, if the particular response type detects access token 
>>   injection and the issued access tokens are sender-constrained.
>> 
>> Or we just state:
>> 
>>  In order to avoid these issues, Clients SHOULD NOT use the response type „token". The response types
>> „token id_token“ and „code token id_token“ SOULD NOT be used, if the issued access tokens are not 
>> sender-constrained.
>> 
>>> 
>>> In fact, I would further go and say MUST NOT but that probably is too much for a security consideration.
>>> 
>> 
>> Mike suggested to go with a SHOULD NOT to get the message out but give implementors time to move/change.
>> 
>>> Best,
>>> 
>>> Nat
>>> 
>>> Nat Sakimura / n-sakimura@nri.co.jp / +81-90-6013-6276
>>> 
>>> このメールには、本来の宛先の方のみに限定された機密情報が含まれている場合がございます。お心あたりのない場合は、誠に申し訳ございませんが、送信者までお知らせ頂き、また受信されたメールは削除してくださいますようお願い申し上げます。
>>> 
>>> PLEASE READ :This e-mail is confidential and intended for the named recipient only.
>>> If you are not an intended recipient, please notify the sender and delete this e-mail.
>>> 
>>> 差出人: Torsten Lodderstedt <torsten@lodderstedt.net>
>>> 送信日時: 水曜日, 11月 28, 2018 11:38 午後
>>> 宛先: n-sakimura
>>> Cc: Dick Hardt; Hannes Tschofenig; oauth@ietf.org
>>> 件名: Re: [OAUTH-WG] OAuth Security Topics -- Recommend authorization code instead of implicit
>>> 
>>> Hi Nat, 
>>> 
>>>> Am 28.11.2018 um 21:10 schrieb n-sakimura <n-sakimura@nri.co.jp>:
>>>> 
>>>> I would support
>>>> 
>>>> 1) clearly defining Implicit as the flow that returns access token from the authorization endpoint ( some people confuses implicit as the flow that returns ID Token in the front channel)
>>> 
>>> That’s the current text: 
>>> 
>>> In order to avoid these issues, Clients SHOULD NOT use the implicit
>>>   grant or any other response type causing the authorization server to
>>>   issue an access token in the authorization response.
>>> 
>>> What would you like to modify? 
>>> 
>>>> 
>>>> 2) Banning the returning of the access token that are not sender constrained from the authorization endpoint
>>> 
>>> In order to avoid these issues, Clients SHOULD NOT use the implicit
>>>   grant or any other response type causing the authorization server to
>>>   issue an access token in the authorization response, if this access tokens is not sender-constraint.
>>> 
>>> What about this?
>>> 
>>> kind regards,
>>> Torsten.
>>> 
>>>> 
>>>> Best,
>>>> 
>>>> Nat
>>>> 
>>>> 
>>>> Outlook for iOS を入手
>>>> 
>>>> 差出人: OAuth <oauth-bounces@ietf.org> (Dick Hardt <dick.hardt@gmail.com> の代理)
>>>> 送信日時: 水曜日, 11月 28, 2018 8:58 午後
>>>> 宛先: Hannes Tschofenig
>>>> Cc: oauth@ietf.org
>>>> 件名: Re: [OAUTH-WG] OAuth Security Topics -- Recommend authorization code instead of implicit
>>>> 
>>>> +1
>>>> 
>>>> While there are various mechanisms to alleviate some of the issues of implicit, I don't think we can recommend specifics, and there may be future ones in the future. I think we all agree that implicit without any mitigation is problematic. 
>>>> 
>>>> How about we recommend against using implicit alone? 
>>>> 
>>>> 
>>>> On Mon, Nov 19, 2018 at 2:34 AM Hannes Tschofenig <Hannes.Tschofenig@arm.com> wrote:
>>>> Hi all,
>>>> 
>>>> The authors of the OAuth Security Topics draft came to the conclusion that it is not possible to adequately secure the implicit flow against token injection since potential solutions like token binding or JARM are in an early stage of adoption. For this reason, and since CORS allows browser-based apps to send requests to the token endpoint, Torsten suggested to use the authorization code instead of the implicit grant in call cases in his presentation (see https://datatracker.ietf.org/meeting/103/materials/slides-103-oauth-sessb-draft-ietf-oauth-security-topics-01).
>>>> 
>>>> A hum in the room at IETF#103 concluded strong support for his recommendations. We would like to confirm the discussion on the list.
>>>> 
>>>> Please provide a response by December 3rd.
>>>> 
>>>> Ciao
>>>> 
>>>> Hannes & Rifaat
>>>> 
>>>> 
>>>> 
>>>> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
>>>> _______________________________________________
>>>> 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
>