[ismisepaul]

PR 7 of the Jakarta migration stack (#861)

Stacked on PR 6.

src/main/webapp/translation.jsp is the only JSP that declares taglibs; the seven JSTL pages (index, login, register, setup, getStarted, readyToPlay, blockedMessage) include it. Its two URIs move to jakarta.tags.core / jakarta.tags.fmt, which the bundled jakarta.servlet.jsp.jstl 3.0.1 TLDs declare.

JSTL 3.0 still resolves the legacy java.sun.com URIs, so this is modernisation, not a fix.

Rest of the #861 checklist

Swept the tree for anything else on the old namespace: no javax.* in any JSP, web.xml or resources. The remaining javax imports are JDK APIs that keep the name (javax.crypto, javax.xml.parsers, javax.xml.namespace, javax.sql.rowset), and jakarta.mail was already in place.

Verification

Docker smoke test on Tomcat 11: login.jsp?lang=es renders Spanish strings via fmt:message, with no unprocessed tags in the output.

Closes #880
Refs #861

🤖 Generated with Claude Code

[Copilot]

🟢 Approval recommended

Both replacements match the bundled JSTL version and leave existing translation behavior unchanged.

No open issues 🎉

What changed in this PR

Modernizes the shared JSP tag library declarations as part of the Jakarta migration, completing the two-line change described in #880.

Changes:

• Replaces the legacy JSTL core and formatting URIs with jakarta.tags.core and jakarta.tags.fmt.
• Preserves translation logic and the shared include used by seven JSP pages.

File	Description
src/​main/​webapp/​translation.jsp	Updates both tag library URIs to their canonical JSTL 3.0 names.

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

Both URI replacements match the configured JSTL 3.0 implementation, and no blocking issues were identified.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

The two URI replacements match the configured JSTL dependencies, and the reported Spanish-rendering smoke test supports preserved translation behavior.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

Both URI replacements match the bundled JSTL 3.0 implementation, and the reported Spanish-page smoke test confirms translation rendering.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

Both replacements match the configured JSTL implementation, and the reported Spanish-language smoke test confirms translation rendering.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

The two URI replacements match the configured JSTL implementation, with no unresolved findings and a reported successful Spanish-rendering smoke test.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

Both URI replacements are supported by the configured JSTL implementation, with no consumer changes or unresolved issues.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

The two URI replacements match JSTL 3.0, leave locale handling unchanged, and are supported by the reported Spanish-rendering smoke test.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

The two URI replacements match the bundled JSTL version, and the reported Spanish-language smoke test confirms translation rendering.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

Both replacements match JSTL 3.0, preserve the shared translation behavior, and have no unresolved review findings.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

Both URI replacements match the configured JSTL implementation, and the narrowly scoped change has no unresolved findings.

No open issues 🎉

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

Both URI replacements match the configured JSTL 3.0 implementation, with no changes to translation behavior or unresolved findings.

No open issues 🎉

_{Review effort: Balanced}