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 notification template without escaping #18171
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.
So the requirement is:
|
I wouldn't make the distinction between titles and descriptions.
|
I went with your initial suggestion of modifying rendering of titles in the FE. It achieves the desired outcome and doesn't require any changes to templates. |
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.
Looks good, works as expected.
Resolves Graylog2/graylog-plugin-enterprise#6525
Provides a notification template that does not escape parameter strings. This is useful if the message is supposed to contain HTML characters, e.g.
"Index <i1> is unavailable"
.How Has This Been Tested?
Unit test and hand-crafted notifications
Types of changes
Checklist: