Skip to content
This repository has been archived by the owner on Jul 7, 2024. It is now read-only.

ne pas créer de player sur les sons en http, seulement https #176

Closed
Fil opened this issue Feb 25, 2017 · 4 comments
Closed

ne pas créer de player sur les sons en http, seulement https #176

Fil opened this issue Feb 25, 2017 · 4 comments
Assignees
Labels

Comments

@Fil
Copy link
Member

Fil commented Feb 25, 2017

cf. https://seenthis.net/messages/572384

@Fil
Copy link
Member Author

Fil commented Feb 25, 2017

une autre solution serait d'avoir un proxy sur seenthis, mais ça risque de poser de nouvaux problèmes

@brunob brunob self-assigned this Mar 2, 2017
@brunob brunob added the bug label Mar 2, 2017
@brunob
Copy link
Member

brunob commented Dec 21, 2019

Est-ce que que ça vaut toujours le coup d'ajouter une exception pour ça aujourd'hui maintenant que la plupart des sites sont accessibles en https ?

@brunob
Copy link
Member

brunob commented Dec 22, 2019

Ça se passe là https://zone.spip.org/trac/spip-zone/browser/spip-zone/_plugins_/plugins_seenthis/autoembed/autoembed.php#L134 il suffit de retirer le ? après https pour que l'embed des mp3 n'ait lieu que si le site source est en https.

Si on souhaite être sympa avec les sites qui utilisent autoembed et sont encore en http, on pourrait conditionner la modification sur un test simple comme (!empty($_SERVER["HTTPS"]) and $_SERVER["HTTPS"] == 'on') même si ça ne matchera pas tous les cas possibles. Sinon, si on considère que autoembed est forcément lié à seenthis on peut directement tester la constante _HTTPS.

Vos avis ?

@brunob
Copy link
Member

brunob commented Dec 25, 2019

Devrait être corrigé par https://zone.spip.net/trac/spip-zone/changeset/119335 que je pousse en prod dans la foulée.

@brunob brunob closed this as completed Dec 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants