Skip to content
This repository has been archived by the owner on Jan 18, 2023. It is now read-only.

BF/PY3: Error handling #31

Merged
merged 3 commits into from Aug 6, 2016
Merged

BF/PY3: Error handling #31

merged 3 commits into from Aug 6, 2016

Conversation

effigies
Copy link
Owner

@effigies effigies commented Aug 1, 2016

  • Check that a peerid appears in the cache before deleting.
  • Permit socket.shutdown to fail silently, as failure to close a stream means it's not connected.
  • socket.error is OSError in Python 3, so remove socket.error and check errno.

Closes #30.

@effigies effigies mentioned this pull request Aug 1, 2016
Merged
@effigies effigies force-pushed the issue_30 branch 2 times, most recently from 87d9a00 to 242a294 Compare August 4, 2016 01:49
@effigies effigies merged commit 969c3a3 into master Aug 6, 2016
@effigies effigies deleted the issue_30 branch August 6, 2016 13:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

OSError: [Errno 107] Transport endpoint is not connected
1 participant
@effigies