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.

Sign up for GitHub

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

Jump to bottom

Key exception on resource proxy on CKAN < 2.3 #2425

Closed
amercader opened this issue May 11, 2015 · 1 comment
Closed

Key exception on resource proxy on CKAN < 2.3 #2425

amercader opened this issue May 11, 2015 · 1 comment
Labels
zBackport 2.0.7 zBackport 2.1.5 zBackport 2.2.3

Comments

@amercader
Copy link
Member

Because the headers are accessed like these and modern requests versions will fail if the header is not present:

cl = r.headers['content-length']
@amercader
Copy link
Member Author

Backported

amercader added a commit that referenced this issue May 11, 2015
@amercader
[#2425] Key error on resource proxy
78e9fd0 
Old requests versions use to return None if a header was not present
amercader added a commit that referenced this issue May 11, 2015
@amercader
[#2425] Key error on resource proxy
88d0e4d 
Old requests versions use to return None if a header was not present
amercader added a commit that referenced this issue May 11, 2015
@amercader
[#2425] Key error on resource proxy
ef6d3cd 
Old requests versions use to return None if a header was not present
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
zBackport 2.0.7 zBackport 2.1.5 zBackport 2.2.3
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@amercader