[SPIP Zone] r107766 - _plugins_/webfonts/trunk

spip-zone-commit@rezo.net a écrit le 27/11/2017 à 11:36 :

Author: cedric@yterium.com
Date: 2017-11-27 11:36:08 +0100 (Mon, 27 Nov 2017)
New Revision: 107766

Modified:
    _plugins_/webfonts/trunk/plugin.xml
    _plugins_/webfonts/trunk/webfonts_pipeline.php
Log:
compatibilite https

Details: Connexion · GitLab

Je suis surpris que tu fasses ça alors qu'on avait dit il y a déjà quelques temps que ça n'était pas une bonne pratique puisque :
dans une page http, on peut inclure du https
dans une page https, on doit inclure du https

Donc, inclure du https marche tout le temps.

Cf : The protocol-relative URL - Paul Irish

--
RealET

Le 27/11/2017 à 21:14, RealET a écrit :

spip-zone-commit@rezo.net a écrit le 27/11/2017 à 11:36 :

Author: cedric@yterium.com
Date: 2017-11-27 11:36:08 +0100 (Mon, 27 Nov 2017)
New Revision: 107766

Modified:
_plugins_/webfonts/trunk/plugin.xml
_plugins_/webfonts/trunk/webfonts_pipeline.php
Log:
compatibilite https

Details: Connexion · GitLab

Je suis surpris que tu fasses ça alors qu'on avait dit il y a déjà quelques temps que ça n'était pas une bonne pratique puisque :
dans une page http, on peut inclure du https
dans une page https, on doit inclure du https

Donc, inclure du https marche tout le temps.

Cf : The protocol-relative URL - Paul Irish

Non tu as une alerte dans ta console si tu charge une webfont en http dans une page https … tout du moins sous chrome et quand c'est une ressource externe, comme un cdn aussi

ou alors je sais pas pourquoi on me l'a fait corrigé sur tous les sites que je viens de passer en 3.2 + https

--
Bonne journée
Arnaud B. (Mist. GraphX)

Le 28/11/2017 à 06:19, Mist. GraphX a écrit :

Le 27/11/2017 à 21:14, RealET a écrit :

spip-zone-commit@rezo.net a écrit le 27/11/2017 à 11:36 :

Author: cedric@yterium.com
Date: 2017-11-27 11:36:08 +0100 (Mon, 27 Nov 2017)
New Revision: 107766

Modified:
_plugins_/webfonts/trunk/plugin.xml
_plugins_/webfonts/trunk/webfonts_pipeline.php
Log:
compatibilite https

Details: Connexion · GitLab

Je suis surpris que tu fasses ça alors qu'on avait dit il y a déjà quelques temps que ça n'était pas une bonne pratique puisque :
dans une page http, on peut inclure du https
dans une page https, on doit inclure du https

Donc, inclure du https marche tout le temps.

Cf : The protocol-relative URL - Paul Irish

Non tu as une alerte dans ta console si tu charge une webfont en http dans une page https … tout du moins sous chrome et quand c'est une ressource externe, comme un cdn aussi

ou alors je sais pas pourquoi on me l'a fait corrigé sur tous les sites que je viens de passer en 3.2 + https

aussi apparemment c'est la syntaxe recommandé, pour que ça fonctionne dans tous les cas sans changer le markup si je me fie à l'article que tu cite… non ?

--
Bonne journée
Arnaud B. (Mist. GraphX)

Mist. GraphX a écrit le 28/11/2017 à 07:30 :

Le 28/11/2017 à 06:19, Mist. GraphX a écrit :

Le 27/11/2017 à 21:14, RealET a écrit :

spip-zone-commit@rezo.net a écrit le 27/11/2017 à 11:36 :

Author: cedric@yterium.com
Date: 2017-11-27 11:36:08 +0100 (Mon, 27 Nov 2017)
New Revision: 107766

Modified:
_plugins_/webfonts/trunk/plugin.xml
_plugins_/webfonts/trunk/webfonts_pipeline.php
Log:
compatibilite https

Details: Connexion · GitLab

Je suis surpris que tu fasses ça alors qu'on avait dit il y a déjà quelques temps que ça n'était pas une bonne pratique puisque :
dans une page http, on peut inclure du https
dans une page https, on doit inclure du https

Donc, inclure du https marche tout le temps.

Cf : The protocol-relative URL - Paul Irish

Non tu as une alerte dans ta console si tu charge une webfont en http dans une page https … tout du moins sous chrome et quand c'est une ressource externe, comme un cdn aussi

ou alors je sais pas pourquoi on me l'a fait corrigé sur tous les sites que je viens de passer en 3.2 + https

aussi apparemment c'est la syntaxe recommandé, pour que ça fonctionne dans tous les cas sans changer le markup si je me fie à l'article que tu cite… non ?

Justement, l'article est précédé d'une mention que ce n'est plus une bonne pratique de faire // seulement (anti-patern).
Ça a été une bonne pratique, mais ça ne l'est plus.
Il est préférable de mettre https pour toutes les ressources externes.

--
RealET

Mist. GraphX a écrit le 28/11/2017 à 06:19 :

Le 27/11/2017 à 21:14, RealET a écrit :

spip-zone-commit@rezo.net a écrit le 27/11/2017 à 11:36 :

Author: cedric@yterium.com
Date: 2017-11-27 11:36:08 +0100 (Mon, 27 Nov 2017)
New Revision: 107766

Modified:
_plugins_/webfonts/trunk/plugin.xml
_plugins_/webfonts/trunk/webfonts_pipeline.php
Log:
compatibilite https

Details: Connexion · GitLab

Je suis surpris que tu fasses ça alors qu'on avait dit il y a déjà quelques temps que ça n'était pas une bonne pratique puisque :
dans une page http, on peut inclure du https
dans une page https, on doit inclure du https

Donc, inclure du https marche tout le temps.

Cf : The protocol-relative URL - Paul Irish

Non tu as une alerte dans ta console si tu charge une webfont en http dans une page https … tout du moins sous chrome et quand c'est une ressource externe, comme un cdn aussi

C'est bien ce que je disais : tu viens de citer le seul cas qui pose problème : inclure du http dans une page en https.

Mais le commit que viens de faire Cédric concerne un httpS// qui a été remplacé par l'anti-patern //

D'où ma question.

--
RealET

Indeed, mon commit n'a pas d'intérêt, je me suis emmelé les pinceaux :
j'avais une ancienne version avec http que j'ai voulu corriger, du coup j'ai up et j'ai pas vu que ça avait été passé en https déjà.

Tu peux revert si tu veux, c'est un peu égal
--
Cédric

RealET a écrit :

spip-zone-commit@rezo.net a écrit le 27/11/2017 à 11:36 :

Author: cedric@yterium.com
Date: 2017-11-27 11:36:08 +0100 (Mon, 27 Nov 2017)
New Revision: 107766

Modified:
_plugins_/webfonts/trunk/plugin.xml
_plugins_/webfonts/trunk/webfonts_pipeline.php
Log:
compatibilite https

Details: Connexion · GitLab

Je suis surpris que tu fasses ça alors qu'on avait dit il y a déjà
quelques temps que ça n'était pas une bonne pratique puisque :
dans une page http, on peut inclure du https
dans une page https, on doit inclure du https

Donc, inclure du https marche tout le temps.

Cf : The protocol-relative URL - Paul Irish