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

[Improvement] FileInfo should include actual attachments in the report #839

Closed
xeaon opened this issue Aug 14, 2020 · 5 comments
Closed
Assignees

Comments

@xeaon
Copy link

xeaon commented Aug 14, 2020

Request Type

Feature / Improvement

Description

Since extracting file observables from reports was implemented in TheHive-Project/TheHive#982 (i still can't wrap my head around how I'm supposed build the report that TheHive picks the file due to lack of knowledge and documentation) it would be extremely useful, if the FileInfo Analyzer would this, especially in the Outlook submodule.

Example workflow after Improvement

  1. upload any malspam.msg which includes a malicious .doc and analyze with FileInfo
  2. import malicious .doc attachment provided in the report as file observable
  3. scan attachment to gain more insight real quick

If you could point me to the right direction, for example how a report has to look like to trigger TheHive to pick up file observables, I'd try to implement this improvement by myself and create a pull request.

@dadokkio dadokkio self-assigned this Nov 23, 2020
@dadokkio
Copy link
Contributor

Some wip [not yet fully functional] from sample email:
image

@xeaon
Copy link
Author

xeaon commented Nov 23, 2020

Thank you for your efforts so far! Let me know if I can help in any way.

@garanews
Copy link
Contributor

garanews commented Mar 9, 2021

@xeaon merged in devel with #915
It needs latest cortexutils v 2.1.0

@garanews garanews closed this as completed Mar 9, 2021
@v1p3r0u5
Copy link
Contributor

v1p3r0u5 commented May 4, 2021

@dadokkio Exactly what I'm looking for. Any chances that this will also be merged into TH4 soon?
Many thanks for your effort. Highly appreciated.

@xeaon
Copy link
Author

xeaon commented May 6, 2021

@xeaon merged in devel with #915
It needs latest cortexutils v 2.1.0

Thank you and sry for the late response. I'll give it a try!

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

No branches or pull requests

4 participants