@pnp/cli-microsoft365 6.0.0-beta.2979870 → 6.0.0-beta.3261c97

package/dist/m365/spo/commands/sitescript/sitescript-set.js

@@ -33,46 +33,43 @@ class SpoSiteScriptSetCommand extends SpoCommand_1.default {
     get description() {
         return 'Updates existing site script';
     }
-    commandAction(logger, args, cb) {
-        let spoUrl = '';
-        utils_1.spo
-            .getSpoUrl(logger, this.debug)
-            .then((_spoUrl) => {
-            spoUrl = _spoUrl;
-            return utils_1.spo.getRequestDigest(spoUrl);
-        })
-            .then((res) => {
-            const updateInfo = {
-                Id: args.options.id
-            };
-            if (args.options.title) {
-                updateInfo.Title = args.options.title;
-            }
-            if (args.options.description) {
-                updateInfo.Description = args.options.description;
-            }
-            if (args.options.version) {
-                updateInfo.Version = parseInt(args.options.version);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const spoUrl = yield utils_1.spo.getSpoUrl(logger, this.debug);
+                const formDigest = yield utils_1.spo.getRequestDigest(spoUrl);
+                const updateInfo = {
+                    Id: args.options.id
+                };
+                if (args.options.title) {
+                    updateInfo.Title = args.options.title;
+                }
+                if (args.options.description) {
+                    updateInfo.Description = args.options.description;
+                }
+                if (args.options.version) {
+                    updateInfo.Version = parseInt(args.options.version);
+                }
+                if (args.options.content) {
+                    updateInfo.Content = args.options.content;
+                }
+                const requestOptions = {
+                    url: `${spoUrl}/_api/Microsoft.Sharepoint.Utilities.WebTemplateExtensions.SiteScriptUtility.UpdateSiteScript`,
+                    headers: {
+                        'X-RequestDigest': formDigest.FormDigestValue,
+                        'content-type': 'application/json;charset=utf-8',
+                        accept: 'application/json;odata=nometadata'
+                    },
+                    data: { updateInfo: updateInfo },
+                    responseType: 'json'
+                };
+                const res = yield request_1.default.post(requestOptions);
+                logger.log(res);
             }
-            if (args.options.content) {
-                updateInfo.Content = args.options.content;
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
-            const requestOptions = {
-                url: `${spoUrl}/_api/Microsoft.Sharepoint.Utilities.WebTemplateExtensions.SiteScriptUtility.UpdateSiteScript`,
-                headers: {
-                    'X-RequestDigest': res.FormDigestValue,
-                    'content-type': 'application/json;charset=utf-8',
-                    accept: 'application/json;odata=nometadata'
-                },
-                data: { updateInfo: updateInfo },
-                responseType: 'json'
-            };
-            return request_1.default.post(requestOptions);
-        })
-            .then((res) => {
-            logger.log(res);
-            cb();
-        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+        });
     }
 }
 _SpoSiteScriptSetCommand_instances = new WeakSet(), _SpoSiteScriptSetCommand_initTelemetry = function _SpoSiteScriptSetCommand_initTelemetry() {

package/dist/m365/spo/commands/spo-get.js

@@ -1,4 +1,13 @@
 "use strict";
+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());
+    });
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 const Auth_1 = require("../../../Auth");
 const SpoCommand_1 = require("../../base/SpoCommand");
@@ -10,12 +19,13 @@ class SpoGetCommand extends SpoCommand_1.default {
     get description() {
         return 'Gets the context URL for the root SharePoint site collection and SharePoint tenant admin site';
     }
-    commandAction(logger, args, cb) {
-        const spoContext = {
-            SpoUrl: Auth_1.default.service.spoUrl ? Auth_1.default.service.spoUrl : ''
-        };
-        logger.log(spoContext);
-        cb();
+    commandAction(logger) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const spoContext = {
+                SpoUrl: Auth_1.default.service.spoUrl ? Auth_1.default.service.spoUrl : ''
+            };
+            logger.log(spoContext);
+        });
     }
 }
 module.exports = new SpoGetCommand();

package/dist/m365/spo/commands/spo-search.js

