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

Add fdscan() method to ClamdUnixSocket #19

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

Conversation

joelsmith
Copy link

This method is useful when clamd is running as a non-root user
and can't directly open a file that needs to be scanned. In
such cases, the clamd client can open the file and submit
a scan job via the open file descriptor.

@joelsmith joelsmith force-pushed the master branch 3 times, most recently from 0c23eff to 96f766b Compare July 24, 2017 14:49
@joelsmith
Copy link
Author

@graingert Please take a look.

It looks like the py26 test is failing because no python 2.6 interpreter is available in the test environment. All the other tests are passing (except for pypy, which is skipping the new tests because it lacks the needed functionality in its python for the new feature).

Please let me know anything I should do differently.

Thanks!

@GreatBahram
Copy link

What is the problem with this pull request? @graingert

For future readers, the socket module has a send_fds (Python 3.9+) method, and multiprocessing.reduction.sendfds is available as well.

@graingert
Copy link
Owner

Please do drop stages for EOL pythons and support only 3.6,3.7,3.8,3.9 and 3.10

You'll probably need to add GitHub actions as Travis is EOL

@GreatBahram GreatBahram mentioned this pull request Aug 1, 2021
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.

3 participants