This throws and error and exception when you try to download a folder that doesn't exist yet.

[kevinsteffer]

CLI/bin/downloader.js

Line 14 in 8c0b627

throw new Error(`No files found in directory '${dirPath}', if you want to download all folders recursively include the -r flag`);

In version 1.0.13 this was only a console.log with a nice "return" - we can live with that.
Now the CLI exists with this throw new Error thing :(
And the error message has a ReferenceError because dirPath doesn't exist.

A disaster in an automated setup.

[dw-jea]

Hi @kevinsteffer,

This has been changed with #39.

[kevinsteffer]

@dw-jea - unfortunately it doesn't quite do it because of the console.error (EDITED) - which stops execution of a script.
And I can't see that the CLI should return an Error just because an attempt to download something is empty.
The console.log and return in v1.0.13 was more automation friendly.

[nicped]

I am changing this to this:

export function tryGetFileNameFromResponse(res, dirPath, verbose = false) {
    try {
        return getFileNameFromResponse(res, dirPath);
    } catch (err) {
        if (verbose) {
            console.log(err.message);
        }
        return null;
    }
}

So no more console.error - and it only shows a log when in verbose.

Will be released in 1.1.1.