Le filtre d'enveloppe efface le message de toute erreur ≥ 500 : un 503 « non configuré » délibéré ne peut pas dire ce qui manque #58
Closed
opened 2026-09-12 11:33:45 +00:00 by nicolas-fryder
·
2 comments
No Branch/Tag specified
main
dev
@eqs/tooling@0.24.3
@eqs/testing@0.24.3
@eqs/telemetry@0.24.3
@eqs/platform-next@0.24.3
@eqs/platform-nest@0.24.3
@eqs/http@0.24.3
@eqs/errors@0.24.3
@eqs/config@0.24.3
@eqs/cli@0.24.3
@eqs/auth@0.24.3
@eqs/tooling@0.24.2
@eqs/testing@0.24.2
@eqs/telemetry@0.24.2
@eqs/platform-next@0.24.2
@eqs/platform-nest@0.24.2
@eqs/http@0.24.2
@eqs/errors@0.24.2
@eqs/config@0.24.2
@eqs/cli@0.24.2
@eqs/auth@0.24.2
@eqs/tooling@0.24.1
@eqs/testing@0.24.1
@eqs/telemetry@0.24.1
@eqs/platform-next@0.24.1
@eqs/platform-nest@0.24.1
@eqs/http@0.24.1
@eqs/errors@0.24.1
@eqs/config@0.24.1
@eqs/cli@0.24.1
@eqs/auth@0.24.1
@eqs/tooling@0.24.0
@eqs/testing@0.24.0
@eqs/telemetry@0.24.0
@eqs/platform-next@0.24.0
@eqs/platform-nest@0.24.0
@eqs/http@0.24.0
@eqs/errors@0.24.0
@eqs/config@0.24.0
@eqs/cli@0.24.0
@eqs/auth@0.24.0
@eqs/tooling@0.23.0
@eqs/testing@0.23.0
@eqs/telemetry@0.23.0
@eqs/platform-next@0.23.0
@eqs/platform-nest@0.23.0
@eqs/http@0.23.0
@eqs/errors@0.23.0
@eqs/config@0.23.0
@eqs/cli@0.23.0
@eqs/auth@0.23.0
@eqs/tooling@0.22.0
@eqs/testing@0.22.0
@eqs/telemetry@0.22.0
@eqs/platform-next@0.22.0
@eqs/platform-nest@0.22.0
@eqs/http@0.22.0
@eqs/errors@0.22.0
@eqs/config@0.22.0
@eqs/cli@0.22.0
@eqs/auth@0.22.0
@eqs/tooling@0.21.0
@eqs/testing@0.21.0
@eqs/telemetry@0.21.0
@eqs/platform-next@0.21.0
@eqs/platform-nest@0.21.0
@eqs/http@0.21.0
@eqs/errors@0.21.0
@eqs/config@0.21.0
@eqs/cli@0.21.0
@eqs/auth@0.21.0
@eqs/tooling@0.20.0
@eqs/testing@0.20.0
@eqs/telemetry@0.20.0
@eqs/platform-next@0.20.0
@eqs/platform-nest@0.20.0
@eqs/http@0.20.0
@eqs/errors@0.20.0
@eqs/config@0.20.0
@eqs/cli@0.20.0
@eqs/auth@0.20.0
@eqs/tooling@0.19.0
@eqs/testing@0.19.0
@eqs/telemetry@0.19.0
@eqs/platform-next@0.19.0
@eqs/platform-nest@0.19.0
@eqs/http@0.19.0
@eqs/errors@0.19.0
@eqs/config@0.19.0
@eqs/cli@0.19.0
@eqs/auth@0.19.0
@eqs/tooling@0.17.1
@eqs/testing@0.17.1
@eqs/telemetry@0.17.1
@eqs/platform-next@0.17.1
@eqs/platform-nest@0.17.1
@eqs/http@0.17.1
@eqs/errors@0.17.1
@eqs/config@0.17.1
@eqs/cli@0.17.1
@eqs/auth@0.17.1
@eqs/tooling@0.17.0
@eqs/testing@0.17.0
@eqs/telemetry@0.17.0
@eqs/platform-next@0.17.0
@eqs/platform-nest@0.17.0
@eqs/http@0.17.0
@eqs/errors@0.17.0
@eqs/config@0.17.0
@eqs/cli@0.17.0
@eqs/auth@0.17.0
@eqs/tooling@0.16.0
@eqs/testing@0.16.0
@eqs/telemetry@0.16.0
@eqs/platform-next@0.16.0
@eqs/platform-nest@0.16.0
@eqs/http@0.16.0
@eqs/errors@0.16.0
@eqs/config@0.16.0
@eqs/cli@0.16.0
@eqs/auth@0.16.0
@eqs/tooling@0.15.0
@eqs/testing@0.15.0
@eqs/telemetry@0.15.0
@eqs/platform-next@0.15.0
@eqs/platform-nest@0.15.0
@eqs/http@0.15.0
@eqs/errors@0.15.0
@eqs/config@0.15.0
@eqs/cli@0.15.0
@eqs/auth@0.15.0
@eqs/tooling@0.14.0
@eqs/testing@0.14.0
@eqs/telemetry@0.14.0
@eqs/platform-next@0.14.0
@eqs/platform-nest@0.14.0
@eqs/http@0.14.0
@eqs/errors@0.14.0
@eqs/config@0.14.0
@eqs/cli@0.14.0
@eqs/auth@0.14.0
@eqs/tooling@0.13.0
@eqs/testing@0.13.0
@eqs/telemetry@0.13.0
@eqs/platform-next@0.13.0
@eqs/platform-nest@0.13.0
@eqs/http@0.13.0
@eqs/errors@0.13.0
@eqs/config@0.13.0
@eqs/cli@0.13.0
@eqs/auth@0.13.0
@eqs/tooling@0.12.2
@eqs/testing@0.12.2
@eqs/telemetry@0.12.2
@eqs/platform-next@0.12.2
@eqs/platform-nest@0.12.2
@eqs/http@0.12.2
@eqs/errors@0.12.2
@eqs/config@0.12.2
@eqs/cli@0.12.2
@eqs/auth@0.12.2
@eqs/tooling@0.12.1
@eqs/testing@0.12.1
@eqs/telemetry@0.12.1
@eqs/platform-next@0.12.1
@eqs/platform-nest@0.12.1
@eqs/http@0.12.1
@eqs/errors@0.12.1
@eqs/config@0.12.1
@eqs/cli@0.12.1
@eqs/auth@0.12.1
@eqs/tooling@0.12.0
@eqs/testing@0.12.0
@eqs/telemetry@0.12.0
@eqs/platform-next@0.12.0
@eqs/platform-nest@0.12.0
@eqs/http@0.12.0
@eqs/errors@0.12.0
@eqs/config@0.12.0
@eqs/cli@0.12.0
@eqs/auth@0.12.0
@eqs/tooling@0.11.2
@eqs/testing@0.11.2
@eqs/telemetry@0.11.2
@eqs/platform-next@0.11.2
@eqs/platform-nest@0.11.2
@eqs/http@0.11.2
@eqs/errors@0.11.2
@eqs/config@0.11.2
@eqs/cli@0.11.2
@eqs/auth@0.11.2
@eqs/tooling@0.11.1
@eqs/testing@0.11.1
@eqs/telemetry@0.11.1
@eqs/platform-next@0.11.1
@eqs/platform-nest@0.11.1
@eqs/http@0.11.1
@eqs/errors@0.11.1
@eqs/config@0.11.1
@eqs/cli@0.11.1
@eqs/auth@0.11.1
@eqs/tooling@0.11.0
@eqs/testing@0.11.0
@eqs/telemetry@0.11.0
@eqs/platform-next@0.11.0
@eqs/platform-nest@0.11.0
@eqs/http@0.11.0
@eqs/errors@0.11.0
@eqs/config@0.11.0
@eqs/cli@0.11.0
@eqs/auth@0.11.0
@eqs/tooling@0.10.1
@eqs/testing@0.10.1
@eqs/telemetry@0.10.1
@eqs/platform-next@0.10.1
@eqs/platform-nest@0.10.1
@eqs/http@0.10.1
@eqs/errors@0.10.1
@eqs/config@0.10.1
@eqs/cli@0.10.1
@eqs/auth@0.10.1
@eqs/tooling@0.10.0
@eqs/testing@0.10.0
@eqs/telemetry@0.10.0
@eqs/platform-next@0.10.0
@eqs/platform-nest@0.10.0
@eqs/http@0.10.0
@eqs/errors@0.10.0
@eqs/config@0.10.0
@eqs/cli@0.10.0
@eqs/auth@0.10.0
@eqs/tooling@0.9.0
@eqs/testing@0.9.0
@eqs/telemetry@0.9.0
@eqs/platform-next@0.9.0
@eqs/platform-nest@0.9.0
@eqs/http@0.9.0
@eqs/errors@0.9.0
@eqs/config@0.9.0
@eqs/cli@0.9.0
@eqs/auth@0.9.0
@eqs/tooling@0.8.1
@eqs/testing@0.8.1
@eqs/telemetry@0.8.1
@eqs/platform-next@0.8.1
@eqs/platform-nest@0.8.1
@eqs/http@0.8.1
@eqs/errors@0.8.1
@eqs/config@0.8.1
@eqs/cli@0.8.1
@eqs/auth@0.8.1
@eqs/tooling@0.8.0
@eqs/testing@0.8.0
@eqs/telemetry@0.8.0
@eqs/platform-next@0.8.0
@eqs/platform-nest@0.8.0
@eqs/http@0.8.0
@eqs/errors@0.8.0
@eqs/config@0.8.0
@eqs/cli@0.8.0
@eqs/auth@0.8.0
@eqs/tooling@0.7.0
@eqs/testing@0.7.0
@eqs/telemetry@0.7.0
@eqs/platform-next@0.7.0
@eqs/platform-nest@0.7.0
@eqs/http@0.7.0
@eqs/errors@0.7.0
@eqs/config@0.7.0
@eqs/cli@0.7.0
@eqs/auth@0.7.0
@eqs/tooling@0.6.0
@eqs/testing@0.6.0
@eqs/telemetry@0.6.0
@eqs/platform-next@0.6.0
@eqs/platform-nest@0.6.0
@eqs/http@0.6.0
@eqs/errors@0.6.0
@eqs/config@0.6.0
@eqs/cli@0.6.0
@eqs/auth@0.6.0
@eqs/tooling@0.5.1
@eqs/testing@0.5.1
@eqs/telemetry@0.5.1
@eqs/platform-next@0.5.1
@eqs/platform-nest@0.5.1
@eqs/http@0.5.1
@eqs/errors@0.5.1
@eqs/config@0.5.1
@eqs/cli@0.5.1
@eqs/auth@0.5.1
@eqs/tooling@0.5.0
@eqs/testing@0.5.0
@eqs/telemetry@0.5.0
@eqs/platform-next@0.5.0
@eqs/platform-nest@0.5.0
@eqs/http@0.5.0
@eqs/errors@0.5.0
@eqs/config@0.5.0
@eqs/cli@0.5.0
@eqs/auth@0.5.0
@eqs/tooling@0.4.0
@eqs/testing@0.4.0
@eqs/telemetry@0.4.0
@eqs/platform-next@0.4.0
@eqs/platform-nest@0.4.0
@eqs/http@0.4.0
@eqs/errors@0.4.0
@eqs/config@0.4.0
@eqs/cli@0.4.0
@eqs/auth@0.4.0
@eqs/tooling@0.2.0
@eqs/telemetry@0.2.0
@eqs/errors@0.2.0
@eqs/config@0.2.0
@eqs/tooling@0.1.0
Labels
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set.
Reference
Infrastructure/eqs-platform#58
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@eqs/errorsmasque le message de toute réponse de statut ≥ 500, ce qui est juste par défaut — on ne fuite pas l'interne sur une panne. Mais ça rend impossible un cas légitime et courant : un 503 délibéré qui dit à l'appelant ce qu'il faut configurer.Le cas
Nous exposons une opération dont le fournisseur externe n'est pas encore branché. Le contrat est écrit, la route existe, et elle doit répondre « non branchée, il manque telle variable » — c'est une attente affichée, pas une panne.
Écrit de la façon la plus naturelle :
La réponse qui sort ne porte plus le message. L'appelant reçoit un 503 générique, et l'écran ne peut donc pas dire ce qui manque — alors que c'est toute l'information utile.
Ce que ça coûte
Le développeur écrit le message, le teste à la main, ne le voit pas, et ne comprend pas pourquoi : rien n'indique que le filtre l'a retiré. On soupçonne d'abord son propre code, puis Nest, puis le proxy. Chez nous, ça a coûté une recherche en plein milieu d'une tâche, et le contournement n'était pas devinable.
C'est d'autant plus piégeux que le comportement est correct pour les 5xx accidentels — donc il n'y a aucune raison de le suspecter.
Le contournement, qui marche
Passer par
EqsErrorplutôt que par l'exception Nest standard : le message survit et le code d'erreur (IA_FOURNISSEUR) arrive jusqu'à l'écran.Ce n'est pas écrit dans le README de
@eqs/errors, et c'est la seule chose qu'il fallait savoir.Ce que je proposerais
Par ordre de préférence :
EqsErrorgarde son message quel que soit son statut (c'est déjà le cas), une exception non typée est masquée au-dessus de 500 (c'est déjà le cas aussi). Il ne manque donc que de le documenter — voir 3.@eqs/errors: « au-dessus de 500, le message est retiré de l'enveloppe ; pour un refus délibéré qui doit s'expliquer, utilisezEqsError». Une ligne aurait suffi à nous éviter la recherche.Et un complément utile dans les deux cas : journaliser côté serveur qu'un message a été retiré, avec le message d'origine. Aujourd'hui, la suppression est parfaitement silencieuse des deux côtés.
Versions
@eqs/errors,@eqs/platform-nest: 0.8.1Trouvé en développant le satellite Atelier, en posant une opération volontairement « non branchée » dont l'écran doit nommer ce qui manque.
Vrai, et j'ai pris vos trois propositions — parce que votre 1 était déjà exacte, et que c'est justement ce qui manquait.
Votre option 1 : vous aviez raison, c'était déjà le cas
traduirea bien deux branches. Celle d'EqsErrorgarde le message quel que soit le statut ; celle des exceptions Nest le masque au-dessus de 500 :La distinction que vous décrivez existait donc, et rien ne la disait. C'est maintenant dans le README de
@eqs/errors, avec le tableau des statuts.Votre option 2 : prise aussi, et votre argument la justifie
J'ajoute le critère qui rend la liste non arbitraire : le cadre ne fabrique jamais un 501 ni un 503. Une panne accidentelle rend 500 — c'est le fourre-tout de tout ce qui remonte sans être typé. Un
NotImplementedExceptionou unServiceUnavailableExceptionexige une ligne écrite exprès, donc leur prose a été choisie par quelqu'un.Votre
ServiceUnavailableException("Aucun fournisseur IA n'est configuré…")arrive donc jusqu'à l'écran, tel quel.Votre complément, et une correction
À moitié seulement, et la nuance compte pour la suite : le filtre Nest journalisait déjà
exception.messagepour tout 5xx. Votre message était donc dans la trace serveur — ce qui manquait, c'est de dire qu'il n'était pas parti chez l'appelant.traduirerend maintenantmessageRetire, et les deux filtres (Nest et Next) l'ajoutent à leur ligne. C'est ce champ qui rend la suppression visible plutôt que devinable.Ce que j'ai éprouvé
Six cas, dont les deux qui comptent :
messageRetireestundefined;messageRetireporte l'original — sans ce second test, on saurait que le message part sans savoir qu'on le dit.Votre contournement
EqsErrorreste parfaitement valable, et c'est même mieux dans votre cas : elle porte uncode(IA_FOURNISSEUR) que l'écran peut tester, là où un 503 nu ne donne qu'un statut. Rien à retirer chez vous — vous y gagnez simplement de pouvoir écrire l'exception Nest quand le code d'erreur n'apporte rien.Ce que je n'ai pas fait
Je n'ai pas touché aux 502/504. Un amont peut renvoyer n'importe quoi, y compris une trace : les laisser passer transformerait
@eqs/errorsen relais de prose étrangère.Branche
chasse/knip-et-code-mort. Ticket laissé ouvert jusqu'à la publication de 0.9.0, qui part maintenant.connect ETIMEDOUT 127.0.0.1:<port éphémère>quand la suite complète tourne — jamais fichier par fichier #57Corrigé. Le filtre d'enveloppe ne mange plus le message d'un refus délibéré :
packages/errors/src/nest.tsdistingue une exception voulue (EqsError, dont le message est conservé) d'une panne inattendue (dont le message est retiré, et le retrait est signalé). Un 503 « non configuré » peut donc dire ce qui manque.Fermé après vérification contre le code de
devle 2026-09-21. Ce ticket restait ouvert faute d'unCloses #58dans le commit, pas faute de correctif.