I like the idea of using youtube-nocookie.com.  I think making the changes in 
macro.py and utils.py (and tests probably) is the right way to go.  Iframes are 
only possible using the `[[embed` macro, project admins can't add a youtube.com 
iframe themselves.  So I think the types of you changes you describe will 
handle the embed macro and that'll be good.


---

** [tickets:#8261] Embed youtube videos without cookies**

**Status:** open
**Milestone:** unreleased
**Created:** Thu Nov 01, 2018 12:48 PM UTC by Ingo
**Last Updated:** Thu Nov 01, 2018 12:48 PM UTC
**Owner:** nobody


Embedded youtube videos are trackers, as they are storing coockies for a 
foreign host. With the new european GDPR that's a problem. Currently I patched 
macros.py and utils.py to patch the URL:

~~~
embed_url = html.find('iframe').get('src').replace("youtube.com", 
"youtube-nocookie.com")
~~~

But I don't know what the correct solution for this problem might be. At least 
we need to allow the nocookie URLs if the user adds those. But I guess that's 
not enough, as I don't think that you can't easily pass the responsibility for 
trackers on project pages to the admin of the page.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed 
to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.

Reply via email to