Fix/refactoring query_echappe_textes() qui ne detectait parfois pas completement et correctement les chaines #4884

Merged
cerdic merged 3 commits from fix-query_echappe_textes into master 1 month ago
cerdic commented 1 month ago
Owner

On robustifie la fonction avec controle en amont et en aval, en preferant ne rien faire si on a un doute plutot que de risquer de casser la requete sql.
On en profite pour rendre encore moins plausible la presence des sequences d'echappement dans la chaine
Et on modernise le code dans query_reinjecte_textes() en utilisant un argument splat

On modifie aussi l'usage dans req/mysql en privilegiant de garder la requete initiale intacte si il n'y a rien a faire dessus

A noter la mise a jour du test unitaire qui va avec
0df1140262

On robustifie la fonction avec controle en amont et en aval, en preferant ne rien faire si on a un doute plutot que de risquer de casser la requete sql. On en profite pour rendre encore moins plausible la presence des sequences d'echappement dans la chaine Et on modernise le code dans query_reinjecte_textes() en utilisant un argument splat On modifie aussi l'usage dans req/mysql en privilegiant de garder la requete initiale intacte si il n'y a rien a faire dessus A noter la mise a jour du test unitaire qui va avec https://git.spip.net/spip/tests/commit/0df11402625712ac4a224b0a5bce215b3be153eb
cerdic added 1 commit 1 month ago
e6f5553e98 Fix/refactoring query_echappe_textes() qui ne detectait parfois pas completement et correctement les chaines

Une remarque que me faisais real (il peut pas répondre ici) :

[08/09/2021 09:15] <cfreal> autre chose, cette regex: "/((['])[^']*(\\2))|(([\"])[^\"]*(\\5))/S"
[08/09/2021 09:16] <cfreal> c'est simplement /'[^']*'|"[^"]*"/
Une remarque que me faisais real (il peut pas répondre ici) : ``` [08/09/2021 09:15] <cfreal> autre chose, cette regex: "/((['])[^']*(\\2))|(([\"])[^\"]*(\\5))/S" [08/09/2021 09:16] <cfreal> c'est simplement /'[^']*'|"[^"]*"/ ```
cerdic added 1 commit 1 month ago
413ca3cc58 Fix/refactoring query_echappe_textes() qui ne detectait parfois pas completement et correctement les chaines
cerdic added 2 commits 1 month ago
cerdic merged commit 2c90e6e0b5 into master 1 month ago
Poster
Owner

C'est donc dans la boite, avec simplification de la regexp

C'est donc dans la boite, avec simplification de la regexp
cerdic deleted branch fix-query_echappe_textes 1 month ago
The pull request has been merged as 2c90e6e0b5.
Sign in to join this conversation.
Loading…
There is no content yet.