search

YesWiki / yeswiki

YesWiki is a wiki system written in PHP, including extensions making collaboration more simple (databases, maps, easy editing, bootstrap themes,...).
https://yeswiki.net
GNU Affero General Public License v3.0
107 stars 55 forks source link

refactor(FILTER_SANITIZE_STRING): use new arg for SecurityController::filterInput and make the update work #1163

Closed J9rem closed 3 months ago

J9rem commented 4 months ago

Contexte:

Propositionn:

mrflos commented 4 months ago

Hello je ne suis pas chaud pour supporter une constante que personne d'autre dans le monde utilise... Si j'en croies la doc : image C'est plutot htmlspecialchars qui ferai office de remplacement

J9rem commented 3 months ago

@mrflos je pense que je me suis mal expliqué.

Effectivement, il faut utiliser quelque chose comme https://stackoverflow.com/questions/69207368/constant-filter-sanitize-string-is-deprecated et c'est l'objectif de la nouvelle méthode de SecurityController (qui permet de factoriser le code).

Ma première proposition porte effectivement à confusion avec la constante dépréciée. Je viens de faire une nouvelle proposition qui ajoute un argument à la méthode SecurityController::input pour déclencher l'émulation de FILTER_SANITIZE_STRING sans utiliser la valeur de cette constante dépréciée.

J9rem commented 3 months ago

J'ai testé ceci: