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

Use VirusTotal with python3 (issue #361) #446

Merged
merged 3 commits into from
May 14, 2019

Conversation

Nergie
Copy link

@Nergie Nergie commented Mar 26, 2019

Force VirusTotal analyzers to use python3 (issue #361)

  • Fix error ImportError: No module named 'StringIO'
    Old:
 import os
import StringIO
try:
    import requests
except ImportError:
pass

New:

import os
try:
    import requests
    import StringIO
except ImportError:
    from io import StringIO
    pass
  • Fix "fatal error"
    Old: print >> sys.stderr, str(results.get('verbose_msg'))
    New: print (str(results.get('verbose_msg')), sys.stderr)

@Nergie Nergie changed the title Use VirusTotal with python3 Use VirusTotal with python3 (issue #361) Mar 26, 2019
@Nergie Nergie changed the base branch from master to develop March 26, 2019 15:53
@nadouani nadouani added this to the 1.17.0 milestone Apr 2, 2019
@nadouani nadouani added the category:bug Issue is related to a bug label Apr 2, 2019
@nadouani nadouani self-assigned this Apr 26, 2019
@nadouani nadouani mentioned this pull request Apr 26, 2019
5 tasks
@jeromeleonard jeromeleonard self-assigned this May 14, 2019
@jeromeleonard jeromeleonard merged commit 97a1ff4 into TheHive-Project:develop May 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:bug Issue is related to a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants