Longueur des noms de fichier non vérifiée #67

New Issue

Closed

opened 2020-08-01 22:09:22 +02:00 by Darks · 3 comments

Darks commented

2020-08-01 22:09:22 +02:00

Owner

Si on essaie d'uploader un fichier dont le nom est trop long, ça fait planter SQLAlchemy qui essaie de faire rentrer plus de 32 caractères dans un champ de 32.

En profiter pour faire des tests avec des caractères UTF-8 à la con. Normalement secure_filename fait les choses bien, mais bon, à vérifier.

Si on essaie d'uploader un fichier dont le nom est trop long, ça fait planter SQLAlchemy qui essaie de faire rentrer plus de 32 caractères dans un champ de 32. En profiter pour faire des tests avec des caractères UTF-8 à la con. Normalement `secure_filename` fait les choses bien, mais bon, à vérifier.

Darks added the

label 2020-08-01 22:09:22 +02:00

Darks commented

2020-08-02 13:56:54 +02:00

Author

Owner

Au passage, je pense qu'on peut monter la limite à 64 caractères…

Au passage, je pense qu'on peut monter la limite à 64 caractères…

Lephenixnoir commented

2020-08-02 14:14:29 +02:00

Owner

Oui, de toute façon ça passe à secure_filename() donc il n'y pas de risque de sécurité, et c'est pas avec 64 caractères que ça va spammer.

Oui, de toute façon ça passe à `secure_filename()` donc il n'y pas de risque de sécurité, et c'est pas avec 64 caractères que ça va spammer.

Darks commented

2020-08-02 14:30:14 +02:00

Author

Owner

Parfait, je m'en occupe :)

Parfait, je m'en occupe :)

Darks referenced this issue from a commit

2020-08-02 15:03:24 +02:00

attachments: fix #67

Darks closed this issue

2020-08-02 15:06:57 +02:00

Sign in to join this conversation.

No Label

No Milestone

No Assignees

2 Participants

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: devs/PCv5#67

No description provided.