Contrib.social

Load a placeholder/thumbnail instead of the video's first frame to make the module GDPR compliant again

Open on Drupal.org →
Created on 11 July 2021, almost 3 years ago
Updated 23 October 2023, 8 months ago

Problem/Motivation

By now the Module loads the first Frame of the video. as background image.

So It has to interact with Youtube or Vimeo to fetch the image right? This is not GDPR-Compliant because even the user's IP is personal data in terms of GDPR.
Therefore you could not send the user's IP to Youtube or Vimeo before the Agreement-Button is clicked. So There must be a way to store Background images locally to make sure A Module, that aims to be helpful when it comes to achieve GDPR Compliance must not load data from an external server before consent is given.

Steps to reproduce

Install the Module
Use a pubkic Video on Vimeo or Youtube
Analyse HTTP headers
See Browser Console error

Proposed resolution

Implement the Module as formatter for the video Embed field and make it possible to select a field within a particular content type that contains the background image in a particular responsive image style.

After that add the URL of the image selected that way as background image url() to .video-text-overlay

In general the problem is, tht the Module is exactly doing what it should do, but it does not take in account two things

  1. To load the first Thumbinail of a video the Module needs to connet to an external server. Therefore the Users IP has to be send to the external server before the user has accepted that you do so.
  2. When you take a look at professional sites useing vimeo you'll soon discover that many of them useing Vimeo in a way that is not supportet by the module right now. They use vimeo as a video hoster and useing it to host their videos and embed them on their sites exclusively. If you useing that feature and hide your Video from Vimeo there are no data send by vimeo. So therte is no thumbinail, to that could be downloaded by the script.

    3. Therefore the mpdule needs an option to host background images localy. The Approach works for public Videos on Vimeo and on youtube as well. As soon as someone uses vimeo in a more professional way, this module is not quite usable right now.

Remaining tasks

User interface changes

implement a formatter for video embed field

API changes

Data model changes

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇩🇪Germany Joachim Namyslo Bayreuth 🇩🇪 🇪🇺

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024