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] Add SHA256 Indicators and visual change #3311

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

Noxurge
Copy link
Contributor

@Noxurge Noxurge commented Jan 17, 2025

Added some features in MalwareBazaar Connector.

Proposed changes

  • Added indicator creation with sha256 for each artifact
  • Added relationship between indicator and artifact
  • Added a visual change to show the last execution in connector screen

Related issues

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

@richard-julien richard-julien force-pushed the master branch 2 times, most recently from 9818df2 to 0f4b30d Compare February 4, 2025 14:35
@SamuelHassine SamuelHassine force-pushed the master branch 2 times, most recently from b513b96 to 11a05bf Compare February 14, 2025 12:49
@helene-nguyen helene-nguyen self-assigned this Feb 25, 2025
@helene-nguyen helene-nguyen added the community use to identify PR from community label Feb 25, 2025
@helene-nguyen helene-nguyen linked an issue Feb 25, 2025 that may be closed by this pull request
Copy link
Member

@helene-nguyen helene-nguyen left a comment

Choose a reason for hiding this comment

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

@Noxurge Thanks for your contribution!! Could you please sign your commits with GPG to allow us to merge your PR ?

@Noxurge
Copy link
Contributor Author

Noxurge commented Feb 27, 2025

@helene-nguyen Sure! I just did it

@helene-nguyen
Copy link
Member

Thank you ! It still fails due to the format, could you rebase master on your branch ?

…t, and relationship between the two, also added a visual change to show the last execution in connector screen. Signed with GPG.
@Noxurge
Copy link
Contributor Author

Noxurge commented Feb 27, 2025

Done, thanks for asking!

Copy link
Member

@helene-nguyen helene-nguyen left a comment

Choose a reason for hiding this comment

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

Thank you !

@helene-nguyen helene-nguyen merged commit bc63dde into OpenCTI-Platform:master Feb 28, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community use to identify PR from community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Graphical items are missing in Malware Bazaar connector
2 participants