Skip to content

Commit

Permalink
Stop offering the PnP Management shell as option for authenticating t…
Browse files Browse the repository at this point in the history
…he Assessment tool + rename to Entra where needed
  • Loading branch information
bjansen committed Sep 10, 2024
1 parent bff345f commit 84bea51
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1058,7 +1058,7 @@ private async Task<List<SyntexField>> PrepareSyntexFieldsAsync(IList list, List<
ScanId = ScanId,
TermSetId = syntexField.TermSetId,
Title = "N/A",
Description = "Please grant the used Azure AD application TermStore.Read.All permissions to populate taxonomy data",
Description = "Please grant the used Entra application TermStore.Read.All permissions to populate taxonomy data",
});
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,8 +131,9 @@ public StartCommandHandler(ScannerManager processManagerInstance, IDataProtectio
applicationIdOption = new(
name: $"--{Constants.StartApplicationId}",
// Default application to use is the PnP Management shell application
getDefaultValue: () => Guid.Parse("31359c7f-bd7e-475c-86db-fdb8c937548e"),
description: "Azure AD application id to use for authenticating the Microsoft 365 Assessment")
// We stopped using the PnP Management shell application as default as it's not available anymore
//getDefaultValue: () => Guid.Parse("31359c7f-bd7e-475c-86db-fdb8c937548e"),
description: "Entra application id to use for authenticating the Microsoft 365 Assessment")
{
IsRequired = true
};
Expand Down

0 comments on commit 84bea51

Please sign in to comment.