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

malwarebazaar hash search #728

Merged
merged 4 commits into from
Apr 24, 2020
Merged

malwarebazaar hash search #728

merged 4 commits into from
Apr 24, 2020

Conversation

dadokkio
Copy link
Contributor

Address #722
At the moment only search has been implemented because only bad sample can be submitted. Probably a responder for that feature will be better.

@dadokkio dadokkio changed the base branch from master to develop March 24, 2020 16:36
@dadokkio dadokkio added this to the 2.7.0 milestone Mar 24, 2020
Copy link
Contributor

@garanews garanews left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add in long template other useful info:

  • reporter
  • delivery_method
  • intelligence
  • file_information
  • comments

Copy link
Contributor

@garanews garanews left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Long template ok.
Need manage when hash not found:

{
"query_status": "hash_not_found"
}

@dadokkio dadokkio changed the base branch from develop to release/2.7.0 April 1, 2020 14:00
@jeromeleonard jeromeleonard added scope:analyzer Issue is analyzer related category:new-analyzer New analyzer submitted labels Apr 6, 2020
@garanews garanews merged commit 0ad2263 into release/2.7.0 Apr 24, 2020
@garanews garanews deleted the feature/malwarebazaar branch April 24, 2020 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-analyzer New analyzer submitted scope:analyzer Issue is analyzer related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants