Bug #10672
wcs : ne pas logguer d'erreur si au moins un wcs retourne un résultat
0%
Description
Dans un environnement multi-collectivités un utilisateur ne va pas nécessairement être provisionné sur tous les services mais la cellule des demandes en cours elle va interroger tous les wcs et logguer une erreur pour ceux qui répondront avec une 403.
Ma proposition c'est de logguer une erreur uniquement si aucun wcs n'était provisionné.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 8 ans
- Fichier 0001-wcs-don-t-log-errors-if-at-least-one-wcs-answers-pos.patch 0001-wcs-don-t-log-errors-if-at-least-one-wcs-answers-pos.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 8 ans
J'ai pas bien compris pourquoi tu t'encombres d'un set() alors qu'un booléan suffirait ? (False au départ, devient True dans le « if response.status_code == 200 »)
En plus les set ça existe même pas en python 1.5.
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Vous ne voudriez pas juste ignorer les 403 ? Ça indique clairement que la personne n'a pas de compte sur le service, ou alors faire en sorte que ça retourne une erreur claire, « ok la signature elle est bonne mais je ne connais pas NameID ».
Mis à jour par Frédéric Péters il y a environ 8 ans
J'ai pas bien compris pourquoi tu t'encombres d'un set() alors qu'un booléan suffirait ? (False au départ, devient True dans le « if response.status_code == 200 »)
Pour pouvoir logguer les codes reçus.
Vous ne voudriez pas juste ignorer les 403 ? Ça indique clairement que la personne n'a pas de compte sur le service, ou alors faire en sorte que ça retourne une erreur claire, « ok la signature elle est bonne mais je ne connais pas NameID ».
Non, je préfère ne pas être silencieux sur les erreurs.
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Sauf que ce n'est pas une erreur... juste une condition normale mal rapportée.
Mis à jour par Frédéric Péters il y a environ 8 ans
Dans un déploiement publik normal, avoir une erreur 403 est vraiment une erreur et quelque chose à investiguer.
Mis à jour par Thomas Noël il y a environ 8 ans
Yep, en fait on s'est dit que si aucun wcs ne répond, ça veut dire qu'il y a une erreur (le provisionning n'a marché nulle part, c'est abusééé). En revanche on laisse silencieux le cas où un tas de wcs ne réponds pas (les autres villes).
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Il n'y pas de provisionning des utilisateurs enregistrés sur les instances des communes, donc c'est normal. Le provisionning ce n'est que pour les agents.
Mis à jour par Frédéric Péters il y a environ 8 ans
(je pense qu'on continue à ne pas se comprendre).
Aujourd'hui un utilisateur se connecte sur le portail citoyen de Montpellier, la cellule "demandes en cours" interroge tous les wcs, et pour tous les wcs (sauf celui de la Métropole, sauf ceux sur lesquels l'utilisateur s'est explicitement ocnnecté), il y a une erreur 403. Erreurs 403 normales vu les circonstances. On ne veut pas en entendre parler.
Quand un utilisateur se connecte sur le portail de mettons Alfortville, la cellule "demandes en cours" interroge le wcs, et s'il y a une erreur on veut en être notifié.
Mis à jour par Thomas Noël il y a environ 8 ans
Pour moi c'est "Ack" (modulo ma remarque sur la forme, l'utilisation d'un set, mais c'est pas bien grave)
Mis à jour par Frédéric Péters il y a environ 8 ans
- Statut changé de En cours à Résolu (à déployer)
Je le pousse, je le valide sur la dev de Montpellier puis je taggue, que ça puisse arriver sur la prod (cf message de Serghei).
commit 2a989d995aaae46cf10a7f4e299af1727a721f53
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Apr 15 15:42:54 2016 +0200
wcs: don't log errors if at least one wcs answers positively (#10672)
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de Résolu (à déployer) à Fermé
wcs: don't log errors if at least one wcs answers positively (#10672)