Last modified: 2014-03-30 18:54:11 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 T62562, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 60562 - transcode error reporting should truncate long errors.
transcode error reporting should truncate long errors.
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
TimedMediaHandler (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Umang
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-01-29 05:21 UTC by Bawolff (Brian Wolff)
Modified: 2014-03-30 18:54 UTC (History)
8 users (show)

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


Attachments
Error message truncated by simply creating another one (1.10 KB, patch)
2014-03-29 22:01 UTC, Umang
Details

Description Bawolff (Brian Wolff) 2014-01-29 05:21:29 UTC
If there's a really long error in transcode (for example due to a "cli progress bar", entire thing is output). This is to the point where it has crashed my browser. Would be better if say the first 750 bytes and the last 750 bytes are shown, with the middle being omitted. Probably best not to even put the super-long error message in the db. (Also be cool if we figure out a command line option to disable the progress bar from being output)
Comment 1 Bawolff (Brian Wolff) 2014-03-26 20:02:04 UTC
Example of bug:
* https://commons.wikimedia.org/wiki/File:2013-11-22_Ein_Entz%C3%BCcklop%C3%A4discher_Abend.webm
*Click on Error in transcode on 2014-02-15T02:06:28 for the 720p transcode
*Note how its a couple megabytes worth of error message! (may freeze your browser)
Comment 2 Umang 2014-03-29 22:01:33 UTC
Created attachment 14977 [details]
Error message truncated by simply creating another one

Shows the difference between the earlier WebTranscodeJobs.php and the updated one.
Comment 3 Bawolff (Brian Wolff) 2014-03-30 02:26:06 UTC
Id prefer if this used something like $wgContLang->truncate(). C style string munching doesnt work too well for multibyte characters, and seems out of place in php.
Comment 4 Andre Klapper 2014-03-30 18:54:11 UTC
Hi Umang! Thanks for your patch!
You are welcome to use Developer access
  https://www.mediawiki.org/wiki/Developer_access
to submit this as a Git branch directly into Gerrit:
  https://www.mediawiki.org/wiki/Git/Tutorial
Putting your branch in Git makes it easier to review it quickly. If you don't want to set up Git/Gerrit, you can also use https://tools.wmflabs.org/gerrit-patch-uploader/

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


Navigation
Links