Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update requests requirement to >= 0.13.0 #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update requests requirement to >= 0.13.0 #26

wants to merge 1 commit into from

Conversation

kelvinn
Copy link

@kelvinn kelvinn commented Jun 25, 2014

Response.json property was added to requests in 0.12.1, so any call to the CKAN api will fail if requests is < 0.13.0

Response.json property was added to requests in 0.12.1
@rshk
Copy link
Contributor

rshk commented Jul 9, 2014

Is requests <0.13 still around? I considered it old enough to be long forgotten, but maybe it's still packaged in some distro..? If that's the case, it might be worth using a "fallback" json.loads(response.text) as well..

@kelvinn
Copy link
Author

kelvinn commented Jul 9, 2014

Hi Sam,

Somehow I was hitting this error on Ubuntu 12.04, so it must ship with an
older version.

Regards,

Kelvin
04 2499 0842
On 9 Jul 2014 22:24, "Samuele Santi" notifications@github.com wrote:

Is requests <0.13 still around? I considered it old enough to be long
forgotten, but maybe it's still packaged in some distro..? If that's the
case, it might be worth using a "fallback" json.loads(response.text) as
well..


Reply to this email directly or view it on GitHub
#26 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants