Skip to content

[8.17] change reporting usage of handlebars to @kbn/handlebars (#217778)#219543

Merged
pmuellr merged 1 commit intoelastic:8.17from
pmuellr:backport/8.17/pr-217778
Apr 29, 2025
Merged

[8.17] change reporting usage of handlebars to @kbn/handlebars (#217778)#219543
pmuellr merged 1 commit intoelastic:8.17from
pmuellr:backport/8.17/pr-217778

Conversation

@pmuellr
Copy link
Copy Markdown
Contributor

@pmuellr pmuellr commented Apr 29, 2025

Backport

This will backport the following commits from main to 8.17:

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)

# Conflicts:
#	x-pack/plugins/screenshotting/server/browsers/chromium/templates/index.test.ts
@pmuellr pmuellr requested a review from kibanamachine as a code owner April 29, 2025 12:54
@pmuellr pmuellr added the backport This PR is a backport of another PR label Apr 29, 2025
@pmuellr pmuellr enabled auto-merge (squash) April 29, 2025 12:54
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

@pmuellr pmuellr merged commit f0386a1 into elastic:8.17 Apr 29, 2025
11 checks passed
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