Possible bug in upload handling (when other parameters are provided) about nodemw

[macbre]

@minecrafter, please log the values of file and from variables or create the nodemw client with debug config option set to true - this will make console logging more verbose.

[minecrafter]

Here's a run of the script with uploadByUrl: http://paste.ubuntu.com/11726991/

For now I just reimplemented this part of the script using MediaWiki's native upload-by-URL capabilities, though I'd love to use a nicer API for this.

[macbre]

The log contains huge amount of repeated files and URL fetches.

Some URLs can not be fetched from the origin:

error:   Failed to fetch <http://media1.clubpenguin.com/play/v2/content/local/en/news/papers/20150610/content/overlays/supportMore.swf>
error:   HTTP status 404
error:   Unable to upload ENNews503SupportMore.swf Error: HTTP status 404

Please remove the duplicates and provide the log of a single failing upload. It's a challenge currently to dig through the log :)

[minecrafter]

Modified the code to upload one random file from the selection, here's the log from that: http://paste.ubuntu.com/11727420/

The origin doesn't have some of the files and that's intended, I'll be cleaning that up.

[macbre]

@minecrafter, I somehow missed that you're using iojs. Are you able to run your code on node.js and reproduce the issue?

[minecrafter]

I can take a look at the issue with node 0.10 in a bit.

[minecrafter]

I managed to get some time to look into the issue. Under node 0.10 the issue is still present.

[macbre]

@minecrafter: logs from iojs are kind of mysterious (Request to API failed: %s Error: Argument error, options.body. - not showing the parameters). Can you provide logs from nodejs run?

[minecrafter]

I'm going to update to ~0.7.0 and try it again. You want node 0.10 or 4.x?

[macbre]

Let's try with 4.x - the newer the better ;)

[minecrafter]

Doesn't work: https://gist.github.com/minecrafter/394ed57817648a32c6a4

Running node v4.0.0

[macbre]

@minecrafter: can you skip ignorewarnings: false option in a call to uploadByUrl? Or replace it with "0" (as a string)?

[minecrafter]

Looks like that worked.

[macbre]

Great!