Last modified: 2014-03-26 21:08:42 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T64928, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 62928 - X-Content-Duration header not applied to derivative transcoded .ogg and .ogv files
X-Content-Duration header not applied to derivative transcoded .ogg and .ogv ...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
TimedMediaHandler (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Michael Dale
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-03-21 15:24 UTC by Brion Vibber
Modified: 2014-03-26 21:08 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Brion Vibber 2014-03-21 15:24:03 UTC
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...
Comment 1 Gerrit Notification Bot 2014-03-22 02:28:49 UTC
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
Comment 2 Gerrit Notification Bot 2014-03-22 02:33:34 UTC
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
Comment 3 Gerrit Notification Bot 2014-03-24 06:09:35 UTC
Change 120184 merged by jenkins-bot:
Add support for specifying headers in FileRepo::quickImport

https://gerrit.wikimedia.org/r/120184
Comment 4 Gerrit Notification Bot 2014-03-26 21:04:38 UTC
Change 120185 merged by jenkins-bot:
Set X-Content-Duration header on ogg transcodes.

https://gerrit.wikimedia.org/r/120185
Comment 5 Bawolff (Brian Wolff) 2014-03-26 21:08:42 UTC
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 (?)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links