I was running into a problem with flickrapi using oauth_connect's DrupalOAuthClient get function - there's a place where it uses http_build_query without providing explicit options and arg_separator was getting a bad default (& instead of &) - this caused requests to be malformed.

I'll attach a patch that provides a default set of arguments to http_build_query as used in other places in this module.

CommentFileSizeAuthor
#1 oauth-http_build_query_params-1982862-1.patch410 bytesbc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bc’s picture

Status: Active » Needs review
FileSize
410 bytes

patch attached.

bc’s picture

just rolled out another site with oauth, and ran into this bug again. i'd love to see this pulled into the main branch! :)

bc’s picture

Issue summary: View changes

& instead of &