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

[virustotal_downloader] Resolve "RuntimeError: Timeout context manager should be used inside a task" #629

Merged

nor3th merged 3 commits into OpenCTI-Platform:master from YungBinary:vt_fixes

Mar 11, 2022

Contributor

YungBinary commented Mar 10, 2022

Proposed changes

Upon testing in AWS I discovered the connector fails with RuntimeError and this commit fixes that. I decided to stop using the vt-py API library as the simplest solution.

Checklist

I consider the submitted work as finished
I tested the code for its functionality using different use cases
I added/update the relevant documentation (either on github or on notion)
Where necessary I refactored code to improve the overall quality

leetn00b added 3 commits

March 10, 2022 16:22


          Fix RuntimeError

b88840a


          Resolve flake

c31edb8


          Black

497bd38

Contributor

nor3th commented Mar 11, 2022

Hey @YungBinary

Thanks a lot!! :)

Regards

nor3th merged commit 2c10a66 into OpenCTI-Platform:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet