Update Services/Soundcloud.php #13

Merged
merged 1 commit into from Jan 16, 2013

Conversation

Projects
None yet
2 participants
Contributor

hmanprod commented Jan 16, 2013

I ve add some fix in order to handle CURLOPT_HEADER correctly.
Perhaps it will be helpfull.
In my case curl_getinfo was returning a bad header_size so i has a problem to get use the json correctly.

Perhaps there change may to be to apply on this part to

$this->_lastHttpResponseHeaders = $this->_parseHttpHeaders(
substr($data, 0, $info['header_size'])
);

and use CURL_NOBODY options instead

@hmanprod hmanprod Update Services/Soundcloud.php
I ve add some fix in order to handle CURLOPT_HEADER correctly.
Perhaps it will be helpfull. 
In my case curl_getinfo was returning a bad header_size so i has a problem to get use the json correctly.
3aaeb49

@mptre mptre pushed a commit that referenced this pull request Jan 16, 2013

Anton Lindqvist Merge pull request #13 from hmanprod/patch-1
Update Services/Soundcloud.php
8070975

@mptre mptre merged commit 8070975 into mptre:master Jan 16, 2013

Owner

mptre commented Jan 16, 2013

Thx, merged!

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