Implement and access check for show video path

Created on 29 June 2023, over 1 year ago
Updated 3 July 2023, over 1 year ago

Problem/Motivation

We are getting notices in logs on one site, because some sites are embedding links to 'vef/load' path with an invalid video URL. This is not the intended use for the URL but I think that this should lead to something like denied access rather than logging notices.
Our sites are using the patch introduced in this issue: https://www.drupal.org/project/video_embed_field/issues/3278279 β†’

Steps to reproduce

1. While using the patch linked in the related issue try to visit the vef/load path with specifying an invalid URL parameter - anything that cannot be resolved to a valid provider.
2. Check logs.

Proposed resolution

Add an access check that will try to load a provider for the video URL and if no provider is found deny access to the page.

πŸ› Bug report
Status

RTBC

Version

2.0

Component

Video Embed Field (base module)

Created by

πŸ‡ΈπŸ‡°Slovakia gresko8

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

Comments & Activities

Production build 0.71.5 2024