@pnp/cli-microsoft365 5.3.0-beta.f028e14 → 5.3.0

package/.eslintrc.js

@@ -29,6 +29,7 @@ const dictionary = [
   'home',
   'hub',
   'in',
+  'inheritance',
   'init',
   'install',
   'installed',

package/dist/Auth.js

@@ -540,6 +540,11 @@ class Auth {
             // we need to use https://management.azure.com/ instead
             resource = 'https://management.azure.com/';
         }
+        if (resource === 'https://api.powerbi.com') {
+            // api.powerbi.com is not a valid resource
+            // we need to use https://analysis.windows.net/powerbi/api instead
+            resource = 'https://analysis.windows.net/powerbi/api';
+        }
         return resource;
     }
     getServiceConnectionInfo() {

package/dist/m365/aad/commands/app/app-add.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 fs = require("fs");
 const uuid_1 = require("uuid");
@@ -31,6 +40,9 @@ class AadAppAddCommand extends GraphCommand_1.default {
         telemetryProps.scopeName = typeof args.options.scopeName !== 'undefined';
         telemetryProps.uri = typeof args.options.uri !== 'undefined';
         telemetryProps.withSecret = args.options.withSecret;
+        telemetryProps.certificateFile = typeof args.options.certificateFile !== 'undefined';
+        telemetryProps.certificateBase64Encoded = typeof args.options.certificateBase64Encoded !== 'undefined';
+        telemetryProps.certificateDisplayName = typeof args.options.certificateDisplayName !== 'undefined';
         return telemetryProps;
     }
     commandAction(logger, args, cb) {
@@ -66,43 +78,55 @@ class AadAppAddCommand extends GraphCommand_1.default {
         }, (rawRes) => this.handleRejectedODataJsonPromise(rawRes, logger, cb));
     }
     createAppRegistration(args, apis, logger) {
-        const applicationInfo = {
-            displayName: args.options.name,
-            signInAudience: args.options.multitenant ? 'AzureADMultipleOrgs' : 'AzureADMyOrg'
-        };
-        if (!applicationInfo.displayName && this.manifest) {
-            applicationInfo.displayName = this.manifest.name;
-        }
-        this.appName = applicationInfo.displayName;
-        if (apis.length > 0) {
-            applicationInfo.requiredResourceAccess = apis;
-        }
-        if (args.options.redirectUris) {
-            applicationInfo[args.options.platform] = {
-                redirectUris: args.options.redirectUris.split(',').map(u => u.trim())
+        return __awaiter(this, void 0, void 0, function* () {
+            const applicationInfo = {
+                displayName: args.options.name,
+                signInAudience: args.options.multitenant ? 'AzureADMultipleOrgs' : 'AzureADMyOrg'
             };
-        }
-        if (args.options.implicitFlow) {
-            if (!applicationInfo.web) {
-                applicationInfo.web = {};
+            if (!applicationInfo.displayName && this.manifest) {
+                applicationInfo.displayName = this.manifest.name;
+            }
+            this.appName = applicationInfo.displayName;
+            if (apis.length > 0) {
+                applicationInfo.requiredResourceAccess = apis;
+            }
+            if (args.options.redirectUris) {
+                applicationInfo[args.options.platform] = {
+                    redirectUris: args.options.redirectUris.split(',').map(u => u.trim())
+                };
+            }
+            if (args.options.implicitFlow) {
+                if (!applicationInfo.web) {
+                    applicationInfo.web = {};
+                }
+                applicationInfo.web.implicitGrantSettings = {
+                    enableAccessTokenIssuance: true,
+                    enableIdTokenIssuance: true
+                };
             }
-            applicationInfo.web.implicitGrantSettings = {
-                enableAccessTokenIssuance: true,
-                enableIdTokenIssuance: true
+            if (args.options.certificateFile || args.options.certificateBase64Encoded) {
+                const certificateBase64Encoded = this.getCertificateBase64Encoded(args, logger);
+                const newKeyCredential = {
+                    type: "AsymmetricX509Cert",
+                    usage: "Verify",
+                    displayName: args.options.certificateDisplayName,
+                    key: certificateBase64Encoded
+                };
+                applicationInfo.keyCredentials = [newKeyCredential];
+            }
+            if (this.verbose) {
+                logger.logToStderr(`Creating Azure AD app registration...`);
+            }
+            const createApplicationRequestOptions = {
+                url: `${this.resource}/v1.0/myorganization/applications`,
+                headers: {
+                    accept: 'application/json;odata.metadata=none'
+                },
+                responseType: 'json',
+                data: applicationInfo
             };
-        }
-        if (this.verbose) {
-            logger.logToStderr(`Creating Azure AD app registration...`);
-        }
-        const createApplicationRequestOptions = {
-            url: `${this.resource}/v1.0/myorganization/applications`,
-            headers: {
-                accept: 'application/json;odata.metadata=none'
-            },
-            responseType: 'json',
-            data: applicationInfo
-        };
-        return request_1.default.post(createApplicationRequestOptions);
+            return request_1.default.post(createApplicationRequestOptions);
+        });
     }
     updateAppFromManifest(args, appInfo) {
         if (!args.options.manifest) {
@@ -325,7 +349,7 @@ class AadAppAddCommand extends GraphCommand_1.default {
             logger.logToStderr('Resolving requested APIs...');
         }
         return utils_1.odata
-            .getAllItems(`${this.resource}/v1.0/myorganization/servicePrincipals?$select=servicePrincipalNames,appId,oauth2PermissionScopes,appRoles`, logger)
+            .getAllItems(`${this.resource}/v1.0/myorganization/servicePrincipals?$select=servicePrincipalNames,appId,oauth2PermissionScopes,appRoles`)
             .then(servicePrincipals => {
             try {
                 const resolvedApis = this.getRequiredResourceAccessForApis(servicePrincipals, args.options.apisDelegated, 'Scope', logger);
@@ -437,6 +461,20 @@ class AadAppAddCommand extends GraphCommand_1.default {
             value: password.secretText
         }));
     }
+    getCertificateBase64Encoded(args, logger) {
+        if (args.options.certificateBase64Encoded) {
+            return args.options.certificateBase64Encoded;
+        }
+        if (this.debug) {
+            logger.logToStderr(`Reading existing ${args.options.certificateFile}...`);
+        }
+        try {
+            return fs.readFileSync(args.options.certificateFile, { encoding: 'base64' });
+        }
+        catch (e) {
+            throw new Error(`Error reading certificate file: ${e}. Please add the certificate using base64 option '--certificateBase64Encoded'.`);
+        }
+    }
     saveAppInfo(args, appInfo, logger) {
         if (!args.options.save) {
             return Promise.resolve(appInfo);
@@ -519,6 +557,15 @@ class AadAppAddCommand extends GraphCommand_1.default {
             {
                 option: '--scopeAdminConsentDescription [scopeAdminConsentDescription]'
             },
+            {
+                option: '--certificateFile [certificateFile]'
+            },
+            {
+                option: '--certificateBase64Encoded [certificateBase64Encoded]'
+            },
+            {
+                option: '--certificateDisplayName [certificateDisplayName]'
+            },
             {
                 option: '--manifest [manifest]'
             },
@@ -540,6 +587,15 @@ class AadAppAddCommand extends GraphCommand_1.default {
         if (args.options.redirectUris && !args.options.platform) {
             return `When you specify redirectUris you also need to specify platform`;
         }
+        if (args.options.certificateFile && args.options.certificateBase64Encoded) {
+            return 'Specify either certificateFile or certificateBase64Encoded but not both';
+        }
+        if (args.options.certificateDisplayName && !args.options.certificateFile && !args.options.certificateBase64Encoded) {
+            return 'When you specify certificateDisplayName you also need to specify certificateFile or certificateBase64Encoded';
+        }
+        if (args.options.certificateFile && !fs.existsSync(args.options.certificateFile)) {
+            return 'Certificate file not found';
+        }
         if (args.options.scopeName) {
             if (!args.options.uri) {
                 return `When you specify scopeName you also need to specify uri`;

package/dist/m365/aad/commands/app/app-role-list.js

@@ -24,7 +24,7 @@ class AadAppRoleListCommand extends GraphCommand_1.default {
     commandAction(logger, args, cb) {
         this
             .getAppObjectId(args, logger)
-            .then(objectId => utils_1.odata.getAllItems(`${this.resource}/v1.0/myorganization/applications/${objectId}/appRoles`, logger))
+            .then(objectId => utils_1.odata.getAllItems(`${this.resource}/v1.0/myorganization/applications/${objectId}/appRoles`))
             .then(appRoles => {
             logger.log(appRoles);
             cb();

package/dist/m365/aad/commands/app/app-set.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+const fs = require("fs");
 const request_1 = require("../../../../request");
 const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
@@ -19,6 +20,9 @@ class AadAppSetCommand extends GraphCommand_1.default {
         telemetryProps.redirectUris = typeof args.options.redirectUris !== 'undefined';
         telemetryProps.redirectUrisToRemove = typeof args.options.redirectUrisToRemove !== 'undefined';
         telemetryProps.uri = typeof args.options.uri !== 'undefined';
+        telemetryProps.certificateFile = typeof args.options.certificateFile !== 'undefined';
+        telemetryProps.certificateBase64Encoded = typeof args.options.certificateBase64Encoded !== 'undefined';
+        telemetryProps.certificateDisplayName = typeof args.options.certificateDisplayName !== 'undefined';
         return telemetryProps;
     }
     commandAction(logger, args, cb) {
@@ -26,6 +30,7 @@ class AadAppSetCommand extends GraphCommand_1.default {
             .getAppObjectId(args, logger)
             .then(objectId => this.configureUri(args, objectId, logger))
             .then(objectId => this.configureRedirectUris(args, objectId, logger))
+            .then(objectId => this.configureCertificate(args, objectId, logger))
             .then(_ => cb(), (rawRes) => this.handleRejectedODataJsonPromise(rawRes, logger, cb));
     }
     getAppObjectId(args, logger) {
@@ -156,6 +161,80 @@ class AadAppSetCommand extends GraphCommand_1.default {
         })
             .then(_ => Promise.resolve(objectId));
     }
+    configureCertificate(args, objectId, logger) {
+        if (!args.options.certificateFile && !args.options.certificateBase64Encoded) {
+            return Promise.resolve();
+        }
+        if (this.verbose) {
+            logger.logToStderr(`Setting certificate for Azure AD app...`);
+        }
+        const certificateBase64Encoded = this.getCertificateBase64Encoded(args, logger);
+        return this
+            .getCurrentKeyCredentialsList(args, objectId, certificateBase64Encoded, logger)
+            .then(currentKeyCredentials => {
+            if (this.verbose) {
+                logger.logToStderr(`Adding new keyCredential to list`);
+            }
+            // The KeyCredential graph type defines the 'key' property as 'NullableOption<number>'
+            // while it is a base64 encoded string. This is why a cast to any is used here.
+            const keyCredentials = currentKeyCredentials.filter(existingCredential => existingCredential.key !== certificateBase64Encoded);
+            const newKeyCredential = {
+                type: "AsymmetricX509Cert",
+                usage: "Verify",
+                displayName: args.options.certificateDisplayName,
+                key: certificateBase64Encoded
+            };
+            keyCredentials.push(newKeyCredential);
+            return Promise.resolve(keyCredentials);
+        })
+            .then(keyCredentials => this.updateKeyCredentials(objectId, keyCredentials, logger));
+    }
+    getCertificateBase64Encoded(args, logger) {
+        if (args.options.certificateBase64Encoded) {
+            return args.options.certificateBase64Encoded;
+        }
+        if (this.debug) {
+            logger.logToStderr(`Reading existing ${args.options.certificateFile}...`);
+        }
+        try {
+            return fs.readFileSync(args.options.certificateFile, { encoding: 'base64' });
+        }
+        catch (e) {
+            throw new Error(`Error reading certificate file: ${e}. Please add the certificate using base64 option '--certificateBase64Encoded'.`);
+        }
+    }
+    // We first retrieve existing certificates because we need to specify the full list of certificates when updating the app.
+    getCurrentKeyCredentialsList(args, objectId, certificateBase64Encoded, logger) {
+        if (this.verbose) {
+            logger.logToStderr(`Retrieving current keyCredentials list for app`);
+        }
+        const getAppRequestOptions = {
+            url: `${this.resource}/v1.0/myorganization/applications/${objectId}?$select=keyCredentials`,
+            headers: {
+                'content-type': 'application/json;odata.metadata=none'
+            },
+            responseType: 'json'
+        };
+        return request_1.default.get(getAppRequestOptions).then((application) => {
+            return Promise.resolve(application.keyCredentials || []);
+        });
+    }
+    updateKeyCredentials(objectId, keyCredentials, logger) {
+        if (this.verbose) {
+            logger.logToStderr(`Updating keyCredentials in AAD app`);
+        }
+        const requestOptions = {
+            url: `${this.resource}/v1.0/myorganization/applications/${objectId}`,
+            headers: {
+                'content-type': 'application/json;odata.metadata=none'
+            },
+            responseType: 'json',
+            data: {
+                keyCredentials: keyCredentials
+            }
+        };
+        return request_1.default.patch(requestOptions);
+    }
     options() {
         const options = [
             { option: '--appId [appId]' },
@@ -163,6 +242,9 @@ class AadAppSetCommand extends GraphCommand_1.default {
             { option: '-n, --name [name]' },
             { option: '-u, --uri [uri]' },
             { option: '-r, --redirectUris [redirectUris]' },
+            { option: '--certificateFile [certificateFile]' },
+            { option: '--certificateBase64Encoded [certificateBase64Encoded]' },
+            { option: '--certificateDisplayName [certificateDisplayName]' },
             {
                 option: '--platform [platform]',
                 autocomplete: AadAppSetCommand.aadApplicationPlatform
@@ -183,6 +265,15 @@ class AadAppSetCommand extends GraphCommand_1.default {
             (args.options.objectId && args.options.name)) {
             return 'Specify either appId, objectId or name but not both';
         }
+        if (args.options.certificateFile && args.options.certificateBase64Encoded) {
+            return 'Specify either certificateFile or certificateBase64Encoded but not both';
+        }
+        if (args.options.certificateDisplayName && !args.options.certificateFile && !args.options.certificateBase64Encoded) {
+            return 'When you specify certificateDisplayName you also need to specify certificateFile or certificateBase64Encoded';
+        }
+        if (args.options.certificateFile && !fs.existsSync(args.options.certificateFile)) {
+            return 'Certificate file not found';
+        }
         if (args.options.redirectUris && !args.options.platform) {
             return `When you specify redirectUris you also need to specify platform`;
         }

package/dist/m365/aad/commands/approleassignment/approleassignment-list.js

@@ -22,29 +22,14 @@ class AadAppRoleAssignmentListCommand extends GraphCommand_1.default {
         return ['resourceDisplayName', 'roleName'];
     }
     commandAction(logger, args, cb) {
-        let sp;
-        // get the service principal associated with the appId
-        let spMatchQuery = '';
-        if (args.options.appId) {
-            spMatchQuery = `appId eq '${encodeURIComponent(args.options.appId)}'`;
-        }
-        else if (args.options.objectId) {
-            spMatchQuery = `id eq '${encodeURIComponent(args.options.objectId)}'`;
-        }
-        else {
-            spMatchQuery = `displayName eq '${encodeURIComponent(args.options.displayName)}'`;
-        }
-        this
-            .getServicePrincipalForApp(spMatchQuery)
-            .then((resp) => {
-            if (!resp.value.length) {
-                return Promise.reject('app registration not found');
-            }
-            sp = resp.value[0];
+        let spAppRoleAssignments;
+        this.getAppRoleAssignments(args.options)
+            .then((appRoleAssignments) => {
+            spAppRoleAssignments = appRoleAssignments;
             // the role assignment has an appRoleId but no name. To get the name,
             // we need to get all the roles from the resource. the resource is
             // a service principal. Multiple roles may have same resource id.
-            const resourceIds = sp.appRoleAssignments.map((item) => item.resourceId);
+            const resourceIds = appRoleAssignments.map((item) => item.resourceId);
             const tasks = [];
             for (let i = 0; i < resourceIds.length; i++) {
                 tasks.push(this.getServicePrincipal(resourceIds[i]));
@@ -55,7 +40,7 @@ class AadAppRoleAssignmentListCommand extends GraphCommand_1.default {
             // loop through all appRoleAssignments for the servicePrincipal
             // and lookup the appRole.Id in the resources[resourceId].appRoles array...
             const results = [];
-            sp.appRoleAssignments.map((appRoleAssignment) => {
+            spAppRoleAssignments.map((appRoleAssignment) => {
                 const resource = resources.find((r) => r.id === appRoleAssignment.resourceId);
                 if (resource) {
                     const appRole = resource.appRoles.find((r) => r.id === appRoleAssignment.appRoleId);
@@ -65,7 +50,9 @@ class AadAppRoleAssignmentListCommand extends GraphCommand_1.default {
                             resourceDisplayName: appRoleAssignment.resourceDisplayName,
                             resourceId: appRoleAssignment.resourceId,
                             roleId: appRole.id,
-                            roleName: appRole.value
+                            roleName: appRole.value,
+                            created: appRoleAssignment.createdDateTime,
+                            deleted: appRoleAssignment.deletedDateTime
                         });
                     }
                 }
@@ -74,6 +61,52 @@ class AadAppRoleAssignmentListCommand extends GraphCommand_1.default {
             cb();
         }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
     }
+    getAppRoleAssignments(argOptions) {
+        return new Promise((resolve, reject) => {
+            if (argOptions.objectId) {
+                this.getSPAppRoleAssignments(argOptions.objectId)
+                    .then((spAppRoleAssignments) => {
+                    if (!spAppRoleAssignments.value.length) {
+                        reject('no app role assignments found');
+                    }
+                    resolve(spAppRoleAssignments.value);
+                })
+                    .catch((err) => {
+                    reject(err);
+                });
+            }
+            else {
+                // Use existing way to get service principal object
+                let spMatchQuery = '';
+                if (argOptions.appId) {
+                    spMatchQuery = `appId eq '${encodeURIComponent(argOptions.appId)}'`;
+                }
+                else {
+                    spMatchQuery = `displayName eq '${encodeURIComponent(argOptions.displayName)}'`;
+                }
+                this.getServicePrincipalForApp(spMatchQuery)
+                    .then((resp) => {
+                    if (!resp.value.length) {
+                        reject('app registration not found');
+                    }
+                    resolve(resp.value[0].appRoleAssignments);
+                })
+                    .catch((err) => {
+                    reject(err);
+                });
+            }
+        });
+    }
+    getSPAppRoleAssignments(spId) {
+        const spRequestOptions = {
+            url: `${this.resource}/v1.0/servicePrincipals/${spId}/appRoleAssignments`,
+            headers: {
+                accept: 'application/json'
+            },
+            responseType: 'json'
+        };
+        return request_1.default.get(spRequestOptions);
+    }
     getServicePrincipalForApp(filterParam) {
         const spRequestOptions = {
             url: `${this.resource}/v1.0/servicePrincipals?$expand=appRoleAssignments&$filter=${filterParam}`,

package/dist/m365/aad/commands/group/group-list.js

@@ -21,7 +21,7 @@ class AadGroupListCommand extends GraphCommand_1.default {
     commandAction(logger, args, cb) {
         const endpoint = args.options.deleted ? 'directory/deletedItems/microsoft.graph.group' : 'groups';
         utils_1.odata
-            .getAllItems(`${this.resource}/v1.0/${endpoint}`, logger)
+            .getAllItems(`${this.resource}/v1.0/${endpoint}`)
             .then((groups) => {
             if (args.options.output === 'text') {
                 groups.forEach((group) => {

package/dist/m365/aad/commands/groupsetting/groupsetting-list.js

@@ -15,7 +15,7 @@ class AadGroupSettingListCommand extends GraphCommand_1.default {
     }
     commandAction(logger, args, cb) {
         utils_1.odata
-            .getAllItems(`${this.resource}/v1.0/groupSettings`, logger)
+            .getAllItems(`${this.resource}/v1.0/groupSettings`)
             .then((groupSettings) => {
             logger.log(groupSettings);
             cb();

package/dist/m365/aad/commands/groupsettingtemplate/groupsettingtemplate-get.js

@@ -19,7 +19,7 @@ class AadGroupSettingTemplateGetCommand extends GraphCommand_1.default {
     }
     commandAction(logger, args, cb) {
         utils_1.odata
-            .getAllItems(`${this.resource}/v1.0/groupSettingTemplates`, logger)
+            .getAllItems(`${this.resource}/v1.0/groupSettingTemplates`)
             .then((templates) => {
             const groupSettingTemplate = templates.filter(t => args.options.id ? t.id === args.options.id : t.displayName === args.options.displayName);
             if (groupSettingTemplate && groupSettingTemplate.length > 0) {

package/dist/m365/aad/commands/groupsettingtemplate/groupsettingtemplate-list.js

@@ -15,7 +15,7 @@ class AadGroupSettingTemplateListCommand extends GraphCommand_1.default {
     }
     commandAction(logger, args, cb) {
         utils_1.odata
-            .getAllItems(`${this.resource}/v1.0/groupSettingTemplates`, logger)
+            .getAllItems(`${this.resource}/v1.0/groupSettingTemplates`)
             .then((templates) => {
             logger.log(templates);
             cb();

package/dist/m365/aad/commands/o365group/o365group-conversation-list.js

@@ -15,7 +15,7 @@ class AadO365GroupConversationListCommand extends GraphCommand_1.default {
     }
     commandAction(logger, args, cb) {
         utils_1.odata
-            .getAllItems(`${this.resource}/v1.0/groups/${args.options.groupId}/conversations`, logger)
+            .getAllItems(`${this.resource}/v1.0/groups/${args.options.groupId}/conversations`)
             .then((conversations) => {
             logger.log(conversations);
             cb();

package/dist/m365/aad/commands/o365group/o365group-conversation-post-list.js

@@ -24,7 +24,7 @@ class AadO365GroupConversationPostListCommand extends GraphCommand_1.default {
         this
             .getGroupId(args)
             .then((retrievedgroupId) => {
-            return utils_1.odata.getAllItems(`${this.resource}/v1.0/groups/${retrievedgroupId}/threads/${args.options.threadId}/posts`, logger);
+            return utils_1.odata.getAllItems(`${this.resource}/v1.0/groups/${retrievedgroupId}/threads/${args.options.threadId}/posts`);
         })
             .then((posts) => {
             logger.log(posts);

package/dist/m365/aad/commands/o365group/o365group-list.js

@@ -35,7 +35,7 @@ class AadO365GroupListCommand extends GraphCommand_1.default {
         }
         let groups = [];
         utils_1.odata
-            .getAllItems(endpoint, logger)
+            .getAllItems(endpoint)
             .then((_groups) => {
             groups = _groups;
             if (args.options.orphaned) {

package/dist/m365/aad/commands/o365group/o365group-recyclebinitem-clear.js

@@ -19,7 +19,7 @@ class AadO365GroupRecycleBinItemClearCommand extends GraphCommand_1.default {
     }
     commandAction(logger, args, cb) {
         const clearO365GroupRecycleBinItems = () => {
-            this.processRecycleBinItemsClear(logger).then(_ => cb(), (rawRes) => this.handleRejectedODataJsonPromise(rawRes, logger, cb));
+            this.processRecycleBinItemsClear().then(_ => cb(), (rawRes) => this.handleRejectedODataJsonPromise(rawRes, logger, cb));
         };
         if (args.options.confirm) {
             clearO365GroupRecycleBinItems();
@@ -40,12 +40,12 @@ class AadO365GroupRecycleBinItemClearCommand extends GraphCommand_1.default {
             });
         }
     }
-    processRecycleBinItemsClear(logger) {
+    processRecycleBinItemsClear() {
         const filter = `?$filter=groupTypes/any(c:c+eq+'Unified')`;
         const topCount = '&$top=100';
         const endpoint = `${this.resource}/v1.0/directory/deletedItems/Microsoft.Graph.Group${filter}${topCount}`;
         return utils_1.odata
-            .getAllItems(endpoint, logger)
+            .getAllItems(endpoint)
             .then((recycleBinItems) => {
             if (recycleBinItems.length === 0) {
                 return Promise.resolve();

package/dist/m365/aad/commands/o365group/o365group-recyclebinitem-list.js

@@ -26,7 +26,7 @@ class AadO365GroupRecycleBinItemListCommand extends GraphCommand_1.default {
         const topCount = '&$top=100';
         const endpoint = `${this.resource}/v1.0/directory/deletedItems/Microsoft.Graph.Group${filter}${displayNameFilter}${mailNicknameFilter}${topCount}`;
         utils_1.odata
-            .getAllItems(endpoint, logger)
+            .getAllItems(endpoint)
             .then((recycleBinItems) => {
             logger.log(recycleBinItems);
             cb();

package/dist/m365/aad/commands/o365group/o365group-user-list.js

@@ -38,7 +38,7 @@ class AadO365GroupUserListCommand extends GraphCommand_1.default {
     getOwners(logger, groupId) {
         const endpoint = `${this.resource}/v1.0/groups/${groupId}/owners?$select=id,displayName,userPrincipalName,userType`;
         return utils_1.odata
-            .getAllItems(endpoint, logger)
+            .getAllItems(endpoint)
             .then(users => {
             // Currently there is a bug in the Microsoft Graph that returns Owners as
             // userType 'member'. We therefore update all returned user as owner
@@ -50,7 +50,7 @@ class AadO365GroupUserListCommand extends GraphCommand_1.default {
     }
     getMembersAndGuests(logger, groupId) {
         const endpoint = `${this.resource}/v1.0/groups/${groupId}/members?$select=id,displayName,userPrincipalName,userType`;
-        return utils_1.odata.getAllItems(endpoint, logger);
+        return utils_1.odata.getAllItems(endpoint);
     }
     options() {
         const options = [

package/dist/m365/aad/commands/o365group/o365group-user-set.js

@@ -92,7 +92,7 @@ class AadO365GroupUserSetCommand extends GraphCommand_1.default {
     getOwners(logger, groupId) {
         const endpoint = `${this.resource}/v1.0/groups/${groupId}/owners?$select=id,displayName,userPrincipalName,userType`;
         return utils_1.odata
-            .getAllItems(endpoint, logger)
+            .getAllItems(endpoint)
             .then(users => {
             // Currently there is a bug in the Microsoft Graph that returns Owners as
             // userType 'member'. We therefore update all returned user as owner
@@ -104,7 +104,7 @@ class AadO365GroupUserSetCommand extends GraphCommand_1.default {
     }
     getMembersAndGuests(logger, groupId) {
         const endpoint = `${this.resource}/v1.0/groups/${groupId}/members?$select=id,displayName,userPrincipalName,userType`;
-        return utils_1.odata.getAllItems(endpoint, logger);
+        return utils_1.odata.getAllItems(endpoint);
     }
     options() {
         const options = [

package/dist/m365/aad/commands/user/user-list.js

@@ -27,7 +27,7 @@ class AadUserListCommand extends GraphCommand_1.default {
         const endpoint = args.options.deleted ? 'directory/deletedItems/microsoft.graph.user' : 'users';
         const url = `${this.resource}/v1.0/${endpoint}?$select=${properties.join(',')}${(filter.length > 0 ? '&' + filter : '')}&$top=100`;
         utils_1.odata
-            .getAllItems(url, logger)
+            .getAllItems(url)
             .then((users) => {
             logger.log(users);
             cb();

package/dist/m365/aad/commands/user/user-signin-list.js

@@ -37,7 +37,7 @@ class AadUserSigninListCommand extends GraphCommand_1.default {
         }
         endpoint += filter;
         utils_1.odata
-            .getAllItems(endpoint, logger)
+            .getAllItems(endpoint)
             .then((signins) => {
             logger.log(signins);
             cb();

package/dist/m365/aad/commands.js

@@ -6,11 +6,11 @@ exports.default = {
     APP_DELETE: `${prefix} app delete`,
     APP_GET: `${prefix} app get`,
     APP_REMOVE: `${prefix} app remove`,
-    APP_SET: `${prefix} app set`,
     APP_ROLE_ADD: `${prefix} app role add`,
     APP_ROLE_DELETE: `${prefix} app role delete`,
     APP_ROLE_LIST: `${prefix} app role list`,
     APP_ROLE_REMOVE: `${prefix} app role remove`,
+    APP_SET: `${prefix} app set`,
     APPROLEASSIGNMENT_ADD: `${prefix} approleassignment add`,
     APPROLEASSIGNMENT_LIST: `${prefix} approleassignment list`,
     APPROLEASSIGNMENT_REMOVE: `${prefix} approleassignment remove`,