Page MenuHomePhabricator

UW should not try to upload Flickr videos (resulting in: "This file might be corrupt, or have the wrong extension")
Open, LowPublic


When given a set with videos, UploadWizard Flickr warns about being unable to generate a thumbnail, but agrees to upload the files.

This results in a broken file

Its extension is JPG, its MIME type (according to the API) is video/mp4.

Rename to.mp4 is not possible (fails with “The new file extension does not match its type”)

If relevant: TimedMediaHandler tries to transcode the file, and fails with errors:

'/usr/bin/avconv' -y -i '/tmp/localcopy_4fda164b5870-1.jpg' -threads 1 -skip_threshold 0 -bufsize 6000k -rc_init_occupancy 4000 -qmin 1 -qmax 51 -vb '512000' -vcodec libvpx -g '128' -keyint_min '128' -f webm -s 568x320 -an -pass '1' -passlogfile '/tmp/transcode_360p.webm636dbdaf1f79-1.webm.log' /dev/null

Exitcode: 1
Memory: 4194304

avconv version 0.8.5-4:0.8.5-0ubuntu0.12.04.1, Copyright (c) 2000-2012 the Libav developers

built on Jan 24 2013 18:01:36 with gcc 4.6.3

[image2 @ 0x9b27a0] Could not find codec parameters (Video: mjpeg)
/tmp/localcopy_4fda164b5870-1.jpg: could not find codec parameters

Version: unspecified
Severity: enhancement



Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 1:24 AM
bzimport added a project: UploadWizard.
bzimport set Reference to bz45389.
bzimport added a subscriber: Unknown Object (MLST).

Ccing Ryan to get his opinion,

I think the Flickr upload was meant to support only images currently as the button says "Add images from Flickr". So the best bet we have currently is to just disable videos. This can be done quite easily by just adding the media parameter as per

Due to the security vulnerability in chunked uploads being fixed (bug 48306), it is not possible to upload mp4 files with jpeg extension.

Thus lowering priority and rephrasing summary