Skip to content

Conversation

@jsmnbom
Copy link
Member

@jsmnbom jsmnbom commented Sep 10, 2018

Fixes #1202

@jsmnbom jsmnbom changed the title Patch urllib3RequestField to make it *not* support RFC2231 Patch urllib3.RequestField to make it *not* support RFC2231 Sep 10, 2018
@jsmnbom jsmnbom added the 📋 pending-review work status: pending-review label Sep 10, 2018
"""
Regression test for https://github.com/python-telegram-bot/python-telegram-bot/issues/1202
"""
with open('tests/data/测试.png', 'rb') as f:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs a u'filename' prefix for py2.7 to work

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah great.. that was what I was missing. Thanks :D

@Eldinnie
Copy link
Member

left a review. Needs fixing

Not needed anymore, and also makes it not work... so that was not great
@jsmnbom jsmnbom merged commit e247fa7 into master Sep 10, 2018
@jsmnbom jsmnbom deleted the fix-1202 branch February 14, 2019 12:53
@github-actions github-actions bot locked and limited conversation to collaborators Aug 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

📋 pending-review work status: pending-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Could not send file with Chinese character

4 participants