oauth php & query parameter arrays Options

niyomja / oauth

Automatically exported from code.google.com/p/oauth

0 stars 0 forks source link

oauth php & query parameter arrays Options #89

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

If you perform a request with query parameter array(s) (eg.
http://localhost/?test[1][2]=3) the OAuth PHP lib (svn r908) neglects the
array key(s).

More information at:
http://groups.google.com/group/oauth/browse_thread/thread/a7bd7db14f9e3a5a

Attached patch fixes the issue.

Original issue reported on code.google.com by rgar...@gmail.com on 9 Mar 2009 at 3:27

Attachments:

oauth_php-r908-param_array_fix

GoogleCodeExporter commented 9 years ago

Original comment by rgar...@gmail.com on 9 Mar 2009 at 3:30

Changed state: New
Added labels: Lib-PHP, Type-Defect

GoogleCodeExporter commented 9 years ago

now with good attachment ;)

Original comment by rgar...@gmail.com on 9 Mar 2009 at 3:31

Attachments:

oauth_php-r908-param_array_fix.diff

GoogleCodeExporter commented 9 years ago

I have coded up a branch that solves this issue (I believe), by parsing 
directly from the query-string / raw-post-
data and not storing the parameters as N-level deep arrays..

Please review
http://code.google.com/p/oauth/source/branch?spec=issue104

Original comment by morten.f...@gmail.com on 16 May 2009 at 2:52

Changed state: Started

GoogleCodeExporter commented 9 years ago

The changes has been reviewed by termie, and has been merged into the main 
library. Could you verify this fixes 
your issues?

Original comment by morten.f...@gmail.com on 18 May 2009 at 6:02

Changed state: Fixed