Added spaces to authorization signature. #107

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@alexindigo

There was problem with 500px API, which couldn't parse auth headers with no spaces between elements.

node-oauth header:

authorization: 'OAuth oauth_callback="...",oauth_consumer_key="...",oauth_nonce="...",oauth_signature_method="...",oauth_timestamp="...",oauth_version="...",oauth_signature="..."'

What 500px server expecting:

authorization: 'OAuth oauth_callback="...", oauth_consumer_key="...", oauth_nonce="...", oauth_signature_method="...", oauth_timestamp="...", oauth_version="...", oauth_signature="..."'
@ciaranj
Owner
ciaranj commented Jul 28, 2012

Hiyah, this duplicates Pull Request 105, which I'll merge now if thats ok (that solution is backwards-safe, whereas this might affect existing consumers of the API)

@ciaranj ciaranj closed this Jul 28, 2012
@alexindigo

Sure, any solution would. Thanx a lot.

PS. Funny to see similar pull requests around same time :)

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