response undefined when error occurs.

[milkywayriver]

	signedRequest(base+endpoint, opt, function(error, response) {
		if ( typeof response.msg !== 'undefined' && response.msg === 'Filter failure: MIN_NOTIONAL' ) {
			console.error('Order quantity too small. See exchangeInfo() for minimum amounts');
		}
		if ( callback ) callback(error, response);
		else console.log(side+'('+symbol+','+quantity+','+price+') ',response);
	}, 'POST');
};

[jaggedsoft]

Thanks. Looking in to this

[milkywayriver]

TypeError: Cannot read property 'msg' of undefined
/node_modules/node-binance-api/node-binance-api.js:143:25

/node_modules/node-binance-api/node-binance-api.js:143
if ( typeof response.msg !== 'undefined' && response.msg === 'Filter failure: MIN_NOTIONAL' ) {

[jaggedsoft]

Perhaps something like this would work

		signedRequest(base+endpoint, opt, function(error, response) {
			if ( !response ) {
				if ( callback ) callback(error, response);
				else console.error('Order() error:', error);
				return;
			}
			if ( typeof response.msg !== 'undefined' && response.msg === 'Filter failure: MIN_NOTIONAL' ) {
				console.error('Order quantity too small. See exchangeInfo() for minimum amounts');
			}
			if ( callback ) callback(error, response);
			else console.log(side+'('+symbol+','+quantity+','+price+') ',response);
		}, 'POST');

[milkywayriver]

could be. please fix and i'll test.

[jaggedsoft]

live in v0.4.3
jaggedsoft@b5bb192

[jaggedsoft]

For anyone else reviewing this, it might be better to check if ( error ) instead, not sure the best way to handle this.