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

Added "back" exception handling for the download clients. #9774

Merged
merged 3 commits into from
Aug 7, 2021

Conversation

p0psicles
Copy link
Contributor

@p0psicles p0psicles commented Aug 6, 2021

  • This is needed because we want to raise exceptions when a connection error occurs.
    The downloadhandler will remove tracked info_hashes when it doesn't get them back from the client. We don't want to untrack info_hash because it doesn't get them back, due to a connection error.
  • PR is based on the DEVELOP branch
  • Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review
  • Read the contribution guide

* This is needed because we want to raise exceptions when a connection error occurs.
The downloadhandler will remove tracked info_hashes when it doesn't get them back from the client. We don't want to untrack info_hash because it doesn't get them back, due to a connection error.
@p0psicles p0psicles merged commit 64c8fc5 into develop Aug 7, 2021
@p0psicles p0psicles deleted the feature/dh-fix-removed branch August 7, 2021 07:56
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.

1 participant