Mise en place des services pour le plugins oembed avec : issuu, youtube, vimedo et dailymotion. Surcharge des modeles pour que les services soit bien géré. #10

Closed
Pierretux wants to merge 3 commits from dev/services_video into master
Pierretux commented 1 year ago
Collaborator
There is no content yet.
Pierretux added 1 commit 1 year ago
Pierretux added 1 commit 1 year ago
Owner

Mon com : cette PR surcharge complètement des modèles de Oembed et de la dist (modeles/video), donc ça va péter les mises à jour à la fois de Oembed et de Médias si ces plugins font des améliorations ou des corrections de bugs.

Et par ailleurs, déjà les surcharges ici sont problématiques, en changeant le markup entourage pour Issuu (ya plus de figure), et pour oembed_video qui a 11 lignes alors que l'original en a 18, avec des trucs carrément virés.

Mon com : cette PR surcharge complètement des modèles de Oembed et de la dist (modeles/video), donc ça va péter les mises à jour à la fois de Oembed et de Médias si ces plugins font des améliorations ou des corrections de bugs. Et par ailleurs, déjà les surcharges ici sont problématiques, en changeant le markup entourage pour Issuu (ya plus de figure), et pour oembed_video qui a 11 lignes alors que l'original en a 18, avec des trucs carrément virés.
Pierretux added 1 commit 1 year ago
Collaborator

Pour répondre à ce besoin, je propose de mettre en place dans ce plugin des modèles (youtube.html, vimeo.html, etc.) utilisables dans les articles (je vais commencer par Youtube qui est le plus populaire chez nous). La doc de TarteAucitron.js fournit les modèles nécessaires. Ou alors, il faudrait que le plugin oEmbed intègre ces modèles et, si TarteAucitron est activé, il surcharge ses modèles de base avec ceux nécessaires au fonctionnement de TarteAucitron.

Pour répondre à ce besoin, je propose de mettre en place dans ce plugin des modèles (youtube.html, vimeo.html, etc.) utilisables dans les articles (je vais commencer par Youtube qui est le plus populaire chez nous). La doc de TarteAucitron.js fournit les modèles nécessaires. Ou alors, il faudrait que le plugin oEmbed intègre ces modèles et, si TarteAucitron est activé, il surcharge ses modèles de base avec ceux nécessaires au fonctionnement de TarteAucitron.
Owner

Il y a déjà eu une discussion là-dessus ici, autant faire le lien :
https://discuter.spip.net/t/re-spip-zone-commit-tarteaucitron-services-video-mise-en-place-des-services-pour-le-plugins-oembed-avec/19838

Oembed fait de l'async

  1. on ne doit pas utiliser le code pour charger direct les lecteurs, même si le visiteur a donné l'autorisation des cookies pour ce service
  2. dès que c'est possible, Oembed appelle le service en mode privacy sans cookie (ce que permet parfaitement youtube, le plus courant), et donc il n'y a JAMAIS de cookie tiers posé et donc jamais besoin de demander aucune autorisation

Il faut donc d'abord réfléchir à quels services ont réellement besoin d'être "entourés" par le wrapper de Tarteaucitron pour leur interdire de se lancer tant que les visiteurs n'ont pas donné l'autorisation explicite pour ces services.

Il y a déjà eu une discussion là-dessus ici, autant faire le lien : https://discuter.spip.net/t/re-spip-zone-commit-tarteaucitron-services-video-mise-en-place-des-services-pour-le-plugins-oembed-avec/19838 Oembed fait de l'async 1) on ne doit pas utiliser le code pour charger direct les lecteurs, même si le visiteur a donné l'autorisation des cookies pour ce service 2) dès que c'est possible, Oembed appelle le service en mode privacy sans cookie (ce que permet parfaitement youtube, le plus courant), et donc il n'y a JAMAIS de cookie tiers posé et donc jamais besoin de demander aucune autorisation Il faut donc d'abord réfléchir à quels services ont réellement besoin d'être "entourés" par le wrapper de Tarteaucitron pour leur interdire de se lancer tant que les visiteurs n'ont pas donné l'autorisation explicite pour ces services.
paidge closed this pull request 6 months ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.