Issue with encoding = null about curlrequest HOT 2 CLOSED

I also ran into the problem of using "encoding = null". It turns out that the problem starts within the "copy" function within index.js. When the object copier finds a property that's assigned to null, it replaces it with an empty object. Thus, "encoding = null" becomes "encoding = {}"

Here's one possible solution to that problem:

exports.copy = function (obj) {
    var copy = {};
    for (var i in obj) {
        if (typeof obj[i] === 'object') {
            // handle properties that are set to null
            copy[i] = (obj[i] === null) ? null : exports.copy(obj[i]);
        } else {
            copy[i] = obj[i];
        }
    }
    return copy;
};

The next problem is that it now throws a different error: Failed to initialize.

This is because curl is now receiving the command line arguments --encoding null, due to the way the args are processed/composed, and can be fixed like this:

        values.forEach(function (value) {
            // don't send a flag for null values
            if (value !== null) {
                args.push('--' + key);
                if (true !== value) {
                    args.push(value);
                }
            }
        });

This library is exceptionally handy, and I'd love to see it continue to improve!

from curlrequest.

I've just pushed 0.4.1 which should fix the issue

from curlrequest.