@aws-amplify/predictions 4.0.35 → 4.0.36

package/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.0.36](https://github.com/aws-amplify/amplify-js/compare/@aws-amplify/predictions@4.0.35...@aws-amplify/predictions@4.0.36) (2022-03-28)
+
+**Note:** Version bump only for package @aws-amplify/predictions
+
+
+
+
+
 ## [4.0.35](https://github.com/aws-amplify/amplify-js/compare/@aws-amplify/predictions@4.0.34...@aws-amplify/predictions@4.0.35) (2022-03-22)
 
 **Note:** Version bump only for package @aws-amplify/predictions

package/dist/aws-amplify-predictions.js

@@ -98022,6 +98022,10 @@ function isBlob(body) {
   return typeof Blob !== 'undefined' && body instanceof Blob;
 }
 
+function hasErrorResponse(error) {
+  return typeof error !== 'undefined' && Object.prototype.hasOwnProperty.call(error, 'response') && typeof error.response !== 'undefined' && Object.prototype.hasOwnProperty.call(error.response, 'status') && typeof error.response.status === 'number';
+}
+
 var normalizeHeaders = function normalizeHeaders(headers, normalizedName) {
   var e_1, _a;
 
@@ -98165,16 +98169,20 @@ function () {
         })
       };
     })["catch"](function (error) {
-      var _a, _b, _c; // Error
+      var _a, _b; // Error
 
 
       if (error.message !== _common_StorageErrorStrings__WEBPACK_IMPORTED_MODULE_4__["AWSS3ProviderUploadErrorStrings"].UPLOAD_PAUSED_MESSAGE) {
         logger.error(error.message);
       } // for axios' cancel error, we should re-throw it back so it's not considered an s3client error
-      // if we return empty, or an abitrary error HttpResponse, it will be hard to debug down the line
+      // if we return empty, or an abitrary error HttpResponse, it will be hard to debug down the line.
+      //
+      // for errors that does not have a 'response' object, it's very likely that it is an unexpected error for
+      // example a disconnect. Without it we cannot meaningfully reconstruct a HttpResponse, and the AWS SDK might
+      // consider the request successful by mistake. In this case we should also re-throw the error.
 
 
-      if (axios__WEBPACK_IMPORTED_MODULE_2___default.a.isCancel(error)) {
+      if (axios__WEBPACK_IMPORTED_MODULE_2___default.a.isCancel(error) || !hasErrorResponse(error)) {
         throw error;
       } // otherwise, we should re-construct an HttpResponse from the error, so that it can be passed down to other
       // aws sdk middleware (e.g retry, clock skew correction, error message serializing)
@@ -98182,9 +98190,9 @@ function () {
 
       return {
         response: new _aws_sdk_protocol_http__WEBPACK_IMPORTED_MODULE_0__["HttpResponse"]({
-          statusCode: (_a = error.response) === null || _a === void 0 ? void 0 : _a.status,
-          body: (_b = error.response) === null || _b === void 0 ? void 0 : _b.data,
-          headers: (_c = error.response) === null || _c === void 0 ? void 0 : _c.headers
+          statusCode: error.response.status,
+          body: (_a = error.response) === null || _a === void 0 ? void 0 : _a.data,
+          headers: (_b = error.response) === null || _b === void 0 ? void 0 : _b.headers
         })
       };
     }), requestTimeout(requestTimeoutInMs)];