Skip to content

[Backport 2.19] Disable Mustache partial template resolution in CustomMustacheFactory - #22440

Merged
cwperks merged 1 commit into
opensearch-project:2.19from
opensearch-ci-bot:backport/backport-22438-to-2.19
Jul 10, 2026
Merged

[Backport 2.19] Disable Mustache partial template resolution in CustomMustacheFactory#22440
cwperks merged 1 commit into
opensearch-project:2.19from
opensearch-ci-bot:backport/backport-22438-to-2.19

Conversation

@opensearch-ci-bot

Copy link
Copy Markdown
Contributor

Backport e1fa8aa from #22438.

…opensearch-project#22438)

Override getReader() in CustomMustacheFactory to throw MustacheException
when a partial template ({{>name}}) is encountered at compile time.
Search templates in OpenSearch are self-contained inline strings, so
partial resolution has no legitimate use.

Signed-off-by: Craig Perkins <cwperx@amazon.com>
(cherry picked from commit e1fa8aa)
Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com>
@opensearch-ci-bot
opensearch-ci-bot requested a review from a team as a code owner July 10, 2026 18:01
@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 PR contains tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown
Contributor

✅ Gradle check result for b142c81: SUCCESS

@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.13%. Comparing base (aff3489) to head (b142c81).
⚠️ Report is 18 commits behind head on 2.19.

Additional details and impacted files
@@             Coverage Diff              @@
##               2.19   #22440      +/-   ##
============================================
+ Coverage     71.92%   72.13%   +0.20%     
+ Complexity    66009    64557    -1452     
============================================
  Files          5342     5120     -222     
  Lines        307392   300201    -7191     
  Branches      44862    44090     -772     
============================================
- Hits         221105   216555    -4550     
+ Misses        67823    65459    -2364     
+ Partials      18464    18187     -277     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cwperks
cwperks merged commit e3ab8df into opensearch-project:2.19 Jul 10, 2026
45 of 46 checks passed
@opensearch-ci-bot
opensearch-ci-bot deleted the backport/backport-22438-to-2.19 branch July 10, 2026 19:32
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