Re: [OAUTH-WG] application/x-www-form-urlencoded vs JSON (Proposal)

Pid <pid@pidster.com> Wed, 05 May 2010 22:35 UTC

Return-Path: <pid@pidster.com>
X-Original-To: oauth@core3.amsl.com
Delivered-To: oauth@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id E44FA3A6D3B for <oauth@core3.amsl.com>; Wed, 5 May 2010 15:35:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.001
X-Spam-Level:
X-Spam-Status: No, score=0.001 tagged_above=-999 required=5 tests=[BAYES_50=0.001]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id tKfAoin1ijFU for <oauth@core3.amsl.com>; Wed, 5 May 2010 15:35:42 -0700 (PDT)
Received: from mail-wy0-f172.google.com (mail-wy0-f172.google.com [74.125.82.172]) by core3.amsl.com (Postfix) with ESMTP id 52DA73A6AAD for <oauth@ietf.org>; Wed, 5 May 2010 15:35:41 -0700 (PDT)
Received: by wyb32 with SMTP id 32so1944580wyb.31 for <oauth@ietf.org>; Wed, 05 May 2010 15:35:25 -0700 (PDT)
Received: by 10.227.137.135 with SMTP id w7mr3272913wbt.10.1273098923641; Wed, 05 May 2010 15:35:23 -0700 (PDT)
Received: from Phoenix.local (cpc2-lewi13-2-0-cust269.2-4.cable.virginmedia.com [86.14.119.14]) by mx.google.com with ESMTPS id z33sm2071322wbd.19.2010.05.05.15.35.22 (version=TLSv1/SSLv3 cipher=RC4-MD5); Wed, 05 May 2010 15:35:22 -0700 (PDT)
Message-ID: <4BE1F2A1.9040707@pidster.com>
Date: Wed, 05 May 2010 23:35:13 +0100
From: Pid <pid@pidster.com>
Organization: Pidster Inc
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.1.9) Gecko/20100317 Thunderbird/3.0.4
MIME-Version: 1.0
To: "oauth@ietf.org" <oauth@ietf.org>
References: <9890332F-E759-4E63-96FE-DB3071194D84@gmail.com> <z2yf5bedd151004291440g17693f8du9e19a649bef925e4@mail.gmail.com> <w2odaf5b9571004291509x8895a73k384a4b4ddb12b794@mail.gmail.com> <20100430105935.20255m8kdythy6sc@webmail.df.eu> <90C41DD21FB7C64BB94121FBBC2E723439323D0DB0@P3PW5EX1MB01.EX1.SECURESERVER.NET> <AANLkTik3NSJynWfiNWovruPEOT2Y6G1zcWPFOaS_pHdy@mail.gmail.com> <4BE1AF25.7000308@lodderstedt.net> <AANLkTil2_9KOm1eRoC0jxvH99E55K3BEW-T5cgWLay9H@mail.gmail.com> <AANLkTilWV3VVBROXZuky5OLNzM2hz27pEqwG1l6W2Uc1@mail.gmail.com> <4BE1BB10.7060009@lodderstedt.net> <w2v77facc501005051149pca35de47tfcca515a3b557c81@mail.gmail.com>
In-Reply-To: <w2v77facc501005051149pca35de47tfcca515a3b557c81@mail.gmail.com>
X-Enigmail-Version: 1.0.1
OpenPGP: id=62590808
Content-Type: multipart/signed; micalg="pgp-sha512"; protocol="application/pgp-signature"; boundary="------------enigCBBADDB7E2976075E9676727"
Subject: Re: [OAUTH-WG] application/x-www-form-urlencoded vs JSON (Proposal)
X-BeenThere: oauth@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
Reply-To: pid@pidster.com
List-Id: OAUTH WG <oauth.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/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, 05 May 2010 22:35:44 -0000

On 05/05/2010 19:49, DeWitt Clinton wrote:
> Having written more than one compliant JSON parser myself, it is most
> certainly not "trivial", and not something that can be safely done with
> a regular expression or other hacks.
> 
> That said, it's not /hard/, and that alone is no reason not to mandate
> JSON, but I do want people to be clear about what mandating JSON means.
>  Clients will need a fully compliant parser.  Period.  If the OAuth spec
> requires JSON, then it should require it by reference to RFC 4627, not
> just by giving some examples that demonstrate the curly braces.
> 
> -DeWitt

I know it's late, but can I add my 2 cents - as a developer who'll be
implementing this?


In the original post, Dick suggested that developers were having trouble
with the URL encoding format - but I respectfully suggest that JSON is
going to be more problematic.

There's no guarantee that an external JSON parser will be available for
a given platform/language/business, (perhaps because of licensing, if
not other more technical reasons).  So that means writing one.

Writing a JSON parser just to cover the simple usage proposed won't be
too tricky, but if the JSON response is so simple why bother adding this
dependency at all?  I'm slightly baffled...


URL encoding is required for at least one flow, so IMHO it might as well
stay for the rest.  Simplicity is important.


Can someone from the pro-JSON side offer a clearer explanation as to the
benefits, so I can stop scratching my head about it all, please?



Respectfully,

Pid


> On Wed, May 5, 2010 at 11:38 AM, Torsten Lodderstedt
> <torsten@lodderstedt.net <mailto:torsten@lodderstedt.net>> wrote:
> 
>     Am 05.05.2010 20:01, schrieb Evan Gilbert:
>>
>>
>>     On Wed, May 5, 2010 at 10:59 AM, Evan Gilbert <uidude@google.com
>>     <mailto:uidude@google.com>> wrote:
>>
>>
>>
>>         On Wed, May 5, 2010 at 10:47 AM, Torsten Lodderstedt
>>         <torsten@lodderstedt.net <mailto:torsten@lodderstedt.net>> wrote:
>>
>>             Even if not supported directly by the platform there are
>>             many JSON libraries available these days.
>>
>>
>>         It's not hard to add JSON support, but it's a factor in the
>>         choice.
>>          
>>
>>
>>             http://www.json.org/ lists 3 libraries for Objective-C alone.
>>
>>             Moreover, the JSON documents we are discussing now are
>>             simple, something like
>>
>>
>>             { "access_token": "SlAV32hkKG", "expires_in": "3600",
>>             "refresh_token": "8xLOxBtZp8" }
>>
>>             Parsing such a document is not a challenge even without
>>             library support.
>>
>>
>>         Per notes above - the client needs to do understand form
>>         encoding anyway. The client needs to parse the redirect_uri
>>         and also needs to generate form encoded requests.
>>
>>
>>     Also, for the User-Agent flow, parsing potentially untrusted JSON
>>     in JavaScript is difficult. The normal path of using eval() is
>>     unsafe and leads to XSS holes - you need to run regex matcher to
>>     verify that the JSON content has no executable code.
> 
>     You are right, using eval to parse JSON is dangerous and thus as far
>     as I understand, the recommended way is to use a JSON parser (aka
>     native JSON support)?
> 
>     regards,
>     Torsten.
> 
>     _______________________________________________
>     OAuth mailing list
>     OAuth@ietf.org <mailto:OAuth@ietf.org>
>     https://www.ietf.org/mailman/listinfo/oauth
> 
> 
> 
> 
> _______________________________________________
> OAuth mailing list
> OAuth@ietf.org
> https://www.ietf.org/mailman/listinfo/oauth