Skip to content

Conversation

@balv82
Copy link
Contributor

@balv82 balv82 commented Aug 18, 2025

Description

Cette Pull Request regroupe un ensemble de bug fixes et features pour le projet extract, visant à améliorer la stabilité, la sécurité et l’ergonomie.
Elle adresse les issues suivantes : #308, #321, #323, #333, #337, #344, #346, #347, #351 et #353.

Objectifs

  • Renforcer la sûreté (entrées, erreurs, dépendances, contrôles d’accès).
  • Corriger des régressions et comportements inattendus.
  • Ajouter des fonctionnalités demandées par les utilisateurs.

Sécurité & conformité

Entrées validées (Bean Validation), messages d’erreur non verbeux.
Pas de SQL concaténé ; requêtes paramétrées/ORM uniquement.
Exceptions mappées → réponses 4xx/5xx génériques ; logs sans données sensibles.
Headers HTTP sûrs & cookies ´HttpOnly´/´Secure´/´SameSite´ si concernés.
Dépendances actualisées si nécessaire (notes dans le changelog).

Tests

  • Unitaires : cas heureux, cas d’erreur, cas limites.
  • Intégration : Spring Boot Test (MockMvc/RestAssured).
  • Base de données : Testcontainers PostgreSQL si schéma/DAO touchés.
  • Fonctionnels : profil Maven functional-tests quand applicable.
  • Couverture : ≥ 80 % sur classes modifiées (rapport Jacoco disponible).

@balv82 balv82 changed the title implemented custom error handling Features & Bugfixes Aug 18, 2025
balv82 and others added 23 commits August 20, 2025 14:27
…details template to prevent NPE for cancelled requests without matching rules
…passing

- Create new module extract-task-fmedesktop-v2 to overcome command line length limitations
- Pass parameters via GeoJSON file instead of command line arguments
- Automatic WKT to GeoJSON geometry conversion
- Support for all existing parameters in structured format
- Add comprehensive help documentation explaining GeoJSON usage
- Include PATCH_ISSUE documentation for implementation details
- Login page now uses browser locale instead of persisted session locale
- Notification emails sent in individual user's preferred locale
- Enhanced email system with locale-aware message rendering
- Added ProcessesRepository.getProcessOperators() method for user locale access
- Improved StandbyReminderEmail with individual user locale support
Copy link
Contributor

@benoitregamey benoitregamey left a comment

Choose a reason for hiding this comment

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

#337 : tested, 100% ok
#333 : tested, 100% ok

benoitregamey

This comment was marked as resolved.

benoitregamey

This comment was marked as resolved.

benoitregamey

This comment was marked as resolved.

Copy link
Contributor

@benoitregamey benoitregamey left a comment

Choose a reason for hiding this comment

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

#351 : I couldn't really test it. I'll ask a user that had trouble with that if it's ok for him. Let's say that it's 100% ok for now ;)

@benoitregamey benoitregamey merged commit b82fecf into asit-asso:master Nov 27, 2025
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