@@ -34,28 +34,27 @@ class SpoSearchCommand extends SpoCommand_1.default {
     get description() {
         return 'Executes a search query';
     }
-    commandAction(logger, args, cb) {
-        let webUrl = '';
-        (() => {
-            if (args.options.webUrl) {
-                return Promise.resolve(args.options.webUrl);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            let webUrl = '';
+            try {
+                if (args.options.webUrl) {
+                    webUrl = args.options.webUrl;
+                }
+                else {
+                    webUrl = yield utils_1.spo.getSpoUrl(logger, this.debug);
+                }
+                if (this.verbose) {
+                    logger.logToStderr(`Executing search query '${args.options.queryText}' on site at ${webUrl}...`);
+                }
+                const startRow = args.options.startRow ? args.options.startRow : 0;
+                const results = yield this.executeSearchQuery(logger, args, webUrl, [], startRow);
+                this.printResults(logger, args, results);
             }
-            else {
-                return utils_1.spo.getSpoUrl(logger, this.debug);
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
-        })()
-            .then((_webUrl) => {
-            webUrl = _webUrl;
-            if (this.verbose) {
-                logger.logToStderr(`Executing search query '${args.options.queryText}' on site at ${webUrl}...`);
-            }
-            const startRow = args.options.startRow ? args.options.startRow : 0;
-            return this.executeSearchQuery(logger, args, webUrl, [], startRow);
-        })
-            .then((results) => {
-            this.printResults(logger, args, results);
-            cb();
-        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+        });
     }
     executeSearchQuery(logger, args, webUrl, resultSet, startRow) {
         return (() => {

package/dist/m365/spo/commands/spo-set.js

@@ -16,7 +16,6 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 var _SpoSetCommand_instances, _SpoSetCommand_initOptions, _SpoSetCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
 const Auth_1 = require("../../../Auth");
-const Command_1 = require("../../../Command");
 const utils_1 = require("../../../utils");
 const SpoCommand_1 = require("../../base/SpoCommand");
 const commands_1 = require("../commands");
@@ -33,12 +32,15 @@ class SpoSetCommand extends SpoCommand_1.default {
     get description() {
         return 'Sets the URL of the root SharePoint site collection for use in SPO commands';
     }
-    commandAction(logger, args, cb) {
-        Auth_1.default.service.spoUrl = args.options.url;
-        Auth_1.default.storeConnectionInfo().then(() => {
-            cb();
-        }, err => {
-            cb(new Command_1.CommandError(err));
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            Auth_1.default.service.spoUrl = args.options.url;
+            try {
+                yield Auth_1.default.storeConnectionInfo();
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
+            }
         });
     }
 }

package/dist/m365/spo/commands/storageentity/storageentity-get.js

@@ -1,4 +1,13 @@
 "use strict";
+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 __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, state, kind, f) {
     if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a getter");
     if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
@@ -22,35 +31,36 @@ class SpoStorageEntityGetCommand extends SpoCommand_1.default {
     get description() {
         return 'Get details for the specified tenant property';
     }
-    commandAction(logger, args, cb) {
-        utils_1.spo
-            .getSpoUrl(logger, this.debug)
-            .then((spoUrl) => {
-            const requestOptions = {
-                url: `${spoUrl}/_api/web/GetStorageEntity('${encodeURIComponent(args.options.key)}')`,
-                headers: {
-                    accept: 'application/json;odata=nometadata'
-                },
-                responseType: 'json'
-            };
-            return request_1.default.get(requestOptions);
-        })
-            .then((property) => {
-            if (property["odata.null"] === true) {
-                if (this.verbose) {
-                    logger.logToStderr(`Property with key ${args.options.key} not found`);
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const spoUrl = yield utils_1.spo.getSpoUrl(logger, this.debug);
+                const requestOptions = {
+                    url: `${spoUrl}/_api/web/GetStorageEntity('${encodeURIComponent(args.options.key)}')`,
+                    headers: {
+                        accept: 'application/json;odata=nometadata'
+                    },
+                    responseType: 'json'
+                };
+                const property = yield request_1.default.get(requestOptions);
+                if (property["odata.null"] === true) {
+                    if (this.verbose) {
+                        logger.logToStderr(`Property with key ${args.options.key} not found`);
+                    }
+                }
+                else {
+                    logger.log({
+                        Key: args.options.key,
+                        Value: property.Value,
+                        Description: property.Description,
+                        Comment: property.Comment
+                    });
                 }
             }
-            else {
-                logger.log({
-                    Key: args.options.key,
-                    Value: property.Value,
-                    Description: property.Description,
-                    Comment: property.Comment
-                });
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
-            cb();
-        }, (err) => this.handleRejectedPromise(err, logger, cb));
+        });
     }
 }
 _SpoStorageEntityGetCommand_instances = new WeakSet(), _SpoStorageEntityGetCommand_initOptions = function _SpoStorageEntityGetCommand_initOptions() {

package/dist/m365/spo/commands/storageentity/storageentity-list.js

@@ -32,53 +32,51 @@ class SpoStorageEntityListCommand extends SpoCommand_1.default {
     get description() {
         return 'Lists tenant properties stored on the specified SharePoint Online app catalog';
     }
-    commandAction(logger, args, cb) {
-        if (this.verbose) {
-            logger.logToStderr(`Retrieving details for all tenant properties in ${args.options.appCatalogUrl}...`);
-        }
-        const requestOptions = {
-            url: `${args.options.appCatalogUrl}/_api/web/AllProperties?$select=storageentitiesindex`,
-            headers: {
-                accept: 'application/json;odata=nometadata'
-            },
-            responseType: 'json'
-        };
-        request_1.default
-            .get(requestOptions)
-            .then((web) => {
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            if (this.verbose) {
+                logger.logToStderr(`Retrieving details for all tenant properties in ${args.options.appCatalogUrl}...`);
+            }
+            const requestOptions = {
+                url: `${args.options.appCatalogUrl}/_api/web/AllProperties?$select=storageentitiesindex`,
+                headers: {
+                    accept: 'application/json;odata=nometadata'
+                },
+                responseType: 'json'
+            };
             try {
+                const web = yield request_1.default.get(requestOptions);
                 if (!web.storageentitiesindex ||
                     web.storageentitiesindex.trim().length === 0) {
                     if (this.verbose) {
                         logger.logToStderr('No tenant properties found');
                     }
-                    cb();
-                    return;
-                }
-                const properties = JSON.parse(web.storageentitiesindex);
-                const keys = Object.keys(properties);
-                if (keys.length === 0) {
-                    if (this.verbose) {
-                        logger.logToStderr('No tenant properties found');
-                    }
                 }
                 else {
-                    logger.log(keys.map((key) => {
-                        const property = properties[key];
-                        return {
-                            Key: key,
-                            Value: property.Value,
-                            Description: property.Description,
-                            Comment: property.Comment
-                        };
-                    }));
+                    const properties = JSON.parse(web.storageentitiesindex);
+                    const keys = Object.keys(properties);
+                    if (keys.length === 0) {
+                        if (this.verbose) {
+                            logger.logToStderr('No tenant properties found');
+                        }
+                    }
+                    else {
+                        logger.log(keys.map((key) => {
+                            const property = properties[key];
+                            return {
+                                Key: key,
+                                Value: property.Value,
+                                Description: property.Description,
+                                Comment: property.Comment
+                            };
+                        }));
+                    }
                 }
-                cb();
             }
-            catch (e) {
-                this.handleError(e, logger, cb);
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
             }
-        }, (err) => this.handleRejectedPromise(err, logger, cb));
+        });
     }
 }
 _SpoStorageEntityListCommand_instances = new WeakSet(), _SpoStorageEntityListCommand_initOptions = function _SpoStorageEntityListCommand_initOptions() {

package/dist/m365/spo/commands/storageentity/storageentity-remove.js

@@ -16,7 +16,6 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 var _SpoStorageEntityRemoveCommand_instances, _SpoStorageEntityRemoveCommand_initTelemetry, _SpoStorageEntityRemoveCommand_initOptions, _SpoStorageEntityRemoveCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
 const cli_1 = require("../../../../cli");
-const Command_1 = require("../../../../Command");
 const config_1 = require("../../../../config");
 const request_1 = require("../../../../request");
 const utils_1 = require("../../../../utils");
@@ -36,57 +35,48 @@ class SpoStorageEntityRemoveCommand extends SpoCommand_1.default {
     get description() {
         return 'Removes tenant property stored on the specified SharePoint Online app catalog';
     }
-    commandAction(logger, args, cb) {
-        const removeTenantProperty = () => {
-            let spoAdminUrl = '';
-            if (this.verbose) {
-                logger.logToStderr(`Removing tenant property ${args.options.key} from ${args.options.appCatalogUrl}...`);
-            }
-            utils_1.spo
-                .getSpoAdminUrl(logger, this.debug)
-                .then((_spoAdminUrl) => {
-                spoAdminUrl = _spoAdminUrl;
-                return utils_1.spo.getRequestDigest(spoAdminUrl);
-            })
-                .then((res) => {
-                const requestOptions = {
-                    url: `${spoAdminUrl}/_vti_bin/client.svc/ProcessQuery`,
-                    headers: {
-                        'X-RequestDigest': res.FormDigestValue
-                    },
-                    data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><ObjectPath Id="31" ObjectPathId="30" /><ObjectPath Id="33" ObjectPathId="32" /><ObjectPath Id="35" ObjectPathId="34" /><Method Name="RemoveStorageEntity" Id="36" ObjectPathId="34"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.key)}</Parameter></Parameters></Method></Actions><ObjectPaths><Constructor Id="30" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" /><Method Id="32" ParentId="30" Name="GetSiteByUrl"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.appCatalogUrl)}</Parameter></Parameters></Method><Property Id="34" ParentId="32" Name="RootWeb" /></ObjectPaths></Request>`
-                };
-                return request_1.default.post(requestOptions);
-            })
-                .then((res) => {
-                const json = JSON.parse(res);
-                const response = json[0];
-                if (response.ErrorInfo) {
-                    cb(new Command_1.CommandError(response.ErrorInfo.ErrorMessage));
-                }
-                else {
-                    cb();
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const removeTenantProperty = () => __awaiter(this, void 0, void 0, function* () {
+                if (this.verbose) {
+                    logger.logToStderr(`Removing tenant property ${args.options.key} from ${args.options.appCatalogUrl}...`);
                 }
-            }, (err) => this.handleRejectedPromise(err, logger, cb));
-        };
-        if (args.options.confirm) {
-            removeTenantProperty();
-        }
-        else {
-            cli_1.Cli.prompt({
-                type: 'confirm',
-                name: 'continue',
-                default: false,
-                message: `Are you sure you want to delete the ${args.options.key} tenant property?`
-            }, (result) => {
-                if (!result.continue) {
-                    cb();
+                try {
+                    const spoAdminUrl = yield utils_1.spo.getSpoAdminUrl(logger, this.debug);
+                    const res = yield utils_1.spo.getRequestDigest(spoAdminUrl);
+                    const requestOptions = {
+                        url: `${spoAdminUrl}/_vti_bin/client.svc/ProcessQuery`,
+                        headers: {
+                            'X-RequestDigest': res.FormDigestValue
+                        },
+                        data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><ObjectPath Id="31" ObjectPathId="30" /><ObjectPath Id="33" ObjectPathId="32" /><ObjectPath Id="35" ObjectPathId="34" /><Method Name="RemoveStorageEntity" Id="36" ObjectPathId="34"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.key)}</Parameter></Parameters></Method></Actions><ObjectPaths><Constructor Id="30" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" /><Method Id="32" ParentId="30" Name="GetSiteByUrl"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.appCatalogUrl)}</Parameter></Parameters></Method><Property Id="34" ParentId="32" Name="RootWeb" /></ObjectPaths></Request>`
+                    };
+                    const processQuery = yield request_1.default.post(requestOptions);
+                    const json = JSON.parse(processQuery);
+                    const response = json[0];
+                    if (response.ErrorInfo) {
+                        throw response.ErrorInfo.ErrorMessage;
+                    }
                 }
-                else {
-                    removeTenantProperty();
+                catch (err) {
+                    this.handleRejectedODataJsonPromise(err);
                 }
             });
-        }
+            if (args.options.confirm) {
+                yield removeTenantProperty();
+            }
+            else {
+                const result = yield cli_1.Cli.prompt({
+                    type: 'confirm',
+                    name: 'continue',
+                    default: false,
+                    message: `Are you sure you want to delete the ${args.options.key} tenant property?`
+                });
+                if (result.continue) {
+                    yield removeTenantProperty();
+                }
+            }
+        });
     }
 }
 _SpoStorageEntityRemoveCommand_instances = new WeakSet(), _SpoStorageEntityRemoveCommand_initTelemetry = function _SpoStorageEntityRemoveCommand_initTelemetry() {

package/dist/m365/spo/commands/storageentity/storageentity-set.js

@@ -15,7 +15,6 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 };
 var _SpoStorageEntitySetCommand_instances, _SpoStorageEntitySetCommand_initTelemetry, _SpoStorageEntitySetCommand_initOptions, _SpoStorageEntitySetCommand_initValidators;
 Object.defineProperty(exports, "__esModule", { value: true });
-const Command_1 = require("../../../../Command");
 const config_1 = require("../../../../config");
 const request_1 = require("../../../../request");
 const utils_1 = require("../../../../utils");
@@ -35,41 +34,37 @@ class SpoStorageEntitySetCommand extends SpoCommand_1.default {
     get description() {
         return 'Sets tenant property on the specified SharePoint Online app catalog';
     }
-    commandAction(logger, args, cb) {
-        let spoAdminUrl = '';
-        utils_1.spo
-            .getSpoAdminUrl(logger, this.debug)
-            .then((_spoAdminUrl) => {
-            spoAdminUrl = _spoAdminUrl;
-            return utils_1.spo.getRequestDigest(spoAdminUrl);
-        })
-            .then((res) => {
-            if (this.verbose) {
-                logger.logToStderr(`Setting tenant property ${args.options.key} in ${args.options.appCatalogUrl}...`);
-            }
-            const requestOptions = {
-                url: `${spoAdminUrl}/_vti_bin/client.svc/ProcessQuery`,
-                headers: {
-                    'X-RequestDigest': res.FormDigestValue
-                },
-                data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><ObjectPath Id="24" ObjectPathId="23" /><ObjectPath Id="26" ObjectPathId="25" /><ObjectPath Id="28" ObjectPathId="27" /><Method Name="SetStorageEntity" Id="29" ObjectPathId="27"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.key)}</Parameter><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.value)}</Parameter><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.description || '')}</Parameter><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.comment || '')}</Parameter></Parameters></Method></Actions><ObjectPaths><Constructor Id="23" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" /><Method Id="25" ParentId="23" Name="GetSiteByUrl"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.appCatalogUrl)}</Parameter></Parameters></Method><Property Id="27" ParentId="25" Name="RootWeb" /></ObjectPaths></Request>`
-            };
-            return request_1.default.post(requestOptions);
-        })
-            .then((res) => {
-            const json = JSON.parse(res);
-            const response = json[0];
-            if (response.ErrorInfo) {
-                if (this.verbose && response.ErrorInfo.ErrorMessage.indexOf('Access denied.') > -1) {
-                    logger.logToStderr('');
-                    logger.logToStderr(`This error is often caused by invalid URL of the app catalog site. Verify, that the URL you specified as an argument of the ${commands_1.default.STORAGEENTITY_SET} command is a valid app catalog URL and try again.`);
-                    logger.logToStderr('');
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const spoAdminUrl = yield utils_1.spo.getSpoAdminUrl(logger, this.debug);
+                const res = yield utils_1.spo.getRequestDigest(spoAdminUrl);
+                if (this.verbose) {
+                    logger.logToStderr(`Setting tenant property ${args.options.key} in ${args.options.appCatalogUrl}...`);
+                }
+                const requestOptions = {
+                    url: `${spoAdminUrl}/_vti_bin/client.svc/ProcessQuery`,
+                    headers: {
+                        'X-RequestDigest': res.FormDigestValue
+                    },
+                    data: `<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="${config_1.default.applicationName}" xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009"><Actions><ObjectPath Id="24" ObjectPathId="23" /><ObjectPath Id="26" ObjectPathId="25" /><ObjectPath Id="28" ObjectPathId="27" /><Method Name="SetStorageEntity" Id="29" ObjectPathId="27"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.key)}</Parameter><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.value)}</Parameter><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.description || '')}</Parameter><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.comment || '')}</Parameter></Parameters></Method></Actions><ObjectPaths><Constructor Id="23" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" /><Method Id="25" ParentId="23" Name="GetSiteByUrl"><Parameters><Parameter Type="String">${utils_1.formatting.escapeXml(args.options.appCatalogUrl)}</Parameter></Parameters></Method><Property Id="27" ParentId="25" Name="RootWeb" /></ObjectPaths></Request>`
+                };
+                const processQuery = yield request_1.default.post(requestOptions);
+                const json = JSON.parse(processQuery);
+                const response = json[0];
+                if (response.ErrorInfo) {
+                    if (this.verbose && response.ErrorInfo.ErrorMessage.indexOf('Access denied.') > -1) {
+                        logger.logToStderr('');
+                        logger.logToStderr(`This error is often caused by invalid URL of the app catalog site. Verify, that the URL you specified as an argument of the ${commands_1.default.STORAGEENTITY_SET} command is a valid app catalog URL and try again.`);
+                        logger.logToStderr('');
+                    }
+                    throw response.ErrorInfo.ErrorMessage;
                 }
-                cb(new Command_1.CommandError(response.ErrorInfo.ErrorMessage));
-                return;
             }
-            cb();
-        }, (err) => this.handleRejectedPromise(err, logger, cb));
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
+            }
+        });
     }
 }
 _SpoStorageEntitySetCommand_instances = new WeakSet(), _SpoStorageEntitySetCommand_initTelemetry = function _SpoStorageEntitySetCommand_initTelemetry() {

package/dist/m365/spo/commands/tenant/tenant-appcatalog-add.js

@@ -38,29 +38,28 @@ class SpoTenantAppCatalogAddCommand extends SpoCommand_1.default {
     get description() {
         return 'Creates new tenant app catalog site';
     }
-    commandAction(logger, args, cb) {
-        if (this.verbose) {
-            logger.logToStderr('Checking for existing app catalog URL...');
-        }
-        cli_1.Cli
-            .executeCommandWithOutput(spoTenantAppCatalogUrlGetCommand, { options: { output: 'text', _: [] } })
-            .then((spoTenantAppCatalogUrlGetCommandOutput) => {
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            if (this.verbose) {
+                logger.logToStderr('Checking for existing app catalog URL...');
+            }
+            const spoTenantAppCatalogUrlGetCommandOutput = yield cli_1.Cli.executeCommandWithOutput(spoTenantAppCatalogUrlGetCommand, { options: { output: 'text', _: [] } });
             const appCatalogUrl = spoTenantAppCatalogUrlGetCommandOutput.stdout;
             if (!appCatalogUrl) {
                 if (this.verbose) {
                     logger.logToStderr('No app catalog URL found');
                 }
-                return Promise.resolve();
             }
-            if (this.verbose) {
-                logger.logToStderr(`Found app catalog URL ${appCatalogUrl}`);
+            else {
+                if (this.verbose) {
+                    logger.logToStderr(`Found app catalog URL ${appCatalogUrl}`);
+                }
+                //Using JSON.parse
+                yield this.ensureNoExistingSite(appCatalogUrl, args.options.force, logger);
             }
-            //Using JSON.parse
-            return this.ensureNoExistingSite(appCatalogUrl, args.options.force, logger);
-        })
-            .then(() => this.ensureNoExistingSite(args.options.url, args.options.force, logger))
-            .then(() => this.createAppCatalog(args.options, logger))
-            .then(_ => cb(), (err) => cb(err));
+            yield this.ensureNoExistingSite(args.options.url, args.options.force, logger);
+            yield this.createAppCatalog(args.options, logger);
+        });
     }
     ensureNoExistingSite(url, force, logger) {
         return new Promise((resolve, reject) => {