Opened 7 years ago
Closed 6 years ago
#5458 closed defect (fixed)
batchaddmedia does not properly parse the "media:location" header field
|Reported by:||Tim McIver||Owned by:|
The documentation here: http://mediagoblin.readthedocs.io/en/stable/siteadmin/commandline-upload.html#command-line-batch-uploading states that the path to a media file is given by the "media:location" header field but using this header field name causes a parse error.
Looking at the code here: http://git.savannah.gnu.org/cgit/mediagoblin.git/tree/mediagoblin/gmg_commands/batchaddmedia.py#n111 it seems the code is expecting the header field name to be "location". So, I guess either a code change or a documentation change is needed.
Change History (4)
comment:1 by , 7 years ago
|Priority:||major → minor|
comment:2 by , 6 years ago
comment:3 by , 6 years ago
comment:4 by , 6 years ago
|Status:||new → closed|
I was bitten by the issue while setting up MediaGoblin for the first time, following the guide at http://mediagoblin.readthedocs.io/en/stable/siteadmin/commandline-upload.html#command-line-batch-uploading.
It would be good to align the documentation and the behavior of the code. I can volunteer to create a patch for it, but I need to know who is right in this case: the documentation or the code?