Skip to content

[rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine#2551

Merged
tipsy merged 4 commits into
javalin:masterfrom
elwin013:rendering-javalin-provide-legacy-template-engine
Mar 12, 2026
Merged

[rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine#2551
tipsy merged 4 commits into
javalin:masterfrom
elwin013:rendering-javalin-provide-legacy-template-engine

Conversation

@elwin013

@elwin013 elwin013 commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

In 7.x defaultTemplateEngine was aligned to other template engines and uses class path to load templates. Which is a good change.

To have same behavior in 7.x like in 6.x this PR introduces directoryTemplateEngine that behaves like in 6.x - usage: JavalinJte(JavalinJte.directoryTemplateEngine(). It also allow to pass the path.

To distinguish behavior defaultTemplateEngine is renamed to classPathTemplateEngine. defaultTemplateEngine is now a deprecated 'alias' for `classPathTemplateEngine

#2552

…ename default to classPathTemplateEngine

In 7.x `defaultTemplateEngine` was aligned to other template engines and uses class path to load templates. Which is a good change.

To have same behavior in 7.x like in 6.x this commit introduces `directoryTemplateEngine` that behaves like in 6.x - usage: `JavalinJte(JavalinJte.directoryTemplateEngine()`

To distinguish behavior `defaultTemplateEngine` is renamed to `classPathTemplateEngine`
@elwin013 elwin013 changed the title [rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine #2552 [rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine Mar 6, 2026
@elwin013 elwin013 changed the title #2552 [rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine [rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine Mar 6, 2026
@codecov

codecov Bot commented Mar 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.29%. Comparing base (962e215) to head (7be9b32).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #2551   +/-   ##
=========================================
  Coverage     86.29%   86.29%           
  Complexity     1509     1509           
=========================================
  Files           155      155           
  Lines          4348     4348           
  Branches        523      523           
=========================================
  Hits           3752     3752           
  Misses          363      363           
  Partials        233      233           

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tipsy

tipsy commented Mar 6, 2026

Copy link
Copy Markdown
Member

augment review

@augmentcode

augmentcode Bot commented Mar 6, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: Introduces a filesystem-based JTE engine via directoryTemplateEngine and renames the existing default engine to classPathTemplateEngine.
Why: Keeps 7.x defaults aligned to classpath loading while offering an explicit opt-in to the 6.x directory-based behavior.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@tipsy tipsy merged commit b43f37a into javalin:master Mar 12, 2026
9 checks passed
@tipsy

tipsy commented Mar 12, 2026

Copy link
Copy Markdown
Member

Thanks @elwin013 !

mergify Bot added a commit to robfrank/linklift that referenced this pull request Mar 19, 2026
Bumps `javalin.version` from 7.0.1 to 7.1.0.
Updates `io.javalin:javalin-bundle` from 7.0.1 to 7.1.0
Release notes

*Sourced from [io.javalin:javalin-bundle's releases](https://github.com/javalin/javalin/releases).*

> 7.1.0
> -----
>
> What's Changed
> --------------
>
> * Bump com.fasterxml.jackson.core:jackson-core from 2.21.0 to 2.21.1 by [`@​dependabot`](https://github.com/dependabot)[bot] in [javalin/javalin#2543](https://redirect.github.com/javalin/javalin/pull/2543)
> * [deps]: Bump the dependencies group across 1 directory with 17 updates by [`@​dependabot`](https://github.com/dependabot)[bot] in [javalin/javalin#2545](https://redirect.github.com/javalin/javalin/pull/2545)
> * [websocket] Fix WebSocket over HTTP/2 Extended Connect (RFC 8441) by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2547](https://redirect.github.com/javalin/javalin/pull/2547)
> * fix(README): update artifactId for SSL plugin in `README.md` by [`@​yvasyliev`](https://github.com/yvasyliev) in [javalin/javalin#2556](https://redirect.github.com/javalin/javalin/pull/2556)
> * Add `wsExceptionHandler` to `MicrometerPlugin` for WebSocket exception tagging by [`@​Copilot`](https://github.com/Copilot) in [javalin/javalin#2557](https://redirect.github.com/javalin/javalin/pull/2557)
> * [rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine by [`@​elwin013`](https://github.com/elwin013) in [javalin/javalin#2551](https://redirect.github.com/javalin/javalin/pull/2551)
> * Add javalin-bom module by [`@​Copilot`](https://github.com/Copilot) in [javalin/javalin#2558](https://redirect.github.com/javalin/javalin/pull/2558)
> * [performance] Reduce per-request overhead in hot path by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2559](https://redirect.github.com/javalin/javalin/pull/2559)
> * [performance] Eliminate per-request regex and allocation overhead by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2560](https://redirect.github.com/javalin/javalin/pull/2560)
>
> New Contributors
> ----------------
>
> * [`@​yvasyliev`](https://github.com/yvasyliev) made their first contribution in [javalin/javalin#2556](https://redirect.github.com/javalin/javalin/pull/2556)
> * [`@​elwin013`](https://github.com/elwin013) made their first contribution in [javalin/javalin#2551](https://redirect.github.com/javalin/javalin/pull/2551)
>
> **Full Changelog**: <javalin/javalin@javalin-parent-7.0.1...javalin-parent-7.1.0>


Commits

* [`e458dba`](javalin/javalin@e458dba) [maven-release-plugin] prepare for next development iteration
* [`9766d2f`](javalin/javalin@9766d2f) [maven-release-plugin] prepare release javalin-parent-7.1.0
* [`12acf0b`](javalin/javalin@12acf0b) [performance] Eliminate per-request regex and allocation overhead ([#2560](https://redirect.github.com/javalin/javalin/issues/2560))
* [`f6a1e40`](javalin/javalin@f6a1e40) [performance] Reduce per-request overhead in hot path
* [`1e59eda`](javalin/javalin@1e59eda) [pom] Add javalin-bom module ([#2558](https://redirect.github.com/javalin/javalin/issues/2558))
* [`b43f37a`](javalin/javalin@b43f37a) [rendering] jte - introduce directoryTemplateEngine for (6.x-like behavior)
* [`72b2320`](javalin/javalin@72b2320) [micrometer] Add `wsExceptionHandler` to `MicrometerPlugin`
* [`37c91c4`](javalin/javalin@37c91c4) [readme] Update artifactId for SSL plugin
* [`9acf45e`](javalin/javalin@9acf45e) [cleanup] Use Header constants
* [`0289084`](javalin/javalin@0289084) [headers] Add missing headers to Header.kt
* Additional commits viewable in [compare view](javalin/javalin@javalin-parent-7.0.1...7.1.0)
  
Updates `io.javalin.community.openapi:javalin-openapi-plugin` from 7.0.1 to 7.1.0
Release notes

*Sourced from [io.javalin.community.openapi:javalin-openapi-plugin's releases](https://github.com/javalin/javalin-openapi/releases).*

> 7.1.0
> -----
>
> **Changes**
>
> * [javalin/javalin-openapi#275](https://redirect.github.com/javalin/javalin-openapi/issues/275) [Make ClassLoader used for loading resources configurable](javalin/javalin-openapi@de1b6b7)
> * Support Javalin 7.1.0
>
> **Sponsors**
> Thanks to everyone who supported me this month 💜
>
> **Minimal requirements**
>
> * Java 17+ / Kotlin 2.3+
> * Javalin 7.1.0


Commits

* [`5388862`](javalin/javalin-openapi@5388862) [GH-275](https://redirect.github.com/javalin/javalin-openapi/issues/275) Cover missing resource loaders and release 7.1.1-rc.1
* [`cbbc8d0`](javalin/javalin-openapi@cbbc8d0) [GH-276](https://redirect.github.com/javalin/javalin-openapi/issues/276) Release 7.1.0 (Resolves [#276](https://redirect.github.com/javalin/javalin-openapi/issues/276))
* [`de1b6b7`](javalin/javalin-openapi@de1b6b7) [GH-275](https://redirect.github.com/javalin/javalin-openapi/issues/275) Make ClassLoader used for loading resources configurable (Resolves [#275](https://redirect.github.com/javalin/javalin-openapi/issues/275))
* See full diff in [compare view](javalin/javalin-openapi@7.0.1...7.1.0)
  
Updates `io.javalin.community.openapi:javalin-swagger-plugin` from 7.0.1 to 7.1.0
Release notes

*Sourced from [io.javalin.community.openapi:javalin-swagger-plugin's releases](https://github.com/javalin/javalin-openapi/releases).*

> 7.1.0
> -----
>
> **Changes**
>
> * [javalin/javalin-openapi#275](https://redirect.github.com/javalin/javalin-openapi/issues/275) [Make ClassLoader used for loading resources configurable](javalin/javalin-openapi@de1b6b7)
> * Support Javalin 7.1.0
>
> **Sponsors**
> Thanks to everyone who supported me this month 💜
>
> **Minimal requirements**
>
> * Java 17+ / Kotlin 2.3+
> * Javalin 7.1.0


Commits

* [`5388862`](javalin/javalin-openapi@5388862) [GH-275](https://redirect.github.com/javalin/javalin-openapi/issues/275) Cover missing resource loaders and release 7.1.1-rc.1
* [`cbbc8d0`](javalin/javalin-openapi@cbbc8d0) [GH-276](https://redirect.github.com/javalin/javalin-openapi/issues/276) Release 7.1.0 (Resolves [#276](https://redirect.github.com/javalin/javalin-openapi/issues/276))
* [`de1b6b7`](javalin/javalin-openapi@de1b6b7) [GH-275](https://redirect.github.com/javalin/javalin-openapi/issues/275) Make ClassLoader used for loading resources configurable (Resolves [#275](https://redirect.github.com/javalin/javalin-openapi/issues/275))
* See full diff in [compare view](javalin/javalin-openapi@7.0.1...7.1.0)
  
Updates `io.javalin:javalin-micrometer` from 7.0.1 to 7.1.0
Release notes

*Sourced from [io.javalin:javalin-micrometer's releases](https://github.com/javalin/javalin/releases).*

> 7.1.0
> -----
>
> What's Changed
> --------------
>
> * Bump com.fasterxml.jackson.core:jackson-core from 2.21.0 to 2.21.1 by [`@​dependabot`](https://github.com/dependabot)[bot] in [javalin/javalin#2543](https://redirect.github.com/javalin/javalin/pull/2543)
> * [deps]: Bump the dependencies group across 1 directory with 17 updates by [`@​dependabot`](https://github.com/dependabot)[bot] in [javalin/javalin#2545](https://redirect.github.com/javalin/javalin/pull/2545)
> * [websocket] Fix WebSocket over HTTP/2 Extended Connect (RFC 8441) by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2547](https://redirect.github.com/javalin/javalin/pull/2547)
> * fix(README): update artifactId for SSL plugin in `README.md` by [`@​yvasyliev`](https://github.com/yvasyliev) in [javalin/javalin#2556](https://redirect.github.com/javalin/javalin/pull/2556)
> * Add `wsExceptionHandler` to `MicrometerPlugin` for WebSocket exception tagging by [`@​Copilot`](https://github.com/Copilot) in [javalin/javalin#2557](https://redirect.github.com/javalin/javalin/pull/2557)
> * [rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine by [`@​elwin013`](https://github.com/elwin013) in [javalin/javalin#2551](https://redirect.github.com/javalin/javalin/pull/2551)
> * Add javalin-bom module by [`@​Copilot`](https://github.com/Copilot) in [javalin/javalin#2558](https://redirect.github.com/javalin/javalin/pull/2558)
> * [performance] Reduce per-request overhead in hot path by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2559](https://redirect.github.com/javalin/javalin/pull/2559)
> * [performance] Eliminate per-request regex and allocation overhead by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2560](https://redirect.github.com/javalin/javalin/pull/2560)
>
> New Contributors
> ----------------
>
> * [`@​yvasyliev`](https://github.com/yvasyliev) made their first contribution in [javalin/javalin#2556](https://redirect.github.com/javalin/javalin/pull/2556)
> * [`@​elwin013`](https://github.com/elwin013) made their first contribution in [javalin/javalin#2551](https://redirect.github.com/javalin/javalin/pull/2551)
>
> **Full Changelog**: <javalin/javalin@javalin-parent-7.0.1...javalin-parent-7.1.0>


Commits

* [`e458dba`](javalin/javalin@e458dba) [maven-release-plugin] prepare for next development iteration
* [`9766d2f`](javalin/javalin@9766d2f) [maven-release-plugin] prepare release javalin-parent-7.1.0
* [`12acf0b`](javalin/javalin@12acf0b) [performance] Eliminate per-request regex and allocation overhead ([#2560](https://redirect.github.com/javalin/javalin/issues/2560))
* [`f6a1e40`](javalin/javalin@f6a1e40) [performance] Reduce per-request overhead in hot path
* [`1e59eda`](javalin/javalin@1e59eda) [pom] Add javalin-bom module ([#2558](https://redirect.github.com/javalin/javalin/issues/2558))
* [`b43f37a`](javalin/javalin@b43f37a) [rendering] jte - introduce directoryTemplateEngine for (6.x-like behavior)
* [`72b2320`](javalin/javalin@72b2320) [micrometer] Add `wsExceptionHandler` to `MicrometerPlugin`
* [`37c91c4`](javalin/javalin@37c91c4) [readme] Update artifactId for SSL plugin
* [`9acf45e`](javalin/javalin@9acf45e) [cleanup] Use Header constants
* [`0289084`](javalin/javalin@0289084) [headers] Add missing headers to Header.kt
* Additional commits viewable in [compare view](javalin/javalin@javalin-parent-7.0.1...7.1.0)
  
Updates `io.javalin:javalin-testtools` from 7.0.1 to 7.1.0
Release notes

*Sourced from [io.javalin:javalin-testtools's releases](https://github.com/javalin/javalin/releases).*

> 7.1.0
> -----
>
> What's Changed
> --------------
>
> * Bump com.fasterxml.jackson.core:jackson-core from 2.21.0 to 2.21.1 by [`@​dependabot`](https://github.com/dependabot)[bot] in [javalin/javalin#2543](https://redirect.github.com/javalin/javalin/pull/2543)
> * [deps]: Bump the dependencies group across 1 directory with 17 updates by [`@​dependabot`](https://github.com/dependabot)[bot] in [javalin/javalin#2545](https://redirect.github.com/javalin/javalin/pull/2545)
> * [websocket] Fix WebSocket over HTTP/2 Extended Connect (RFC 8441) by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2547](https://redirect.github.com/javalin/javalin/pull/2547)
> * fix(README): update artifactId for SSL plugin in `README.md` by [`@​yvasyliev`](https://github.com/yvasyliev) in [javalin/javalin#2556](https://redirect.github.com/javalin/javalin/pull/2556)
> * Add `wsExceptionHandler` to `MicrometerPlugin` for WebSocket exception tagging by [`@​Copilot`](https://github.com/Copilot) in [javalin/javalin#2557](https://redirect.github.com/javalin/javalin/pull/2557)
> * [rendering] jte - introduce directoryTemplateEngine for (6.x-like), rename default to classPathTemplateEngine by [`@​elwin013`](https://github.com/elwin013) in [javalin/javalin#2551](https://redirect.github.com/javalin/javalin/pull/2551)
> * Add javalin-bom module by [`@​Copilot`](https://github.com/Copilot) in [javalin/javalin#2558](https://redirect.github.com/javalin/javalin/pull/2558)
> * [performance] Reduce per-request overhead in hot path by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2559](https://redirect.github.com/javalin/javalin/pull/2559)
> * [performance] Eliminate per-request regex and allocation overhead by [`@​tipsy`](https://github.com/tipsy) in [javalin/javalin#2560](https://redirect.github.com/javalin/javalin/pull/2560)
>
> New Contributors
> ----------------
>
> * [`@​yvasyliev`](https://github.com/yvasyliev) made their first contribution in [javalin/javalin#2556](https://redirect.github.com/javalin/javalin/pull/2556)
> * [`@​elwin013`](https://github.com/elwin013) made their first contribution in [javalin/javalin#2551](https://redirect.github.com/javalin/javalin/pull/2551)
>
> **Full Changelog**: <javalin/javalin@javalin-parent-7.0.1...javalin-parent-7.1.0>


Commits

* [`e458dba`](javalin/javalin@e458dba) [maven-release-plugin] prepare for next development iteration
* [`9766d2f`](javalin/javalin@9766d2f) [maven-release-plugin] prepare release javalin-parent-7.1.0
* [`12acf0b`](javalin/javalin@12acf0b) [performance] Eliminate per-request regex and allocation overhead ([#2560](https://redirect.github.com/javalin/javalin/issues/2560))
* [`f6a1e40`](javalin/javalin@f6a1e40) [performance] Reduce per-request overhead in hot path
* [`1e59eda`](javalin/javalin@1e59eda) [pom] Add javalin-bom module ([#2558](https://redirect.github.com/javalin/javalin/issues/2558))
* [`b43f37a`](javalin/javalin@b43f37a) [rendering] jte - introduce directoryTemplateEngine for (6.x-like behavior)
* [`72b2320`](javalin/javalin@72b2320) [micrometer] Add `wsExceptionHandler` to `MicrometerPlugin`
* [`37c91c4`](javalin/javalin@37c91c4) [readme] Update artifactId for SSL plugin
* [`9acf45e`](javalin/javalin@9acf45e) [cleanup] Use Header constants
* [`0289084`](javalin/javalin@0289084) [headers] Add missing headers to Header.kt
* Additional commits viewable in [compare view](javalin/javalin@javalin-parent-7.0.1...7.1.0)
  
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
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