-
Notifications
You must be signed in to change notification settings - Fork 437
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
[ImportFileYARA]: Create an import file connector dedicated to import YARA files/rules #3259
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved as no blockers found, based on code only.
# Expose and entrypoint | ||
COPY entrypoint.sh / | ||
RUN chmod +x /entrypoint.sh | ||
ENTRYPOINT ["/entrypoint.sh"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Expose and entrypoint | |
COPY entrypoint.sh / | |
RUN chmod +x /entrypoint.sh | |
ENTRYPOINT ["/entrypoint.sh"] | |
CMD ["python", "import-file-yara.py"] |
@@ -0,0 +1,3 @@ | |||
pycti==6.4.5 | |||
typing-extensions==4.12.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this does not seems to be used.
""" | ||
:param content: | ||
:return: | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Empty docstring
:param yara_rule: | ||
:param yara_name: | ||
:param yara_metadata: | ||
:return: | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Empty docstring
db61c01
to
8123a77
Compare
@romain-filigran Some comments was added by @flavienSindou (thank you for the review), it will be merged as no blockers found but the connector can be improved later |
Proposed changes
New internal import connector that handle .yar file.
Related issues
Checklist