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 dependency #4431

Closed
jgeerds opened this issue Feb 5, 2017 · 4 comments · Fixed by #5351
Closed

Update requests dependency #4431

jgeerds opened this issue Feb 5, 2017 · 4 comments · Fixed by #5351

Comments

@jgeerds
Copy link

jgeerds commented Feb 5, 2017

A few days ago requests 2.13 has been released. Unfortunately, docker-compose still enforces requests < 2.12:

'requests >= 2.6.1, != 2.11.0, < 2.12',

Is there a reason for this? Changes are mostly bugfixes.

It seems like some distributions already ship docker-compose with newer versions of requests without any issues.

@shin-
Copy link

shin- commented Feb 6, 2017

There are indeed reasons for this:

We make a fairly unconventional use of requests (HTTP over UNIX sockets) which makes us particularly sensitive to updates of the requests package - even when they're just bugfixes.

@neilsh
Copy link

neilsh commented May 19, 2017

Suggestion: I wonder if a sustainable solution going forward would be to submit unit tests to the requests project covering the unconventional functionality that docker-compose relies on, to help ensure their interface doesn't change with new versions.

@hrabalvojta
Copy link

Isn't this reason why you have RC versions?

@voidlily
Copy link

voidlily commented Nov 4, 2017

I tried changing the requirement line for requests in docker-compose's setup.py to match docker-py's as an initial pass and tested some very basic functionality for local dev. I'm not sure what would need to be done next to take this to completion (though I suspect unit tests submitted upstream as from a previous suggestion would be helpful to clarify the unconventional behavior docker-compose relies on).

My motivation for making this change on my end is that in a project I'm working on there's another package I'm using with a transitive dependency on a recent version of requests, causing a conflict with the most recent official release of docker-compose.

requests>2.18.0

'requests >= 2.6.1, != 2.11.0, < 2.12',

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

Successfully merging a pull request may close this issue.

5 participants