Automattic / Automattic/jetpack

Video: JP appends "_hd" to Video URL in Media Library

Open
#11,064 4 comments 0 reactions 0 assignees View on GitHub
[Feature] VideoPress [Pri] Low Customer Report [LEGACY] Enhancement
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

#### To Repro

1. Download a 1080p video like this one: http://mirrors.standaloneinstaller.com/video-sample/jellyfish-25-mbps-hd-hevc.mp4
2. Note the video resolution is 1920x1080.
3. Have a test site with JP connected and video hosting enabled (needs Premium or Business plan).
4. Upload the video. We will process it...
5. Once done processing, open up the attachment editing modal.
6. Note how `videos.files.wordpress.com` URL has appended `_hd` to the filename: https://d.pr/i/EVgWug/ncVp8KOGZ7 This default URL shows the video at lower resolution than we have available.

#### What I expected

That the generic URL to the video be the URL to the original, not a downsized copy.

#### What happened instead

Downsized URL was used.

Came up in 1683169-zen and I can repro on a test site.

Contributor guide

Open the contributing guide

Research direction

Start at the Media Library attachment editing modal and trace how the videos.files.wordpress.com URL is selected after processing a 1080p upload. Compare the generic URL with the `_hd` URL using a test site with JP connected and video hosting enabled. Done means the generic video URL points to the original available resolution rather than the downsized copy.

Written by the indexing model from the issue text.

Assessment

Tech stack
php, wordpress
Domain
backend, web-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.