Re: [OAUTH-WG] nits about definition of using form parameters

Eran Hammer <eran@hueniverse.com> Mon, 18 June 2012 22:17 UTC

Return-Path: <eran@hueniverse.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 C135A11E80D9 for <oauth@ietfa.amsl.com>; Mon, 18 Jun 2012 15:17:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.98
X-Spam-Level:
X-Spam-Status: No, score=-1.98 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_SORBS_WEB=0.619]
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 LEJUqdMDGGVE for <oauth@ietfa.amsl.com>; Mon, 18 Jun 2012 15:17:42 -0700 (PDT)
Received: from p3plex2out04.prod.phx3.secureserver.net (p3plex2out04.prod.phx3.secureserver.net [184.168.131.18]) by ietfa.amsl.com (Postfix) with ESMTP id 0A44F11E80C1 for <oauth@ietf.org>; Mon, 18 Jun 2012 15:17:42 -0700 (PDT)
Received: from P3PWEX2HT002.ex2.secureserver.net ([184.168.131.10]) by p3plex2out04.prod.phx3.secureserver.net with bizsmtp id PyHh1j0030Dcg9U01yHhTz; Mon, 18 Jun 2012 15:17:41 -0700
Received: from P3PWEX2MB008.ex2.secureserver.net ([169.254.8.66]) by P3PWEX2HT002.ex2.secureserver.net ([184.168.131.10]) with mapi id 14.02.0247.003; Mon, 18 Jun 2012 15:17:41 -0700
From: Eran Hammer <eran@hueniverse.com>
To: Mike Jones <Michael.Jones@microsoft.com>, Julian Reschke <julian.reschke@gmx.de>, "OAuth WG (oauth@ietf.org)" <oauth@ietf.org>
Thread-Topic: [OAUTH-WG] nits about definition of using form parameters
Thread-Index: AQHNSHuYNn5GyPRYwECfY+U4MeoAcZcBHIEA//+SpkA=
Date: Mon, 18 Jun 2012 22:17:40 +0000
Message-ID: <0CBAEB56DDB3A140BA8E8C124C04ECA201077978@P3PWEX2MB008.ex2.secureserver.net>
References: <4FD70812.6040108@gmx.de> <4E1F6AAD24975D4BA5B16804296739436655A58E@TK5EX14MBXC283.redmond.corp.microsoft.com>
In-Reply-To: <4E1F6AAD24975D4BA5B16804296739436655A58E@TK5EX14MBXC283.redmond.corp.microsoft.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [80.36.76.35]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Subject: Re: [OAUTH-WG] nits about definition of using form parameters
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: Mon, 18 Jun 2012 22:17:42 -0000

I believe the UTF-8 piece came from Brian Eaton a while back because of some security issue identified at Google.

EH

> -----Original Message-----
> From: oauth-bounces@ietf.org [mailto:oauth-bounces@ietf.org] On Behalf
> Of Mike Jones
> Sent: Monday, June 18, 2012 2:48 PM
> To: Julian Reschke; OAuth WG (oauth@ietf.org)
> Subject: Re: [OAUTH-WG] nits about definition of using form parameters
> 
> Hi Julian,
> 
> Both the Core and Bearer specs already reference W3C.REC-html401-
> 19991224 for the definition of application/x-www-form-urlencoded.
> 
> I'll leave it up to others to comment on whether the ;charset=UTF-8
> parameter is correct or not.
> 
> 				-- Mike
> 
> -----Original Message-----
> From: oauth-bounces@ietf.org [mailto:oauth-bounces@ietf.org] On Behalf
> Of Julian Reschke
> Sent: Tuesday, June 12, 2012 2:13 AM
> To: OAuth WG (oauth@ietf.org)
> Subject: [OAUTH-WG] nits about definition of using form parameters
> 
> Hi there,
> 
> re <http://tools.ietf.org/html/draft-ietf-oauth-v2-27#section-4.3.2>:
> 
> This needs a normative reference to a spec that defines the application/x-
> www-form-urlencoded media type (such as
> <http://www.w3.org/TR/html5/iana.html#application-x-www-form-
> urlencoded>).
> 
> Looking at the media type definition I don't see any mention of a charset
> parameter, so the example probably is wrong. See also
> <http://www.w3.org/TR/html5/form-submission.html#url-encoded-form-
> data>:
> 
> "Note: Parameters on the application/x-www-form-urlencoded MIME type
> are ignored. In particular, this MIME type does not support the charset
> parameter."
> 
> I would also advise to change
> 
>     The client makes a request to the token endpoint by adding the
>     following parameters using the "application/x-www-form-urlencoded"
>     format in the HTTP request entity-body:
> 
>     grant_type
>           REQUIRED.  Value MUST be set to "password".
>     username
>           REQUIRED.  The resource owner username, encoded as UTF-8.
>     password
>           REQUIRED.  The resource owner password, encoded as UTF-8.
>     scope
>           OPTIONAL.  The scope of the access request as described by
>           Section 3.3.
> 
> to
> 
> 
>     The client makes a request to the token endpoint by sending the
>     following parameters using the "application/x-www-form-urlencoded"
>     format (Section 4.10.22.5 of [WD-html5-20120329]) and a
>     character encoding of "UTF-8" in the HTTP request entity-body:
> 
>     grant_type
>           REQUIRED.  Value MUST be set to "password".
>     username
>           REQUIRED.  The resource owner username.
>     password
>           REQUIRED.  The resource owner password.
>     scope
>           OPTIONAL.  The scope of the access request as described by
>           Section 3.3.
> 
> Finally, it would be good if the example used characters that require escaping
> in the body, such as "&", "%", or non-ASCII characters.
> 
> (similar nits apply to other sections using form encoding)
> 
> Best regards, Julian
> _______________________________________________
> 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