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

Add message presenter, updated .pre-commit-config.yaml - EMAIL UPDATE 1/2 #155

Merged
merged 18 commits into from
Sep 27, 2023

Conversation

multiflexi
Copy link
Contributor

@multiflexi multiflexi commented Apr 6, 2023

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.

@multiflexi multiflexi mentioned this pull request Apr 6, 2023
@multiflexi
Copy link
Contributor Author

I added max TLP and link prefix so it can be used in templates when multiple report items are selected.

@multiflexi
Copy link
Contributor Author

Just included recommendations field for link prefix addition.

@multiflexi
Copy link
Contributor Author

Fixed issues that caused MISP generation failure.

@multiflexi multiflexi changed the title Add email presenter, updated .pre-commit-config.yaml - EMAIL UPDATE 1/2 Add message presenter, updated .pre-commit-config.yaml - EMAIL UPDATE 1/2 Jul 25, 2023
@multiflexi
Copy link
Contributor Author

Simplified templates.

@multiflexi
Copy link
Contributor Author

I think it is finished. Please review.

@multiflexi
Copy link
Contributor Author

Added user data without password to product for presenters.

Copy link
Collaborator

@milankowww milankowww 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 for the very fine contribution and sorry for the delay. Could you please in the future split updates to many smaller patches?

@milankowww milankowww merged commit 019f13b into SK-CERT:main Sep 27, 2023
2 checks passed
@multiflexi multiflexi deleted the email_presenter branch February 8, 2024 14:30
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

Successfully merging this pull request may close these issues.

2 participants