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

Requests v2.11.0 causes breakage #1155

Closed
plauche opened this issue Aug 9, 2016 · 2 comments
Closed

Requests v2.11.0 causes breakage #1155

plauche opened this issue Aug 9, 2016 · 2 comments
Milestone

Comments

@plauche
Copy link

plauche commented Aug 9, 2016

This is a known issue within requests and looks like it will be fixed in v2.11.1. Documenting for the benefit of anyone else who runs into this :).

After a recent pip upgrade I found that docker-py was passing along an error from requests:

File "/usr/local/lib/python2.7/dist-packages/docker/client.py", line 307, in _stream_raw_result
for out in response.iter_content(chunk_size=1, decode_unicode=True):
File "/usr/local/lib/python2.7/dist-packages/requests/utils.py", line 372, in stream_decode_response_unicode
raise UnicodeError("Unable to decode contents with encoding %s." % encoding)
UnicodeError: Unable to decode contents with encoding None.

This has already been reported to requests (https://github.com/kennethreitz/requests/issues/3481) and fixed (kennethreitz/requests@d7f56ba) for the next version.

If you are running into this issue, the easiest fix for now appears to be reverting to a pre 2.11 version of requests.

@shin-
Copy link
Contributor

shin- commented Aug 9, 2016

Thank you for the report!

@shin- shin- added this to the 1.10.0 milestone Aug 9, 2016
@maguowei
Copy link

requests v2.11.1 has released. please support it.

http://docs.python-requests.org/en/latest/community/updates/#id1

shin- pushed a commit that referenced this issue Oct 14, 2016
Bug #1155 has been fixed starting with requests 2.11.1 and excluding it
from dependencies causes failures when using latest versions of both
libs together in our project.

Signed-off-by: Yuriy Taraday <yorik.sar@gmail.com>
openstack-gerrit pushed a commit to openstack/openstack that referenced this issue Nov 22, 2016
Project: openstack/requirements  4fc29359129efb66691cc8749c64cd355caf4258

Release docker-py version constraint

The upper constraint of docker-py was limited because there is a
conflicted dependency issue [2]. The issue was fixed in docker-py
1.8.1 [3] so we can release the constraint now.

Note: This patch was proposed [4] in before, but it has been
reverted [5] due to an issue [6]. Now, the issue has been fixed
in docker-py [7], so I re-proposed this patch.

[1] https://review.openstack.org/#/c/304296/
[2] pyca/cryptography#2870
[3] docker/docker-py#1039
[4] https://review.openstack.org/#/c/366948/
[5] https://review.openstack.org/#/c/381780/
[6] docker/docker-py#1155
[7] docker/docker-py#1242

Change-Id: Ie233d54d42f09fc22983fe5627d111f0db8d6252
openstack-gerrit pushed a commit to openstack/requirements that referenced this issue Nov 22, 2016
The upper constraint of docker-py was limited because there is a
conflicted dependency issue [2]. The issue was fixed in docker-py
1.8.1 [3] so we can release the constraint now.

Note: This patch was proposed [4] in before, but it has been
reverted [5] due to an issue [6]. Now, the issue has been fixed
in docker-py [7], so I re-proposed this patch.

[1] https://review.openstack.org/#/c/304296/
[2] pyca/cryptography#2870
[3] docker/docker-py#1039
[4] https://review.openstack.org/#/c/366948/
[5] https://review.openstack.org/#/c/381780/
[6] docker/docker-py#1155
[7] docker/docker-py#1242

Change-Id: Ie233d54d42f09fc22983fe5627d111f0db8d6252
openstack-gerrit pushed a commit to openstack/openstack that referenced this issue Nov 22, 2016
Project: openstack/requirements  4fc29359129efb66691cc8749c64cd355caf4258

Release docker-py version constraint

The upper constraint of docker-py was limited because there is a
conflicted dependency issue [2]. The issue was fixed in docker-py
1.8.1 [3] so we can release the constraint now.

Note: This patch was proposed [4] in before, but it has been
reverted [5] due to an issue [6]. Now, the issue has been fixed
in docker-py [7], so I re-proposed this patch.

[1] https://review.openstack.org/#/c/304296/
[2] pyca/cryptography#2870
[3] docker/docker-py#1039
[4] https://review.openstack.org/#/c/366948/
[5] https://review.openstack.org/#/c/381780/
[6] docker/docker-py#1155
[7] docker/docker-py#1242

Change-Id: Ie233d54d42f09fc22983fe5627d111f0db8d6252
tanaypf9 pushed a commit to tanaypf9/pf9-requirements that referenced this issue May 20, 2024
The upper constraint of docker-py was limited because there is a
conflicted dependency issue [2]. The issue was fixed in docker-py
1.8.1 [3] so we can release the constraint now.

Note: This patch was proposed [4] in before, but it has been
reverted [5] due to an issue [6]. Now, the issue has been fixed
in docker-py [7], so I re-proposed this patch.

[1] https://review.openstack.org/#/c/304296/
[2] pyca/cryptography#2870
[3] docker/docker-py#1039
[4] https://review.openstack.org/#/c/366948/
[5] https://review.openstack.org/#/c/381780/
[6] docker/docker-py#1155
[7] docker/docker-py#1242

Change-Id: Ie233d54d42f09fc22983fe5627d111f0db8d6252
tanaypf9 pushed a commit to tanaypf9/pf9-requirements that referenced this issue May 20, 2024
The upper constraint of docker-py was limited because there is a
conflicted dependency issue [2]. The issue was fixed in docker-py
1.8.1 [3] so we can release the constraint now.

Note: This patch was proposed [4] in before, but it has been
reverted [5] due to an issue [6]. Now, the issue has been fixed
in docker-py [7], so I re-proposed this patch.

[1] https://review.openstack.org/#/c/304296/
[2] pyca/cryptography#2870
[3] docker/docker-py#1039
[4] https://review.openstack.org/#/c/366948/
[5] https://review.openstack.org/#/c/381780/
[6] docker/docker-py#1155
[7] docker/docker-py#1242

Change-Id: Ie233d54d42f09fc22983fe5627d111f0db8d6252
tanaypf9 pushed a commit to tanaypf9/pf9-requirements that referenced this issue May 20, 2024
The upper constraint of docker-py was limited because there is a
conflicted dependency issue [2]. The issue was fixed in docker-py
1.8.1 [3] so we can release the constraint now.

Note: This patch was proposed [4] in before, but it has been
reverted [5] due to an issue [6]. Now, the issue has been fixed
in docker-py [7], so I re-proposed this patch.

[1] https://review.openstack.org/#/c/304296/
[2] pyca/cryptography#2870
[3] docker/docker-py#1039
[4] https://review.openstack.org/#/c/366948/
[5] https://review.openstack.org/#/c/381780/
[6] docker/docker-py#1155
[7] docker/docker-py#1242

Change-Id: Ie233d54d42f09fc22983fe5627d111f0db8d6252
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

No branches or pull requests

3 participants