Page MenuHomePhabricator

X-Content-Duration header not applied to derivative transcoded .ogg and .ogv files
Closed, ResolvedPublic

Description

TimedMediaHandler's OggHandlerTMH applies an X-Content-Duration header on files, which gets returned back through our SWIFT backend etc -- this is useful for the browser or custom player widget which then doesn't have to seek in the file to find the end timestamp.

However this isn't being applied on derivative transcodes, which is a problem for my ogv.js/ogv.swf JS/Flash fallback player testing...

Source video file, includes X-Content-Duration in response:
https://upload.wikimedia.org/wikipedia/commons/f/fb/Cassini_Reveals_New_Ring_Quirks%2C_Shadows_During_Saturn_Equinox.OGG

Derivative video file, does not include X-Content-Duration in response:
https://upload.wikimedia.org/wikipedia/commons/transcoded/f/fb/Cassini_Reveals_New_Ring_Quirks%2C_Shadows_During_Saturn_Equinox.OGG/Cassini_Reveals_New_Ring_Quirks%2C_Shadows_During_Saturn_Equinox.OGG.360p.ogv

Should find a way to apply the custom header on derivatives as well, if possible...


Version: unspecified
Severity: normal

Details

Reference
bz62928

Event Timeline

bzimport raised the priority of this task from to Needs Triage.
bzimport set Reference to bz62928.
brion created this task.Mar 21 2014, 3:24 PM

Change 120184 had a related patch set uploaded by Brian Wolff:
Add support for specifying headers in FileRepo::quickImport

https://gerrit.wikimedia.org/r/120184

Change 120185 had a related patch set uploaded by Brian Wolff:
Set X-Content-Duration header on ogg transcodes.

https://gerrit.wikimedia.org/r/120185

Change 120184 merged by jenkins-bot:
Add support for specifying headers in FileRepo::quickImport

https://gerrit.wikimedia.org/r/120184

Change 120185 merged by jenkins-bot:
Set X-Content-Duration header on ogg transcodes.

https://gerrit.wikimedia.org/r/120185

Should be fixed. Note this will only apply to new transcodes (once this is deployed to commons). Existing transcodes will not have the header set. Maybe should file a separate bug to do something about that (?)

Gilles moved this task from Untriaged to Done on the Multimedia board.Dec 4 2014, 10:11 AM
Gilles triaged this task as Unbreak Now! priority.
Gilles lowered the priority of this task from Unbreak Now! to Needs Triage.Dec 4 2014, 11:23 AM