Skip to content

Register mustache size limit setting #119291

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

Merged
merged 3 commits into from
Dec 26, 2024

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Dec 26, 2024

In #114002 the maximum size of a mustache script output was made configurable. However, the setting was not registered. This commit registers the setting so that it can be set in elasticsearch.yml.

In elastic#114002 the maximum size of a mustache script output was made
configurable. However, the setting was not registered. This commit
registers the setting so that it can be set in elasticsearch.yml.
@rjernst rjernst added >bug :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache v9.0.0 v8.18.0 v7.17.27 labels Dec 26, 2024
@rjernst rjernst requested a review from a team as a code owner December 26, 2024 17:13
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Dec 26, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine
Copy link
Collaborator

Hi @rjernst, I've created a changelog YAML for you.

@rjernst rjernst added the auto-backport Automatically create backport pull requests when merged label Dec 26, 2024
@rjernst rjernst enabled auto-merge (squash) December 26, 2024 17:21
@rjernst rjernst merged commit 291edab into elastic:main Dec 26, 2024
15 of 16 checks passed
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Dec 26, 2024
In elastic#114002 the maximum size of a mustache script output was made
configurable. However, the setting was not registered. This commit
registers the setting so that it can be set in elasticsearch.yml.
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Dec 26, 2024
In elastic#114002 the maximum size of a mustache script output was made
configurable. However, the setting was not registered. This commit
registers the setting so that it can be set in elasticsearch.yml.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.x
7.17

elasticsearchmachine pushed a commit that referenced this pull request Dec 26, 2024
In #114002 the maximum size of a mustache script output was made
configurable. However, the setting was not registered. This commit
registers the setting so that it can be set in elasticsearch.yml.
elasticsearchmachine pushed a commit that referenced this pull request Dec 27, 2024
* Register mustache size limit setting (#119291)

In #114002 the maximum size of a mustache script output was made
configurable. However, the setting was not registered. This commit
registers the setting so that it can be set in elasticsearch.yml.

* fix compile

* fix compile

* fix test compile

* fix license header
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged >bug :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache Team:Core/Infra Meta label for core/infra team v7.17.27 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants