-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add message presenter, updated .pre-commit-config.yaml - EMAIL UPDATE 1/2 #155
Conversation
I added max TLP and link prefix so it can be used in templates when multiple report items are selected. |
Just included recommendations field for link prefix addition. |
Fixed issues that caused MISP generation failure. |
Simplified templates. |
I think it is finished. Please review. |
Added user data without password to product for presenters. |
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.
thank you for the very fine contribution and sorry for the delay. Could you please in the future split updates to many smaller patches?
As name suggests, message presenter outputs title and message body from templates. It outputs message_title and message_body. Email publisher in the second pull request either uses this or works similarly to the current state.
This commit also updates pre-commit configuration:
newer black version
added flake8
newer pre-commit-hooks version
added verification of yaml syntax and check if end of file ends with only newline
Feel free to suggest improvements.