Projet

Général

Profil

Anomalie #1864

Supprimer toutes les requêtes externes faites par notre instance Piwigo photos.april.org

Ajouté par Frédéric Couchet il y a environ 7 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normale
Assigné à:
Christian P. Momon
Catégorie:
-
Version cible:
-
Début:
17/02/2017
Echéance:
% réalisé:

0%

Temps estimé:
Difficulté:
2 Facile

Description

On nous signale :

« Un petit message pour vous signaler que le sous-domaine photos.april.org envoie des requêtes vers
fonts.googleapis.com. J'imagine qu'il s'agit d'un comportement par défaut de Piwigo, mais c'est très moyen, et surtout incohérent pour l'April »

Une correction à faire svp sur notre instance photos.april.org.

Je pensais que cela avait déjà été fait (je crois me souvenir d'une discussion à ce sujet).

Historique

#1

Mis à jour par Christian P. Momon il y a environ 7 ans

Après une courte investigation, cela ne vient pas de Piwigo en lui-même mais du thème par défaut SimpleNG qui contient effectivement une dépendance active indélicate :

https://github.com/Piwigo/simple-theme-piwigo/blob/master/template/header.tpl#L33

Que faire ?

  • mettre la ligne en commentaire (après test, pas d'impact majeur sur le rendu) ?
  • faire modifier le thème par son auteur pour profiter d'une prochaine mise à jour ?
  • les deux ?
#2

Mis à jour par Cédric Heintz il y a environ 7 ans

Oui, ça vient effectivement du thème par défaut.
Pour l'instant, on peut "bêtement" commenter la ligne qui pose problème et en profiter pour remonter ça à l'auteur du thème, même si ça n'est pas indispensable.

#3

Mis à jour par Christian P. Momon il y a environ 7 ans

Ticket ouvert chez l'auteur : https://github.com/Piwigo/simple-theme-piwigo/issues/21

En regardant les logs des sources, la dernière modification remonte à mars 2016, donc les mises à jour ne sont pas trop fréquentes. La modification manuelle temporaire semble être viable.

#4

Mis à jour par Cédric Heintz il y a environ 7 ans

Bien joué :-)

#5

Mis à jour par Christian P. Momon il y a environ 7 ans

  • Statut changé de Nouveau à Résolu

Ligne mise en commentaire en production :

<!-- Cpm 2017-02-17 : https://agir.april.org/issues/1864 neutralisation dépendance active externe vers Google. -->
<!-- link href='http://fonts.googleapis.com/css?family=Open+Sans:400,400italic,700' rel='stylesheet' type='text/css' -->

Je passe ce ticket à « Résolu » \o/

#6

Mis à jour par François Poulain il y a environ 7 ans

Personnellement, je pense qu'il serait préférable d'héberger la fonte open sans et de la servir, plutôt que de la laisser dans le vide.

#7

Mis à jour par Christian P. Momon il y a environ 7 ans

Lorsqu'une fonte n'est pas disponible et pourtant pointée en CSS, elle n'est pas vraiment « dans le vide », le navigateur détecte son absence et lui substitue celle par défaut dans sont type (Sans, etc.).

Moi aussi, en général, je préfère les solutions iso-fonctionnelles. Mais ici, dans notre contexte :
  • le texte a une importance mineure sur le site des photos ;
  • les navigateurs assument sans problèmes la substitution ;
  • les différences de rendu sont minimes ;
  • ça demande du travail en plus (22 fichiers à télécharger et à brancher) ;
  • la modification sera écrasée par une future mise à jour.

Du coup, sont-ce des arguments acceptables pour se passer de l'action d'import local des-dîtes fontes ?

Sinon, pas de soucis, ce n'est pas une tâche herculéenne :-)

#8

Mis à jour par Cédric Heintz il y a environ 7 ans

J'ai commencé une modification dans ce sens, à savoir héberger soit même le font.
Mais j'avoue, ce n'est pas indispensable, on peut très bien s'en passer, le site est très bien comme il est :-)

#9

Mis à jour par Quentin Gibeaux il y a presque 5 ans

  • Statut changé de Résolu à Fermé
#10

Mis à jour par Christian P. Momon il y a plus de 3 ans

  • Assigné à mis à Christian P. Momon

Formats disponibles : Atom PDF