Make rejected promise always yield error instance

Author: ncjones

lib/requests/http.js

@@ -88,21 +88,25 @@ var httpRequest = function (options) {
                     break;
                 }
                 default: // All other statuses are error cases.
-                    var error;
+                    var crmError;
                     try {
                         var errorParsed = JSON.parse(rawData);
 
-                        error = errorParsed.hasOwnProperty('error') && errorParsed.error
+                        crmError = errorParsed.hasOwnProperty('error') && errorParsed.error
                             ? errorParsed.error
                             : { message: errorParsed.Message };
                     } catch (e) {
                         if (rawData.length > 0) {
-                            error = { message: rawData };
+                            crmError = { message: rawData };
                         }
                         else {
-                            error = { message: "Unexpected Error" };
+                            crmError = { message: "Unexpected Error" };
                         }
                     }
+                    var error = new Error();
+                    Object.keys(crmError).forEach(k => {
+                      error[k] = crmError[k];
+                    })
                     error.status = res.statusCode;
                     error.statusText = request.statusText;
                     errorCallback(error);