Bug #67627: POST https://demo-toulouse.sigec.fr/maelisws-toulouse/services/FamilyService (=> 500) - Passerelle - Redmine Entr’ouvert

Bug #67627

POST https://demo-toulouse.sigec.fr/maelisws-toulouse/services/FamilyService (=> 500)

Ajouté par Sentry Io il y a presque 2 ans. Mis à jour il y a plus d'un an.

Statut:

Rejeté

Priorité:

Normal

Assigné à:

Nicolas Roche

Version cible:

Début:

21 juillet 2022

Echéance:

% réalisé:

Temps estimé:

Patch proposed:

Oui

Planning:

Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/62599/

POST https://demo-toulouse.sigec.fr/maelisws-toulouse/services/FamilyService (=> 500)

Fichiers

0001-toulouse-maelis-check-child-id-parameter-67627.patch (23,1 ko) 0001-toulouse-maelis-check-child-id-parameter-67627.patch		Nicolas Roche, 20 octobre 2022 18:20
0002-toulouse-maelis-disable-error-logging-in-requests-wr.patch (1,16 ko) 0002-toulouse-maelis-disable-error-logging-in-requests-wr.patch		Nicolas Roche, 21 octobre 2022 17:13
0001-toulouse-maelis-disable-error-logging-in-requests-wr.patch (1,16 ko) 0001-toulouse-maelis-disable-error-logging-in-requests-wr.patch		Nicolas Roche, 31 octobre 2022 10:26

Demandes liées

Historique

Mis à jour par Nicolas Roche il y a presque 2 ans

Projet changé de Suivi des traces à Passerelle
Assigné à mis à Nicolas Roche

Crash Maélis sur l'envoie d'un item via sa valeur au lieu de son index (ici la civilité).

Une erreur est survenue : java.sql.SQLException: ORA-12899: valeur trop grande pour la colonne "MAELIS"."H_PERS"."ER_CDCIV" (réelle : 8, maximum : 6)

Mis à jour par Nicolas Roche il y a presque 2 ans

Statut changé de Nouveau à Rejeté

En fait c'est Maélis qui renvoie une 500.
Le connecteur gère correctement l'erreur (et renvoie une 200).

{
   "data" : null,
   "err" : "Family-updateFamily-soap:Server",
   "err_class" : "passerelle.utils.jsonresponse.APIError",
   "err_desc" : "Une erreur est survenue : java.sql.SQLException: ORA-12899: valeur trop grande pour la colonne \"MAELIS\".\"H_PERS\".\"ER_CDCIV\" (réelle : 8, maximum : 6)\n" 
}

Mis à jour par Nicolas Roche il y a plus d'un an

Idem sur cette sentry également liée à ce ticket :
https://sentry.entrouvert.org/entrouvert/publik/issues/62599
(c'est une autre erreur où l'identifiant de l'enfant est passé vide au connecteur)
Le connecteur ne contrôle pas le numéro de l'enfant mais retourne bien une 200 suite au crash de maélis :

{
   "data" : null,
   "err" : "Family-createOrUpdateFSL-soap:Server",
   "err_class" : "passerelle.utils.jsonresponse.APIError",
   "err_desc" : "E26 : Le numéro de personne est requis et ne peut être vide" 
}

J’hésite à ré-ouvrir pour éviter ces sentries qui ne sont pas parlantes,
(il faut voir dans le response_headers que la réponse vient du serveur Apache de Sigec)
quitte à devoir ajouter du "code métier" dans le connecteur,
mais alors j'ai peur d'ouvrir la boite de Pandore.

Mis à jour par Benjamin Dauvergne il y a plus d'un an

C'est le wrapper request qui trace, si tu ne veux pas de remonté dans sentry pour ces 500, il faut poser log_requests_errors = False sur ton modèle Resource.

Mis à jour par Nicolas Roche il y a plus d'un an

Statut changé de Rejeté à Nouveau

Je ré-ouvre, parce que finalement #70075 traite le premier cas en contrôlant les clé de référentiel envoyées.
Dans le deuxième cas il s'agirait de vérifier que l'enfant appartient à la famille.
Comme cela on évite le ' log_requests_errors = False ' afin de continuer à garder un œil sur les éventuels crash de maélis.

Mis à jour par Nicolas Roche il y a plus d'un an

Fichier 0001-toulouse-maelis-check-child-id-parameter-67627.patch 0001-toulouse-maelis-check-child-id-parameter-67627.patch ajouté
Statut changé de Nouveau à Solution proposée
Patch proposed changé de Non à Oui

Mis à jour par Nicolas Roche il y a plus d'un an

Fichier 0001-toulouse-maelis-check-child-id-parameter-67627.patch ajouté

Concernant ces erreurs dans un autre ticket (#70560) qui prévoit d'autres sentry plus compliquées à éviter :

il faut mettre log_requests = False de toute façon

0002 : qui pose "log_requests = False".
Peut-être que 0001 est à présent de trop puisqu'ici Maélis renvoie des erreurs compréhensives.
(parce qu'il fait faire une seconde requête pour pas grand chose)