Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

opt-in to YT embeds #961

Closed
vinniefalco opened this issue Feb 14, 2024 · 2 comments · Fixed by #973
Closed

opt-in to YT embeds #961

vinniefalco opened this issue Feb 14, 2024 · 2 comments · Fixed by #973

Comments

@vinniefalco
Copy link
Member

There should be a cookie (for people not logged in) and/or an account setting (for people logged in) about whether or not to allow YouTube embeds (or other similar content). This will let people opt out of the tracking that Google performs, but still allow video content for the people that want it.

@sdarwin
Copy link
Collaborator

sdarwin commented Feb 14, 2024

should be a cookie

To the developers: any time cookies are added, let's take into account how it affects the CDN. Discuss with me or directly adjust Fastly.

  • if it's a simple boolean cookie 0 or 1, yes or no, that may still permit caching. There will be two versions of each page corresponding with 0 or 1.
  • if it's extremely variable (a uuid), it can at least be removed in the VCL from documentation page loads.

@Lastique
Copy link
Member

Here is an interesting article on the topic: https://axbom.com/embed-youtube-videos-without-cookies/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants