Development #32820
utils : vérifier la next_url sans l'encoder
Début:
06 mai 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Je ne comprends pas pourquoi on encode la next_url ici. D'ailleurs la docstring indique que la fonction get_next_url est censée extraire et décoder le champ next_url.
Fichiers
Historique
Mis à jour par Paul Marillonnet il y a presque 5 ans
- Fichier 0001-utils-verify-next_url-without-encoding-it.patch 0001-utils-verify-next_url-without-encoding-it.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
J'en profite pour replacer la docstring en question.
Mis à jour par Thomas Noël il y a presque 5 ans
A priori ça vient de https://dev.entrouvert.org/issues/20181
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Oui c'est ok pour moi (et oui le encode c'est juste pour vérifier que ça ne contient que de l'ASCII, pour faire propre on pourrait se permettre un
def is_ascii(s):
try:
s.encode('ascii')
return True
except UnicodeEncodeError:
return False
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Paul Marillonnet
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Fichier 0001-utils-verify-next_url-without-encoding-it.patch 0001-utils-verify-next_url-without-encoding-it.patch ajouté
- Statut changé de Solution validée à Solution proposée
Ok, faisons propre.
Mis à jour par Paul Marillonnet il y a environ 4 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit 26ebbb2f9cb325568682279dfbb0dba594a34a80
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Mon May 6 15:57:51 2019 +0200
utils: verify next_url without encoding it
Mis à jour par Paul Marillonnet il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée