Re: changer le lien d'un document internet

Hello,

Félicitation !

Tu es le premier à signaler un bug lié à ce patch
https://git.spip.net/spip/medias/commit/be91f3fddf8dd4648207bb49a31a5e87a8626bdc

je pense que ce n’est pas lié au fait que tu sois dans la médiathèque ou dans la rubrique, mais à l’URL que tu essayes de mettre, et qui est détectée comme « dangereuse » par la fonction valider_url_distante()
https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/distant.php#L155

En gros on regarde les URLs fournies en documents distant pour s’assurer que ce ne sont pas des URLs locales du serveur auxquelles PHP peut accéder de l’intérieur alors qu’elles sont normalement bloquées de l’extérieur.
On a honteusement piqué une fonction de Wordpress pour ça, qu’on a adapté, mais je me demandais si on risquait d’avoir des faux positifs ou pas.

Donc plusieurs possibilités :
 - ton url est mal formée
 - c’est une url plus ou moins locales et en effet c’est normal que ça pouiche. Tu peux t’en tirer en déclarant le host en question dans le pipeline declarer_hosts_distants() https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/distant.php#L182
 - c’est une vrai URL légitime et la fonction fait une erreur scandaleuse en la supposant malveillante -> envoie nous un mail sur [email protected] avec l’URL en question pour qu’on puisse reproduire et débugguer le cas échéant

--
Cédric
Le 19 févr. 2020 à 12:25 +0100, gornety <[email protected]>, a écrit :
> salut
>
> soit une rubrique avec des documents joints dedans
> je souhaite modifier l'url internet d'un document joint distant
> je clique sur modifier le document, puis "Changer" dans le paragraphe
> "Fichier"
> je clique sur "Internet" puisqu'il s'agit d'une url que je veux indiquer
> puis sur le bouton "Choisir"
> et là tout va bien
>
> mais
>
> lorsque je vais dans la médiathèque,
> que je tente de faire la même chose sur le même document
> là j'ai un message d'erreur qui m'interdit pratiquement la manœuvre :
> "Le téléchargement des fichiers du type de https://www.xxxxx/histoire/
> n’est pas autorisé"
>
> comme si le formulaire confondait le changement d'url avec le
> téléchargement en local du fichier
>
> qu'est-ce je ne comprends pas, qu'est-ce donc que je fais mal ?
>
> --
> Cordialement, Yanic Gornet
> 0 608 604 681
> _______________________________________________
> liste: https://listes.rezo.net/mailman/listinfo/spip-dev
> doc: https://www.spip.net/
> dev: https://core.spip.net/
> irc://irc.freenode.net/spip