Skip to content

[8.18] change reporting usage of handlebars to @kbn/handlebars (#217778)#219526

Merged
kibanamachine merged 2 commits intoelastic:8.18from
kibanamachine:backport/8.18/pr-217778
Apr 29, 2025
Merged

[8.18] change reporting usage of handlebars to @kbn/handlebars (#217778)#219526
kibanamachine merged 2 commits intoelastic:8.18from
kibanamachine:backport/8.18/pr-217778

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

…217778)

Change reporting's usage of `handlebars` to `@kbn/handlebars`. Also
added a test to ensure user input is HTML escaped (it always has been,
this just tests it).

There should be no change to the final rendered output, at all. These
changes only affect PDF and PNG reports, not CSV reports.

(cherry picked from commit 3b5e96a)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 29, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) April 29, 2025 12:15
@pmuellr
Copy link
Copy Markdown
Contributor

pmuellr commented Apr 29, 2025

/ci

@kibanamachine kibanamachine merged commit 64ac3a1 into elastic:8.18 Apr 29, 2025
8 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @pmuellr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants