Skip to content

fix(dispatcher): set client preferences as top choice for xml conversion#525

Open
Clemsazert wants to merge 1 commit intomainfrom
fix/dispatcher-convert-xml-rules
Open

fix(dispatcher): set client preferences as top choice for xml conversion#525
Clemsazert wants to merge 1 commit intomainfrom
fix/dispatcher-convert-xml-rules

Conversation

@Clemsazert
Copy link
Contributor

@Clemsazert Clemsazert commented Mar 10, 2026

🔎 Détails

Fix d'un bug relevé sur la conversion XML/JSON sur le LRM de test en utilisant le client fr.cisu.sdisz.

Les clients LRM utilisés pour mocker NexSIS avaient un comportement hardcodé dans le dispatcher qui annulait la conversion de JSON vers XML, qui est le comportement attendu pour les clients côtés pompiers.
Ce n'était pas le cas de fr.cisu.sdisz qui recevait ainsi les messages au format XML malgré le champ useXML valorisé à false dans le client-preferences.csv.

Changements :

  • Suppression de la logique hardcodée par clientId pour certains utilisateurs LRM
  • Le contenu du client-preferences.csv fait foi : si il est valorisé à true ou false pour un client, on ne veut pas bypass ce comportement dans el dispatcher (une seule source de vérité)
  • Pour l'instant les communications inter-hubex se font uniquement avec NexSIS via XML, mais il est possible que d'autres forces (police, gendarmerie) souhaitent rester sur du JSON. Ainsi on peut forcer de la conversion JSON -> XML uniquement pour les destinataires en fr.fire.* pour l'instant (le csv faisait fois en cas de conflit).

📄 Documentation

Fonctionnement actuel

image

Fonctionnement souhaité

image

@github-actions
Copy link

File Coverage [87.34%] 🍏
MessageHandler.java 90.6% 🍏
MessageUtils.java 79.61% 🍏
Total Project Coverage 86.92% 🍏

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