Videos with spaces in file name double encoded.

Created on 21 February 2019, over 5 years ago

Updated 7 February 2024, 8 months ago

My code must have been way out of date or something, but when I applied the security update to 8.x-1.4 today, the commit from https://www.drupal.org/project/video/issues/2880589 → broke video rendering for videos with spaces in the filename.

file_create_url and Url::toString are both independently encoding the URL so instead of File%20Name.mp4, I get File%2520Name.mp4.

I'm looking into an alternative way to handle this.

🐛 Bug report

Status

Needs review

Version

2.0

Component

Code

Created by

🇺🇸United States Mike Lewis

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

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

Comment 8 months ago →
🇨🇾Cyprus alex.bukach
Rerolled #3 against 9.1.x.
Open in Jenkins → Open on Drupal.org →
Core: 10.2.x + Environment: PHP 8.1 & MySQL 8
last update 8 months ago
1 pass
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MySQL 8
last update 8 months ago
1 pass

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024