POST requests should have query params in the body #31

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
6 participants
@ammmir

ammmir commented Apr 4, 2011

Some sites (like Instagram) will ignore the query string parameters for POST requests, as they're expecting them in the request body.

@langalex

This comment has been minimized.

Show comment Hide comment
@langalex

langalex Apr 21, 2011

+1

+1

@amcjen

This comment has been minimized.

Show comment Hide comment
@amcjen

amcjen May 20, 2011

I made a similar patch for Instagram as well, which solved this same issue. +1 for a pull request for this fix.

amcjen commented May 20, 2011

I made a similar patch for Instagram as well, which solved this same issue. +1 for a pull request for this fix.

@hsztul

This comment has been minimized.

Show comment Hide comment
@hsztul

hsztul Jul 18, 2011

+1

hsztul commented Jul 18, 2011

+1

@langalex

This comment has been minimized.

Show comment Hide comment
@langalex

langalex Jul 28, 2011

any reason this is not being integrated? i just wasted an hour implementing it myself because i forgot about this fork.

any reason this is not being integrated? i just wasted an hour implementing it myself because i forgot about this fork.

@kytrinyx

This comment has been minimized.

Show comment Hide comment
@kytrinyx

kytrinyx Sep 3, 2011

+1

kytrinyx commented Sep 3, 2011

+1

@ciaranj

This comment has been minimized.

Show comment Hide comment
@ciaranj

ciaranj Apr 24, 2012

Owner

Can anyone confirm if this is still an issue ? I was about to merge this in, when I noted it conflicts with some work around always POSTing to retrieve the access token.. it may no longer be an issue ?

Owner

ciaranj commented Apr 24, 2012

Can anyone confirm if this is still an issue ? I was about to merge this in, when I noted it conflicts with some work around always POSTing to retrieve the access token.. it may no longer be an issue ?

@langalex

This comment has been minimized.

Show comment Hide comment
@langalex

langalex Apr 25, 2012

looks like this line solves it already.

looks like this line solves it already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment