Project

General

Profile

Development #32820

utils : vérifier la next_url sans l'encoder

Added by Paul Marillonnet 9 months ago. Updated 3 months ago.

Status:
Solution proposé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 9 months ago

  • Tracker changed from Support to Development

#2 Updated by Paul Marillonnet 9 months ago

J'en profite pour replacer la docstring en question.

#3 Updated by Thomas Noël 9 months ago

#4 Updated by Benjamin Dauvergne 9 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 3 months ago

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

#6 Updated by Paul Marillonnet 3 months ago

Ok, faisons propre.

Also available in: Atom PDF