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
Upgrade requests to 2.21.0 #19385
Upgrade requests to 2.21.0 #19385
Conversation
setup.py
Outdated
@@ -38,8 +38,8 @@ | |||
'attrs==18.2.0', | |||
'bcrypt==3.1.4', | |||
'certifi>=2018.04.16', | |||
# Dec 5, 2018: Idna released 2.8, requests caps idna at <2.8, CI fails | |||
'idna==2.7', | |||
# Dec 17, 2018: Idna released 2.8, requests 2.21.0 is pinning idna |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If requests can work with the newer idna, we should remove our pinning completely.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
requests
wants 'idna>=2.5,<2.9'
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So shall we drop our pinning then ? We only added it because the newer requests could not work with it but it somehow got installed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, let's remove it. In a worst case scenario we would need to add it again.
c260689
to
686a856
Compare
Description:
Changelog: https://github.com/requests/requests/blob/master/HISTORY.md#2210-2018-12-10
Checklist:
tox
. Your PR cannot be merged unless tests pass