On ajoute a la fonction copie_locale() un argument $callback_valider_url qui permet de tester l'URL finale apres recuperation et de refuser la copie locale si l'URL ne valide pas https://git.spip.net/spip-team/securite/issues/4336
+ fix retour sur un test prealable, pour ne retourner local que si il existe

Refactoring de la fonction recuperer_infos_distantes() qui prend maintenant un tableau d'options en second argument (seul le core utilisait les 2nd et 3e arguments) et une option supplementaire pour passer une callback de validation de l'URL distante finale (apres suivi des redirections eventuelles) https://git.spip.net/spip-team/securite/issues/4336

Refactoring de la gestion des options headers/datas de recuperer_url() pour mieux gerer les sequences POST => 301 + Location => GET sur la nouvelle URL
voire POST => 307 sur url2 => POST sur url2 => 302 sur url3 => GET sur url3 le tout sans perdre les headers d'origine
en consequence on refabrique les donnes formattes a chaque redirection si besoin a partir des options headers+datas dont la derniere est eventuellement videe en cours de route

thx phpcbf, un jour un hook fera ce travail à notre place :)

ref spip/medias#4853

comme lors de l'ajout d'un document local, utliser `corriger_extension()` pour gérer les correspondances d'extension de ce type

fix #4960

cerdic a rédigé 3 years ago et marcimat a validé 3 years ago

Ameliorer valider_url_distante() : on utilise filter_var plutot que des regexp et on ajoute un controle sur le TTL du domaine pour que ce que l'on valide soit la meme chose vue dans la suite du hit

- sort dans une fonction dediee le code charge d'identifier l'extension d'un fichier selon les headers http et l'url source d'origine (code issu de la fonction recuperer_infos_distantes())
- on utilise cette nouvelle fonction distant_trouver_extension_selon_headers() pour identifier l'extension d'un fichier apres une copie locale et sanitizer le fichier local si besoin

* Passe 1 : le Spacing générique et les déclarations de classes et de fonctions 1217 fixes dans 158 fichiers
* Passe 2 : Structures de contrôle (if, for, etc.) 967 fixes dans 168 fichiers
* Passe 3 : Appels de fonctions, instanciations de classes 1810 fixes dans 128 fichiers
* Passe 4 : Quotes, Double-quotes 3179 fixes dans 174 fichiers
* Passe 5 : Short Array syntax 2697 fixes dans 248 fichiers

Co-authored-by: JamesRezo <james@rezo.net>
Reviewed-on: https://git.spip.net/spip/spip/pulls/4868


Co-authored-by: JamesRezo <jamesrezo@noreply.git.spip.net>
Co-committed-by: JamesRezo <jamesrezo@noreply.git.spip.net>

On renomme l'option 'headers' par coherence avec le nom utilise en sortie de la fonction pour les headers + ne pas filtrer les entetes deja fournis par SPIP, on permet de les surcharger par double envoi, ce que Apache permet meme si ce n'est pas super propre

Permettre à recuperer_url de compléter les en-têtes de la requête par des champs spécifiques ce qui est parfois nécessaire pour utiliser des API REST qui intègrent en particulier des données d'authentification dans le header.
+ Eviter d'utiliser POST si une autre méthode est demandée explicitement.

Eviter des formats d'image en dur dans regexp ici, on se refere aux fonctions de format image (et tant pis pour swf qui est mort)

Il restait des liens en http alors que nous sommes en https, cela fonctionnait, mais c’est toujours mieux :)
Il manquait surtout là: https://git.spip.net/spip/spip/src/branch/master/prive/login.html#L44

Il y a deux endroits, je ne pense pas que mettre https posera de problème, mais je préfère le dire des fois que :)
https://git.spip.net/spip/spip/src/branch/master/prive/formulaires/configurer_relayeur.php#L23
et
https://git.spip.net/spip/spip/src/branch/master/prive/ical_prive.html#L9

#4557

Franck a rédigé 4 years ago et marcimat a validé 4 years ago

«avec tendresse depuis 2001»