ibm-cloud-sdk-core 2.17.13 → 3.0.0

package/CHANGELOG.md

@@ -1,3 +1,34 @@
+# [3.0.0](https://github.com/IBM/node-sdk-core/compare/v2.17.15...v3.0.0) (2022-07-28)
+
+
+### Bug Fixes
+
+* update file-type to resolve vulnerability ([#205](https://github.com/IBM/node-sdk-core/issues/205)) ([843e66d](https://github.com/IBM/node-sdk-core/commit/843e66d10aeb43dc5a0f7c9583f93ce6a5dbaa7e)), closes [#204](https://github.com/IBM/node-sdk-core/issues/204)
+
+
+### BREAKING CHANGES
+
+* two synchronous public functions are now asynchronous
+
+The function `getContentType` formerly returned a string but now returns a
+Promise that resolves to a string. The function `buildRequestFileObject`
+formerly returned a `FileObject` but now returns a Promise that resolves to
+a `FileObject`.
+
+## [2.17.15](https://github.com/IBM/node-sdk-core/compare/v2.17.14...v2.17.15) (2022-07-05)
+
+
+### Bug Fixes
+
+* ensure dist/docs is a directory during build ([#202](https://github.com/IBM/node-sdk-core/issues/202)) ([1b3ef62](https://github.com/IBM/node-sdk-core/commit/1b3ef621e3cbfe0586d433269f63acf77dfa7251))
+
+## [2.17.14](https://github.com/IBM/node-sdk-core/compare/v2.17.13...v2.17.14) (2022-05-02)
+
+
+### Bug Fixes
+
+* do not retry on 501 ([#197](https://github.com/IBM/node-sdk-core/issues/197)) ([5cb9081](https://github.com/IBM/node-sdk-core/commit/5cb9081f581c73b3ba16e470eb4c03c9b8f5c266))
+
 ## [2.17.13](https://github.com/IBM/node-sdk-core/compare/v2.17.12...v2.17.13) (2022-04-04)
 
 

package/build/docs/ibm-cloud-sdk-core.buildrequestfileobject.md

@@ -9,7 +9,7 @@ This function builds a `form-data` object for each file parameter.
 <b>Signature:</b>
 
 ```typescript
-export declare function buildRequestFileObject(fileParam: FileWithMetadata): FileObject;
+export declare function buildRequestFileObject(fileParam: FileWithMetadata): Promise<FileObject>;
 ```
 
 ## Parameters
@@ -20,7 +20,7 @@ export declare function buildRequestFileObject(fileParam: FileWithMetadata): Fil
 
 <b>Returns:</b>
 
-[FileObject](./ibm-cloud-sdk-core.fileobject.md)
+Promise&lt;[FileObject](./ibm-cloud-sdk-core.fileobject.md)<!-- -->&gt;
 
 {<!-- -->FileObject<!-- -->}
 

package/build/docs/ibm-cloud-sdk-core.contenttype.md

@@ -8,7 +8,7 @@
 
 ```typescript
 _default: {
-    fromFilename: (file: String | File | Buffer | NodeJS.ReadableStream | FileObject) => string;
+    fromFilename: (file: String | File | FileObject | NodeJS.ReadableStream | Buffer) => string;
     fromHeader: (buffer: Buffer) => string;
 }
 ```

package/build/docs/ibm-cloud-sdk-core.getcontenttype.md

@@ -9,7 +9,7 @@ This function retrieves the content type of the input.
 <b>Signature:</b>
 
 ```typescript
-export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): string;
+export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): Promise<string>;
 ```
 
 ## Parameters
@@ -20,7 +20,7 @@ export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer
 
 <b>Returns:</b>
 
-string
+Promise&lt;string&gt;
 
 {<!-- -->string<!-- -->} the content type of the input.
 

package/{docs → docs/ibm-cloud-sdk-core.api.json}

@@ -1639,11 +1639,24 @@
               "kind": "Content",
               "text": "): "
             },
+            {
+              "kind": "Reference",
+              "text": "Promise",
+              "canonicalReference": "!Promise:interface"
+            },
+            {
+              "kind": "Content",
+              "text": "<"
+            },
             {
               "kind": "Reference",
               "text": "FileObject",
               "canonicalReference": "ibm-cloud-sdk-core!FileObject:interface"
             },
+            {
+              "kind": "Content",
+              "text": ">"
+            },
             {
               "kind": "Content",
               "text": ";"
@@ -1651,7 +1664,7 @@
           ],
           "returnTypeTokenRange": {
             "startIndex": 3,
-            "endIndex": 4
+            "endIndex": 7
           },
           "releaseTag": "Public",
           "overloadIndex": 1,
@@ -2589,8 +2602,8 @@
             },
             {
               "kind": "Reference",
-              "text": "Buffer",
-              "canonicalReference": "!Buffer:interface"
+              "text": "FileObject",
+              "canonicalReference": "ibm-cloud-sdk-core!FileObject:interface"
             },
             {
               "kind": "Content",
@@ -2607,8 +2620,8 @@
             },
             {
               "kind": "Reference",
-              "text": "FileObject",
-              "canonicalReference": "ibm-cloud-sdk-core!FileObject:interface"
+              "text": "Buffer",
+              "canonicalReference": "!Buffer:interface"
             },
             {
               "kind": "Content",
@@ -3185,9 +3198,14 @@
               "kind": "Content",
               "text": "): "
             },
+            {
+              "kind": "Reference",
+              "text": "Promise",
+              "canonicalReference": "!Promise:interface"
+            },
             {
               "kind": "Content",
-              "text": "string"
+              "text": "<string>"
             },
             {
               "kind": "Content",
@@ -3196,7 +3214,7 @@
           ],
           "returnTypeTokenRange": {
             "startIndex": 5,
-            "endIndex": 6
+            "endIndex": 7
           },
           "releaseTag": "Public",
           "overloadIndex": 1,

package/es/lib/content-type.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" />
 import { FileObject } from "./helper";
 declare const _default: {
-    fromFilename: (file: String | File | Buffer | NodeJS.ReadableStream | FileObject) => string;
+    fromFilename: (file: String | File | FileObject | NodeJS.ReadableStream | Buffer) => string;
     fromHeader: (buffer: Buffer) => string;
 };
 export default _default;

package/es/lib/helper.d.ts

@@ -38,7 +38,7 @@ export declare function isEmptyObject(obj: any): boolean;
  * @param {NodeJS.ReadableStream|Buffer} inputData - The data to retrieve content type for.
  * @returns {string} the content type of the input.
  */
-export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): string;
+export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): Promise<string>;
 /**
  *
  * @param {string} url - the url string.
@@ -104,7 +104,7 @@ export declare function getFormat(params: {
  * @param {string} fileParam.contentType The content type of the file.
  * @returns {FileObject}
  */
-export declare function buildRequestFileObject(fileParam: FileWithMetadata): FileObject;
+export declare function buildRequestFileObject(fileParam: FileWithMetadata): Promise<FileObject>;
 /**
  * This function converts an object's keys to lower case.
  * note: does not convert nested keys

package/es/lib/helper.js

@@ -13,11 +13,20 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import fileType from 'file-type';
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
 import { isReadable } from 'isstream';
 import { lookup } from 'mime-types';
 import { basename } from 'path';
 import logger from './logger';
+const FileType = require('file-type');
 // custom type guards
 function isFileObject(obj) {
     return Boolean(obj && obj.value);
@@ -40,17 +49,19 @@ export function isEmptyObject(obj) {
  * @returns {string} the content type of the input.
  */
 export function getContentType(inputData) {
-    let contentType = null;
-    if (isFileStream(inputData)) {
-        // if the inputData is a NodeJS.ReadableStream
-        const mimeType = lookup(inputData.path); // TODO: cleue quick hack, per type definition path could also be a Buffer
-        contentType = { mime: mimeType || null };
-    }
-    else if (Buffer.isBuffer(inputData)) {
-        // if the inputData is a Buffer
-        contentType = fileType(inputData);
-    }
-    return contentType ? contentType.mime : null;
+    return __awaiter(this, void 0, void 0, function* () {
+        let contentType = null;
+        if (isFileStream(inputData)) {
+            // if the inputData is a NodeJS.ReadableStream
+            const mimeType = lookup(inputData.path); // TODO: cleue quick hack, per type definition path could also be a Buffer
+            contentType = { mime: mimeType || null };
+        }
+        else if (Buffer.isBuffer(inputData)) {
+            // if the inputData is a Buffer
+            contentType = yield FileType.fromBuffer(inputData);
+        }
+        return contentType ? contentType.mime : null;
+    });
 }
 /**
  *
@@ -198,44 +209,47 @@ export function getFormat(params, formats) {
  * @returns {FileObject}
  */
 export function buildRequestFileObject(fileParam) {
-    let fileObj;
-    if (isFileObject(fileParam.data)) {
-        // For backward compatibility, we allow the data to be a FileObject.
-        fileObj = { value: fileParam.data.value, options: {} };
-        if (fileParam.data.options) {
-            fileObj.options = {
-                filename: fileParam.filename || fileParam.data.options.filename,
-                contentType: fileParam.contentType || fileParam.data.options.contentType,
+    return __awaiter(this, void 0, void 0, function* () {
+        let fileObj;
+        if (isFileObject(fileParam.data)) {
+            // For backward compatibility, we allow the data to be a FileObject.
+            fileObj = { value: fileParam.data.value, options: {} };
+            if (fileParam.data.options) {
+                fileObj.options = {
+                    filename: fileParam.filename || fileParam.data.options.filename,
+                    contentType: fileParam.contentType || fileParam.data.options.contentType,
+                };
+            }
+        }
+        else {
+            fileObj = {
+                value: fileParam.data,
+                options: {
+                    filename: fileParam.filename,
+                    contentType: fileParam.contentType,
+                },
             };
         }
-    }
-    else {
-        fileObj = {
-            value: fileParam.data,
-            options: {
-                filename: fileParam.filename,
-                contentType: fileParam.contentType,
-            },
-        };
-    }
-    // Also for backward compatibility, we allow data to be a string
-    if (typeof fileObj.value === 'string') {
-        fileObj.value = Buffer.from(fileObj.value);
-    }
-    // build filename
-    // eslint-disable-next-line prefer-destructuring
-    let filename = fileObj.options.filename;
-    if (!filename && isFileStream(fileObj.value)) {
-        // if readable stream with path property
-        filename = fileObj.value.path;
-    }
-    // toString handles the case when path is a buffer
-    fileObj.options.filename = filename ? basename(filename.toString()) : '_';
-    // build contentType
-    if (!fileObj.options.contentType && isFileData(fileObj.value)) {
-        fileObj.options.contentType = getContentType(fileObj.value) || 'application/octet-stream';
-    }
-    return fileObj;
+        // Also for backward compatibility, we allow data to be a string
+        if (typeof fileObj.value === 'string') {
+            fileObj.value = Buffer.from(fileObj.value);
+        }
+        // build filename
+        // eslint-disable-next-line prefer-destructuring
+        let filename = fileObj.options.filename;
+        if (!filename && isFileStream(fileObj.value)) {
+            // if readable stream with path property
+            filename = fileObj.value.path;
+        }
+        // toString handles the case when path is a buffer
+        fileObj.options.filename = filename ? basename(filename.toString()) : '_';
+        // build contentType
+        if (!fileObj.options.contentType && isFileData(fileObj.value)) {
+            fileObj.options.contentType =
+                (yield getContentType(fileObj.value)) || 'application/octet-stream';
+        }
+        return fileObj;
+    });
 }
 /**
  * This function converts an object's keys to lower case.

package/es/lib/request-wrapper.js

@@ -135,14 +135,14 @@ export class RequestWrapper {
                     // Skip keys with undefined/null values or empty object value
                     values
                         .filter((v) => v != null && !isEmptyObject(v))
-                        .forEach((value) => {
+                        .forEach((value) => __awaiter(this, void 0, void 0, function* () {
                         // Special case of empty file object
                         if (Object.prototype.hasOwnProperty.call(value, 'contentType') &&
                             !Object.prototype.hasOwnProperty.call(value, 'data')) {
                             return;
                         }
                         if (isFileWithMetadata(value)) {
-                            const fileObj = buildRequestFileObject(value);
+                            const fileObj = yield buildRequestFileObject(value);
                             multipartForm.append(key, fileObj.value, fileObj.options);
                         }
                         else {
@@ -151,7 +151,7 @@ export class RequestWrapper {
                             }
                             multipartForm.append(key, value);
                         }
-                    });
+                    }));
                 });
             }
             // Path params
@@ -290,6 +290,12 @@ export class RequestWrapper {
             retry: 4,
             retryDelay: 1000,
             httpMethodsToRetry: ['GET', 'HEAD', 'OPTIONS', 'DELETE', 'PUT', 'POST'],
+            // do not retry on 501
+            statusCodesToRetry: [
+                [429, 429],
+                [500, 500],
+                [502, 599],
+            ],
             instance: axiosInstance,
             backoffType: 'exponential',
             checkRetryAfter: true,

package/etc/ibm-cloud-sdk-core.api.md

@@ -94,7 +94,7 @@ export class BearerTokenAuthenticator extends Authenticator {
 }
 
 // @public
-export function buildRequestFileObject(fileParam: FileWithMetadata): FileObject;
+export function buildRequestFileObject(fileParam: FileWithMetadata): Promise<FileObject>;
 
 // @public
 export function checkCredentials(obj: any, credsToCheck: string[]): Error | null;
@@ -155,7 +155,7 @@ export class ContainerTokenManager extends IamRequestBasedTokenManager {
 
 // @public (undocumented)
 export const contentType: {
-    fromFilename: (file: String | File | Buffer | NodeJS.ReadableStream | FileObject) => string;
+    fromFilename: (file: String | File | FileObject | NodeJS.ReadableStream | Buffer) => string;
     fromHeader: (buffer: Buffer) => string;
 };
 
@@ -211,7 +211,7 @@ export interface FileWithMetadata {
 export function getAuthenticatorFromEnvironment(serviceName: string): Authenticator;
 
 // @public
-export function getContentType(inputData: NodeJS.ReadableStream | Buffer): string;
+export function getContentType(inputData: NodeJS.ReadableStream | Buffer): Promise<string>;
 
 // @public
 export function getCurrentTime(): number;

package/ibm-cloud-sdk-core.d.ts

@@ -319,7 +319,7 @@ export declare class BaseService {
      * @param {string} fileParam.contentType The content type of the file.
      * @returns {FileObject}
      */
-    export declare function buildRequestFileObject(fileParam: FileWithMetadata): FileObject;
+    export declare function buildRequestFileObject(fileParam: FileWithMetadata): Promise<FileObject>;
 
     /**
      * Checks credentials for common user mistakes of copying {, }, or " characters from the documentation
@@ -584,7 +584,7 @@ export declare class BaseService {
                 }
 
                 export declare const contentType: {
-                    fromFilename: (file: String | File | Buffer | NodeJS.ReadableStream | FileObject) => string;
+                    fromFilename: (file: String | File | FileObject | NodeJS.ReadableStream | Buffer) => string;
                     fromHeader: (buffer: Buffer) => string;
                 };
 
@@ -683,7 +683,7 @@ export declare class BaseService {
                      * @param {NodeJS.ReadableStream|Buffer} inputData - The data to retrieve content type for.
                      * @returns {string} the content type of the input.
                      */
-                    export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): string;
+                    export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): Promise<string>;
 
                     /**
                      * Get the current time

package/lib/content-type.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" />
 import { FileObject } from "./helper";
 declare const _default: {
-    fromFilename: (file: String | File | Buffer | NodeJS.ReadableStream | FileObject) => string;
+    fromFilename: (file: String | File | FileObject | NodeJS.ReadableStream | Buffer) => string;
     fromHeader: (buffer: Buffer) => string;
 };
 export default _default;

package/lib/helper.d.ts

@@ -38,7 +38,7 @@ export declare function isEmptyObject(obj: any): boolean;
  * @param {NodeJS.ReadableStream|Buffer} inputData - The data to retrieve content type for.
  * @returns {string} the content type of the input.
  */
-export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): string;
+export declare function getContentType(inputData: NodeJS.ReadableStream | Buffer): Promise<string>;
 /**
  *
  * @param {string} url - the url string.
@@ -104,7 +104,7 @@ export declare function getFormat(params: {
  * @param {string} fileParam.contentType The content type of the file.
  * @returns {FileObject}
  */
-export declare function buildRequestFileObject(fileParam: FileWithMetadata): FileObject;
+export declare function buildRequestFileObject(fileParam: FileWithMetadata): Promise<FileObject>;
 /**
  * This function converts an object's keys to lower case.
  * note: does not convert nested keys

package/lib/helper.js

@@ -14,6 +14,42 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __generator = (this && this.__generator) || function (thisArg, body) {
+    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g;
+    return g = { next: verb(0), "throw": verb(1), "return": verb(2) }, typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
+    function verb(n) { return function (v) { return step([n, v]); }; }
+    function step(op) {
+        if (f) throw new TypeError("Generator is already executing.");
+        while (_) try {
+            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
+            if (y = 0, t) op = [op[0] & 2, t.value];
+            switch (op[0]) {
+                case 0: case 1: t = op; break;
+                case 4: _.label++; return { value: op[1], done: false };
+                case 5: _.label++; y = op[1]; op = [0]; continue;
+                case 7: op = _.ops.pop(); _.trys.pop(); continue;
+                default:
+                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
+                    if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
+                    if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
+                    if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
+                    if (t[2]) _.ops.pop();
+                    _.trys.pop(); continue;
+            }
+            op = body.call(thisArg, _);
+        } catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
+        if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
+    }
+};
 var __spreadArrays = (this && this.__spreadArrays) || function () {
     for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
     for (var r = Array(s), k = 0, i = 0; i < il; i++)
@@ -25,11 +61,11 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-var file_type_1 = __importDefault(require("file-type"));
 var isstream_1 = require("isstream");
 var mime_types_1 = require("mime-types");
 var path_1 = require("path");
 var logger_1 = __importDefault(require("./logger"));
+var FileType = require('file-type');
 // custom type guards
 function isFileObject(obj) {
     return Boolean(obj && obj.value);
@@ -55,17 +91,27 @@ exports.isEmptyObject = isEmptyObject;
  * @returns {string} the content type of the input.
  */
 function getContentType(inputData) {
-    var contentType = null;
-    if (isFileStream(inputData)) {
-        // if the inputData is a NodeJS.ReadableStream
-        var mimeType = mime_types_1.lookup(inputData.path); // TODO: cleue quick hack, per type definition path could also be a Buffer
-        contentType = { mime: mimeType || null };
-    }
-    else if (Buffer.isBuffer(inputData)) {
-        // if the inputData is a Buffer
-        contentType = file_type_1.default(inputData);
-    }
-    return contentType ? contentType.mime : null;
+    return __awaiter(this, void 0, void 0, function () {
+        var contentType, mimeType;
+        return __generator(this, function (_a) {
+            switch (_a.label) {
+                case 0:
+                    contentType = null;
+                    if (!isFileStream(inputData)) return [3 /*break*/, 1];
+                    mimeType = mime_types_1.lookup(inputData.path);
+                    contentType = { mime: mimeType || null };
+                    return [3 /*break*/, 3];
+                case 1:
+                    if (!Buffer.isBuffer(inputData)) return [3 /*break*/, 3];
+                    return [4 /*yield*/, FileType.fromBuffer(inputData)];
+                case 2:
+                    // if the inputData is a Buffer
+                    contentType = _a.sent();
+                    _a.label = 3;
+                case 3: return [2 /*return*/, contentType ? contentType.mime : null];
+            }
+        });
+    });
 }
 exports.getContentType = getContentType;
 /**
@@ -220,44 +266,52 @@ exports.getFormat = getFormat;
  * @returns {FileObject}
  */
 function buildRequestFileObject(fileParam) {
-    var fileObj;
-    if (isFileObject(fileParam.data)) {
-        // For backward compatibility, we allow the data to be a FileObject.
-        fileObj = { value: fileParam.data.value, options: {} };
-        if (fileParam.data.options) {
-            fileObj.options = {
-                filename: fileParam.filename || fileParam.data.options.filename,
-                contentType: fileParam.contentType || fileParam.data.options.contentType,
-            };
-        }
-    }
-    else {
-        fileObj = {
-            value: fileParam.data,
-            options: {
-                filename: fileParam.filename,
-                contentType: fileParam.contentType,
-            },
-        };
-    }
-    // Also for backward compatibility, we allow data to be a string
-    if (typeof fileObj.value === 'string') {
-        fileObj.value = Buffer.from(fileObj.value);
-    }
-    // build filename
-    // eslint-disable-next-line prefer-destructuring
-    var filename = fileObj.options.filename;
-    if (!filename && isFileStream(fileObj.value)) {
-        // if readable stream with path property
-        filename = fileObj.value.path;
-    }
-    // toString handles the case when path is a buffer
-    fileObj.options.filename = filename ? path_1.basename(filename.toString()) : '_';
-    // build contentType
-    if (!fileObj.options.contentType && isFileData(fileObj.value)) {
-        fileObj.options.contentType = getContentType(fileObj.value) || 'application/octet-stream';
-    }
-    return fileObj;
+    return __awaiter(this, void 0, void 0, function () {
+        var fileObj, filename, _a;
+        return __generator(this, function (_b) {
+            switch (_b.label) {
+                case 0:
+                    if (isFileObject(fileParam.data)) {
+                        // For backward compatibility, we allow the data to be a FileObject.
+                        fileObj = { value: fileParam.data.value, options: {} };
+                        if (fileParam.data.options) {
+                            fileObj.options = {
+                                filename: fileParam.filename || fileParam.data.options.filename,
+                                contentType: fileParam.contentType || fileParam.data.options.contentType,
+                            };
+                        }
+                    }
+                    else {
+                        fileObj = {
+                            value: fileParam.data,
+                            options: {
+                                filename: fileParam.filename,
+                                contentType: fileParam.contentType,
+                            },
+                        };
+                    }
+                    // Also for backward compatibility, we allow data to be a string
+                    if (typeof fileObj.value === 'string') {
+                        fileObj.value = Buffer.from(fileObj.value);
+                    }
+                    filename = fileObj.options.filename;
+                    if (!filename && isFileStream(fileObj.value)) {
+                        // if readable stream with path property
+                        filename = fileObj.value.path;
+                    }
+                    // toString handles the case when path is a buffer
+                    fileObj.options.filename = filename ? path_1.basename(filename.toString()) : '_';
+                    if (!(!fileObj.options.contentType && isFileData(fileObj.value))) return [3 /*break*/, 2];
+                    _a = fileObj.options;
+                    return [4 /*yield*/, getContentType(fileObj.value)];
+                case 1:
+                    _a.contentType =
+                        (_b.sent()) || 'application/octet-stream';
+                    _b.label = 2;
+                case 2: return [2 /*return*/, fileObj];
+            }
+        });
+    });
 }
 exports.buildRequestFileObject = buildRequestFileObject;
 /**

package/lib/request-wrapper.js

@@ -191,23 +191,32 @@ var RequestWrapper = /** @class */ (function () {
                                 // Skip keys with undefined/null values or empty object value
                                 values
                                     .filter(function (v) { return v != null && !helper_1.isEmptyObject(v); })
-                                    .forEach(function (value) {
-                                    // Special case of empty file object
-                                    if (Object.prototype.hasOwnProperty.call(value, 'contentType') &&
-                                        !Object.prototype.hasOwnProperty.call(value, 'data')) {
-                                        return;
-                                    }
-                                    if (helper_1.isFileWithMetadata(value)) {
-                                        var fileObj = helper_1.buildRequestFileObject(value);
-                                        multipartForm.append(key, fileObj.value, fileObj.options);
-                                    }
-                                    else {
-                                        if (typeof value === 'object' && !helper_1.isFileData(value)) {
-                                            value = JSON.stringify(value);
+                                    .forEach(function (value) { return __awaiter(_this, void 0, void 0, function () {
+                                    var fileObj;
+                                    return __generator(this, function (_a) {
+                                        switch (_a.label) {
+                                            case 0:
+                                                // Special case of empty file object
+                                                if (Object.prototype.hasOwnProperty.call(value, 'contentType') &&
+                                                    !Object.prototype.hasOwnProperty.call(value, 'data')) {
+                                                    return [2 /*return*/];
+                                                }
+                                                if (!helper_1.isFileWithMetadata(value)) return [3 /*break*/, 2];
+                                                return [4 /*yield*/, helper_1.buildRequestFileObject(value)];
+                                            case 1:
+                                                fileObj = _a.sent();
+                                                multipartForm.append(key, fileObj.value, fileObj.options);
+                                                return [3 /*break*/, 3];
+                                            case 2:
+                                                if (typeof value === 'object' && !helper_1.isFileData(value)) {
+                                                    value = JSON.stringify(value);
+                                                }
+                                                multipartForm.append(key, value);
+                                                _a.label = 3;
+                                            case 3: return [2 /*return*/];
                                         }
-                                        multipartForm.append(key, value);
-                                    }
-                                });
+                                    });
+                                }); });
                             });
                         }
                         // Path params
@@ -350,6 +359,12 @@ var RequestWrapper = /** @class */ (function () {
             retry: 4,
             retryDelay: 1000,
             httpMethodsToRetry: ['GET', 'HEAD', 'OPTIONS', 'DELETE', 'PUT', 'POST'],
+            // do not retry on 501
+            statusCodesToRetry: [
+                [429, 429],
+                [500, 500],
+                [502, 599],
+            ],
             instance: axiosInstance,
             backoffType: 'exponential',
             checkRetryAfter: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ibm-cloud-sdk-core",
-  "version": "2.17.13",
+  "version": "3.0.0",
   "description": "Core functionality to support SDKs generated with IBM's OpenAPI SDK Generator.",
   "main": "index.js",
   "typings": "./es/index.d.ts",
@@ -49,7 +49,7 @@
     "dotenv": "^6.2.0",
     "expect": "^26.1.0",
     "extend": "^3.0.2",
-    "file-type": "^7.7.1",
+    "file-type": "16.5.4",
     "form-data": "^2.3.3",
     "isstream": "~0.1.2",
     "jsonwebtoken": "^8.5.1",
@@ -87,7 +87,7 @@
     "build:api": "api-extractor run --local",
     "build:md": "api-documenter markdown -i temp --output-folder build/docs",
     "build:doc": "npm-run-all build:api build:md copy:doc",
-    "copy:doc": "cp \"temp/ibm-cloud-sdk-core.api.json\" dist/docs",
+    "copy:doc": "mkdir -p dist/docs && cp \"temp/ibm-cloud-sdk-core.api.json\" dist/docs",
     "copy:pkg": "package-json-reducer -s \"config devDependencies directories scripts jestSonar jest\" -o ./dist/package.json package.json",
     "postversion": "publisher --no-checks --dry-run",
     "all": "npm-run-all build test lint"

package/temp/ibm-cloud-sdk-core.api.json

@@ -1639,11 +1639,24 @@
               "kind": "Content",
               "text": "): "
             },
+            {
+              "kind": "Reference",
+              "text": "Promise",
+              "canonicalReference": "!Promise:interface"
+            },
+            {
+              "kind": "Content",
+              "text": "<"
+            },
             {
               "kind": "Reference",
               "text": "FileObject",
               "canonicalReference": "ibm-cloud-sdk-core!FileObject:interface"
             },
+            {
+              "kind": "Content",
+              "text": ">"
+            },
             {
               "kind": "Content",
               "text": ";"
@@ -1651,7 +1664,7 @@
           ],
           "returnTypeTokenRange": {
             "startIndex": 3,
-            "endIndex": 4
+            "endIndex": 7
           },
           "releaseTag": "Public",
           "overloadIndex": 1,
@@ -2589,8 +2602,8 @@
             },
             {
               "kind": "Reference",
-              "text": "Buffer",
-              "canonicalReference": "!Buffer:interface"
+              "text": "FileObject",
+              "canonicalReference": "ibm-cloud-sdk-core!FileObject:interface"
             },
             {
               "kind": "Content",
@@ -2607,8 +2620,8 @@
             },
             {
               "kind": "Reference",
-              "text": "FileObject",
-              "canonicalReference": "ibm-cloud-sdk-core!FileObject:interface"
+              "text": "Buffer",
+              "canonicalReference": "!Buffer:interface"
             },
             {
               "kind": "Content",
@@ -3185,9 +3198,14 @@
               "kind": "Content",
               "text": "): "
             },
+            {
+              "kind": "Reference",
+              "text": "Promise",
+              "canonicalReference": "!Promise:interface"
+            },
             {
               "kind": "Content",
-              "text": "string"
+              "text": "<string>"
             },
             {
               "kind": "Content",
@@ -3196,7 +3214,7 @@
           ],
           "returnTypeTokenRange": {
             "startIndex": 5,
-            "endIndex": 6
+            "endIndex": 7
           },
           "releaseTag": "Public",
           "overloadIndex": 1,

package/temp/ibm-cloud-sdk-core.api.md

@@ -94,7 +94,7 @@ export class BearerTokenAuthenticator extends Authenticator {
 }
 
 // @public
-export function buildRequestFileObject(fileParam: FileWithMetadata): FileObject;
+export function buildRequestFileObject(fileParam: FileWithMetadata): Promise<FileObject>;
 
 // @public
 export function checkCredentials(obj: any, credsToCheck: string[]): Error | null;
@@ -155,7 +155,7 @@ export class ContainerTokenManager extends IamRequestBasedTokenManager {
 
 // @public (undocumented)
 export const contentType: {
-    fromFilename: (file: String | File | Buffer | NodeJS.ReadableStream | FileObject) => string;
+    fromFilename: (file: String | File | FileObject | NodeJS.ReadableStream | Buffer) => string;
     fromHeader: (buffer: Buffer) => string;
 };
 
@@ -211,7 +211,7 @@ export interface FileWithMetadata {
 export function getAuthenticatorFromEnvironment(serviceName: string): Authenticator;
 
 // @public
-export function getContentType(inputData: NodeJS.ReadableStream | Buffer): string;
+export function getContentType(inputData: NodeJS.ReadableStream | Buffer): Promise<string>;
 
 // @public
 export function getCurrentTime(): number;