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

OPSWAT Filescan Sandbox Integration #24

Merged
merged 15 commits into from
Nov 8, 2023

Conversation

anikobartos
Copy link
Contributor

OPSWAT Filescan Sandbox

  • Removed old opswat.py file and created a new one
  • OPSWAT integration now use the OPSWAT Filescan Sandbox
  • Added test files and test cases as well
  • Extended the README.rst file

For usage it is necessary to have an OPSWAT Filescan Sandbox API-key,
Default host is OPSWAT Filescan Community

Example availability check under sandboxapi dir (call main directly):

python opswat.py "https://www.filescan.io" "your-api-key" available   

Usage within code:

from sandboxapi import opswat

# connect to the sandbox
sandbox = opswat.OPSWATSandboxAPI("your-api-key")

print("Does sandbox available?")
print(sandbox.is_available())

@battleoverflow battleoverflow self-requested a review October 25, 2023 14:18
@petya2164
Copy link

@azazelm3dj3d Do you have any feedback for us about this PR?

@battleoverflow
Copy link
Contributor

battleoverflow commented Nov 7, 2023

Hi, @petya2164.

So far, the PR is looking good. If no issues pop up, I plan to have this merged in by early next week at the latest. Thank you and your team for all the hard work!

@battleoverflow
Copy link
Contributor

I was able to finish looking at this a little earlier than anticipated. Everything looks great!

@battleoverflow battleoverflow merged commit af7500b into InQuest:master Nov 8, 2023
2 checks passed
@petya2164
Copy link

@azazelm3dj3d Thank you very much for the quick review!

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