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

Virustotal v3 #1111

Merged
merged 7 commits into from
Jul 21, 2022
Merged

Conversation

lamachin3
Copy link
Contributor

Update Virustotal analyzer to v3 API and enriched results with new relations.
Resolves #1012

@azgaviperr
Copy link

Any readon why there is no short report template

Everything else is ok for me

@azgaviperr
Copy link

After some extra testing, I can see issues on some characters no showing correctly on the report
image

@jeromeleonard jeromeleonard self-requested a review July 20, 2022 15:24
@jeromeleonard jeromeleonard self-assigned this Jul 20, 2022
@jeromeleonard jeromeleonard added this to the 3.2.0 milestone Jul 20, 2022
@nadouani
Copy link
Contributor

Any readon why there is no short report template

Everything else is ok for me

Because short reports are no longer used :)

@lamachin3
Copy link
Contributor Author

After some extra testing, I can see issues on some characters no showing correctly on the report image

It seems like your fontawesome icons can't load. You can check if they are imported from the developer console of your browser.
I fixed an issue on the date format in the update section of your screenshot but for this one I can't find a way to simulate the alt attribute in images.

@lamachin3
Copy link
Contributor Author

I changed the icons it should be ok now.

@jeromeleonard jeromeleonard merged commit 15b3d18 into TheHive-Project:develop Jul 21, 2022
jeromeleonard added a commit that referenced this pull request Jul 22, 2022
jeromeleonard added a commit that referenced this pull request Jul 22, 2022
jeromeleonard added a commit that referenced this pull request Jul 22, 2022
jeromeleonard added a commit that referenced this pull request Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants