Project

General

Profile

Development #32820

utils : vérifier la next_url sans l'encoder

Added by Paul Marillonnet 11 months ago. Updated 1 day ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
06 May 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-utils-verify-next_url-without-encoding-it.patch View (996 Bytes) Paul Marillonnet, 06 May 2019 04:04 PM

0001-utils-verify-next_url-without-encoding-it.patch View (1.27 KB) Paul Marillonnet, 30 Oct 2019 03:58 PM

History

#1 Updated by Paul Marillonnet 11 months ago

  • Tracker changed from Support to Development

#2 Updated by Paul Marillonnet 11 months ago

J'en profite pour replacer la docstring en question.

#3 Updated by Thomas Noël 11 months ago

#4 Updated by Benjamin Dauvergne 11 months ago

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

#5 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Solution proposée to Solution validée
  • Assignee set to Paul Marillonnet

#6 Updated by Paul Marillonnet 5 months ago

Ok, faisons propre.

#7 Updated by Paul Marillonnet 2 months ago

  • Status changed from Solution proposée to 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

#8 Updated by Paul Marillonnet 1 day ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF