Skip to content

Conversation

@lea-raya
Copy link
Collaborator

Description / Objectif / Motivation / Contexte

Objectif

Permettre d'utiliser java -jar target/esup-stage-….war
(ou en exploded war java -cp /app/war-extrait org.springframework.boot.loader.launch.WarLauncher)

Problèmes

  • AppConfigService#updateTheme() réécrit une ressource theme.css contenue dans le .war.
  • ImpressionService#generatePDF() utilise un Class#getResource(…).getPath() qui est situé dans le .war pour créer des fichiers temporaires.
  • ImpressionService#getHtmlText() référence le dossier des polices de caractères en présupposant que c'est un répertoire (or il est dans le .war avec une URL nested jar)

Bénéfices

  • permettre d'utiliser le tomcat embarqué par défaut dans spring-boot
  • permettre d'utiliser l'image FROM eclipse-temurin:21-jdk-noble à la place de tomcat
  • permettre de construire l'image docker avec jib-maven-plugin (sans docker, sans privilèges)
  • permettre une exécution en conteneur sans privilèges

Ticket: NFR (non functional request)

Cas d'acceptance (Comment cela a-t-il été testé ?)

  • Étant donné un dossier config avec une configuration de l'application
  • Quand je lance java -jar target/esup-stage-….war
  • Alors l'application fonctionne normalement

Type

  • Correction de bogue (modification non cassante qui résout un problème).
  • Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).
  • Changement cassant (correction qui entraînerait la/une fonctionnalité existante à ne pas fonctionner comme précédemment).
  • Changement nécessitant une mise à jour de la documentation utilisateur.

Definition du fini

  • Les cas d'acceptance ci-dessus ont été vérifiés.
  • Revue par au moins un⋅e relecteur⋅ice autorisé⋅e.
  • Documentation(s) mise(s) à jour (utilisateur, technique, commentaires de code compris).
  • Si des changements cassants sont introduits, ils sont dûment décrits
    et les étapes de montée de version décrites (éventuellement scriptés).

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