Bug #68470
nouvelles erreurs pylint
0%
Description
pylint avertit désormais sur requests appelé sans paramètre timeout et a peut-être un faux positif sur Q() | Q().
https://jenkins.entrouvert.org/job/authentic/2940/pylint/new/
Fichiers
Révisions associées
general: give a timeout to all HTTP requests (#68470)
Historique
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0002-general-give-a-timeout-to-all-HTTP-requests-68470.patch 0002-general-give-a-timeout-to-all-HTTP-requests-68470.patch ajouté
- Fichier 0001-pylint-ignore-unsupported-binary-operation-68470.patch 0001-pylint-ignore-unsupported-binary-operation-68470.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Un commit pour ignorer unsupported-binary-operation, un autre pour ajouter des timeouts partout, en posant une variable dans les settings, comme ce qui est fait dans combo et passerelle.
Mis à jour par A. Berriot il y a plus d'un an
Frédéric Péters a écrit :
Un commit pour ignorer unsupported-binary-operation, un autre pour ajouter des timeouts partout, en posant une variable dans les settings, comme ce qui est fait dans combo et passerelle.
C'est okay pour moi mais est-ce qu'on utiliserait pas plutôt un unique objet `requests.Session` pour mutualiser ce genre de réglages?
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Agate Berriot a écrit :
Frédéric Péters a écrit :
Un commit pour ignorer unsupported-binary-operation, un autre pour ajouter des timeouts partout, en posant une variable dans les settings, comme ce qui est fait dans combo et passerelle.
C'est okay pour moi mais est-ce qu'on utiliserait pas plutôt un unique objet `requests.Session` pour mutualiser ce genre de réglages?
On oublie, j'étais certaine que `requests.Session` permettait de configurer le timeout de manière globale, ce n'est visiblement pas le cas.
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 09b1e27c1f8f3ea964388a1bb2f285005462817c Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Aug 29 08:33:05 2022 +0200 general: give a timeout to all HTTP requests (#68470) commit b3eb113d99cabe271671db13b8d7077939b25d72 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Aug 29 08:32:50 2022 +0200 pylint: ignore unsupported-binary-operation (#68470)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
pylint: ignore unsupported-binary-operation (#68470)