IMPORTANT! Failed transaction still considered as successful!

[artofspeed]

This is an urgent update. The callback for DoExpressCheckoutPayment should check if transaction is successful, it currently checks if getting the GetExpressCheckoutPayment is successful, which is wrong!

To fix, change

if (data.ACK  !== 'Success') {
    return callback(new Error('Error DoExpressCheckoutPayment'), data2);
}

into

if (data.ACK  !== 'Success' || data2.ACK !== 'Success') {
    return callback(new Error('Error DoExpressCheckoutPayment'), data2);
}