Skip to content

Add HTML field formatter#5996

Closed
jimmyjones2 wants to merge 1 commit intoelastic:masterfrom
jimmyjones2:html_formatter
Closed

Add HTML field formatter#5996
jimmyjones2 wants to merge 1 commit intoelastic:masterfrom
jimmyjones2:html_formatter

Conversation

@jimmyjones2
Copy link
Copy Markdown
Contributor

Closes #4216

What are your thoughts on this? Being able to write your own HTML allows neat things like converting a country code to a flag while still showing the code next to it, and having multiple links per field - eg. provide links to multiple reputation sites for IP addresses in weblogs etc.

@ghost
Copy link
Copy Markdown

ghost commented Jan 24, 2016

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

@jimmyjones2
Copy link
Copy Markdown
Contributor Author

@rashidkpc Any thoughts? Is this something that might be accepted or should it be an externally maintained plugin?

@rashidkpc
Copy link
Copy Markdown
Contributor

As noted in #4216 this isn't something we would put in the code base as its a major security issue. I'd advise anyone against installing this as it makes for an immediately exploitable xss vector.

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