ckan / ckanext-archiver

Archive CKAN resources
MIT License
22 stars 46 forks source link

Headers is not JSON serializable #9

Closed KrzysztofMadejski closed 9 years ago

KrzysztofMadejski commented 9 years ago

In https://github.com/ckan/ckanext-archiver/blob/master/ckanext/archiver/tasks.py#L365 return json.dumps(headers)

throws {blabla} is not JSON serializable

headers is instance of CaseInsensitiveDict

Changing this line to: json.dumps(dict(headers)) resolve this issue.

davidread commented 9 years ago

Thanks @KrzysztofMadejski - looks like you have a newer version of 'requests' which caused this. Now fixed.