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

Epic: Improve video embed behaviour #259

Open
2 tasks
stevejalim opened this issue Dec 14, 2023 · 2 comments
Open
2 tasks

Epic: Improve video embed behaviour #259

stevejalim opened this issue Dec 14, 2023 · 2 comments
Assignees

Comments

@stevejalim
Copy link
Collaborator

stevejalim commented Dec 14, 2023

We support video embeds from Youtube and Vimeo right now, although no BB site is using them for the time being.

Vimeo uses only essential cookies and in theory we can use a no-cookie version of YouTube, but that will require custom work.

However, we should go further than that:

  • We need to automatically swap to the youtube-nocookie.com when embedding a youtube.com URL - Create a no-cookie embed option for YouTube embeds #260
  • When the page is rendered, for any third-party provider, it seems courteous to only load the embed after the user has expressly clicked to say they want to see it

If we want to extend support to other providers beyond YouTube and Vimeo, we will need to review how we can do that in a privacy-preserving way.

We could extend support to include:

  • Facebook
  • Instagram
  • TikTok

However, we'll need to do this in a way that definitely doesn't conflict with our privacy policy. This'll need a bit of research and discussion. Hat-tip to @alexgibson for pointing this out.

Footnote: embeds from Meta companies now need an API key, which is another factor to weigh here.

@stevejalim stevejalim converted this from a draft issue Dec 14, 2023
@stevejalim stevejalim moved this from In Progress to Prioritized backlog in Birdbox Dec 14, 2023
@stevejalim stevejalim changed the title Support additional video embed sources [Discussion] Improve video embed behaviour Dec 14, 2023
@stevejalim
Copy link
Collaborator Author

@alexgibson Is there anything you'd like to add to this so far?

@alexgibson
Copy link
Member

@alexgibson Is there anything you'd like to add to this so far?

Only that we should use caution when adding third party embeds, and consult with legal first if we have questions. These iframes can often come loaded with cookies which need to adhere to our privacy policy, and they are also potential third party trackers (e.g. person logged in to social media site X loads one of our pages that contains an embed from X, which tells X that the person has visited our site).

In bedrock we currently only really use YouTube embeds, but we load these using their www.youtube-nocookie.com domain, and also only initiate loading third parts assets after a visitor has expressed an action to indicate they want to watch a video.

@stevejalim stevejalim moved this from Prioritized backlog to In Progress in Birdbox Dec 14, 2023
@stevejalim stevejalim changed the title [Discussion] Improve video embed behaviour Epic: Improve video embed behaviour Dec 14, 2023
@stevejalim stevejalim self-assigned this Dec 14, 2023
@stevejalim stevejalim moved this from In Progress to Prioritized backlog in Birdbox Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Prioritized backlog
Development

No branches or pull requests

2 participants