Skip to content

python 2.7 crashes handling error ConnectionResetError #43

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

Merged
merged 2 commits into from
Mar 13, 2018

Conversation

charlielito
Copy link
Contributor

Hi! I was running the server in python 2.7 but it crashed when it tried to handle the ConnectionResetError, because I think it is not defined in python 2.7. I'm proposing this fix, although I dont know if there's a better way to do it.

Hi! I was running the server in python 2.7 but it crashed when it tried to handle the *ConnectionResetError*, because I think it is not defined in python 2.7. I'm proposing this fix, although I dont know if there's a better way to do it.
@Pithikos
Copy link
Owner

Pithikos commented Mar 9, 2018

Thanks charlielito. It might be worth putting a comment in the code about this - mentioning that ConnectionResetError should be used instead in the future. Since in less than 2 years Python2 will not be supported anyway so it will make it easier to remove any Python2 specific code.

@Pithikos Pithikos merged commit ae6ee7f into Pithikos:master Mar 13, 2018
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

Successfully merging this pull request may close these issues.

2 participants