Skip to content

Fix build: add Jakarta Servlet API and upgrade war plugin for JDK 17 - #56

Draft
stden wants to merge 1 commit into
mainfrom
fix/ci-jakarta-servlet-build
Draft

stden wants to merge 1 commit into
mainfrom
fix/ci-jakarta-servlet-build

Conversation

@stden

@stden stden commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Причина

  • ApplicationController импортирует jakarta.servlet.http, но в pom.xml был только javax.servlet (javaee-web-api 6.0). Компиляция на JDK 17 падает с package jakarta.servlet.http does not exist. Шаг build в ci.yml превращает эту ошибку в warning, поэтому job остаётся зелёным.
  • После устранения этой ошибки шаг verify падает при загрузке maven-war-plugin 2.3: на JDK 17 плагин не работает (TreeMap.comparator не открыт для unnamed module).

Изменения (только pom.xml)

  • Добавлена зависимость jakarta.servlet:jakarta.servlet-api:6.0.0 со scope provided (в WAR не попадает).
  • maven-war-plugin: 2.3 → 3.4.0.

Проверки (локально, JDK 17, те же команды, что в ci.yml)

  • mvn -B clean compile -Dmaven.test.skip=true -Djava.awt.headless=true → BUILD SUCCESS
  • mvn -B test -Djava.awt.headless=true → BUILD SUCCESS (тестов в проекте нет)
  • mvn -B verify -Dmaven.test.skip=true -Djava.awt.headless=true → BUILD SUCCESS, WAR собран

Не охвачено этим PR

  • Dependabot Updates (run 30620337734 на HEAD 8895006) остаётся без подтверждённой причины: логи job недоступны (HTTP 410). Этот PR не доказывает его исправления.
  • ci.yml по-прежнему превращает ошибки сборки и тестов в warning. Изменение оставлено владельцу: рекомендуется сделать такие ошибки фатальными.
  • Тестов нет, поэтому порог покрытия 100% (JaCoCo check) фактически не проверяет код.

🤖 Generated with Claude Code

ApplicationController imports jakarta.servlet.http, but the pom only
provided the javax.servlet API (javaee-web-api 6.0), so compilation
failed. The CI build step turned that failure into a warning, so the
job stayed green.

- Add jakarta.servlet:jakarta.servlet-api 6.0.0 with provided scope.
- Upgrade maven-war-plugin 2.3 to 3.4.0. Version 2.3 cannot load on
  JDK 17 (TreeMap.comparator is not opened to the unnamed module), so
  the verify step failed during packaging.

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
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.

1 participant