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.

Sign up for GitHub

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

Jump to bottom

Use poll implementation to fix file descriptor leak #72

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Use poll implementation to fix file descriptor leak #72

wants to merge 3 commits into from

Commits on Apr 12, 2022

  1. Use poll implementation to fix file descriptor leak 

    Using select() here causes a file descriptor leak. We've replaced this
with a poll() implementation to avoid this.

The polling object needs to be opened with an eventmask that stops it
from blocking paramiko.

Fixes ploxiln#51

Co-authored-by: Dan Porter <[email protected]>
    @mcontoliniproofpoint @Stealthii
    mcontoliniproofpoint and Stealthii committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    1801b49 View commit details
    Browse the repository at this point in the history

  2. Implement select.poll for _forwarder

    @Stealthii
    Stealthii committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    64f6771 View commit details
    Browse the repository at this point in the history

  3. Improve forwarder context manager 

    Mapping has been added to define the sender and receiver for each event.
Additionally we continue with other events to combat the case where a
sender has sent data and also hung up unexpectedly.
    @Stealthii
    Stealthii committed Apr 12, 2022
    Configuration menu
    Copy the full SHA
    ccbabd3 View commit details
    Browse the repository at this point in the history