Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mettre en place un system pour rattraper les erreurs de noms de fichiers #261

Open
MestourBS opened this issue May 29, 2020 · 6 comments
Open

Comments

@MestourBS
Copy link

Sous Windows, par exemple, si on nomme un questionnaire con, le site affiche une erreur. Ceci n'arrive pas sur le site officiel avec con, mais pourrait arriver avec tout autre nom de fichier réservé.

@jubnl
Copy link

jubnl commented Aug 19, 2020

Liste des noms de fichiers réservés par Windows :

  • CON
  • PRN
  • AUX
  • NUL
  • COM1, COM2, COM3, COM4, COM5, COM6, COM7, COM8, COM9
  • LPT1, LPT2, LPT3, LPT4, LPT5, LPT6, LPT7, LPT8, LPT9

Caractères non autorisés :

  • \ / : * ? " < > |

@MestourBS
Copy link
Author

Il faut voir pour l'utilisation de $CI->security->sanitize_filename

@DidierViret
Copy link
Contributor

Vérifier s'il existe une fonction PHP pour éliminer ces noms de fichiers, dans ce cas utiliser la fonction.
Faire des proposition de traitement du cas où l'utilisateur entre un nom de fichier réservé.

@jubnl jubnl assigned jubnl and unassigned jubnl Aug 19, 2020
@jubnl
Copy link

jubnl commented Aug 24, 2020

Je viens de faire des tests, quand on nomme un questionnaire par un nom de fichier réservé il est automatiquement enregistré comme ceci : "nomDeFichierReserve-1.pdf". Pour le corrigé il n'y a pas à s'en faire, il est crée sous ce nom : "nomDeFichierReserve_corrige.pdf". Si un caractère spécial est mis dans le nom d'un questionnaire, ce caractère est remplacé par un "-".

@jubnl
Copy link

jubnl commented Aug 24, 2020

Lors de la création d'un nouveau questionnaire il est possible d'entrer des balises HTML qui sont prises en compte! Il est donc potentiellement possible de faire de l'injection SQL. C'est quelque chose à vérifier sur tout les projets!
Capture

@jubnl
Copy link

jubnl commented Aug 24, 2020

linked issue #275

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants