Add warning to Get-PnPTenantInfo about required permissions (#4473)

* Add warning to Get-PnPTenantInfo about required permissions * Updated Get-PnPTenantInfo documentation with Required Permissions * Corrected the pull request build error * Changed the required permissions attribute to use Delegated or Application Permissions --------- Co-authored-by: Gautam Sheth <[email protected]>
pnp · Oct 29, 2024 · 3d6072f · 3d6072f
1 parent 1b8af72
commit 3d6072f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/documentation/Get-PnPTenantInfo.md b/documentation/Get-PnPTenantInfo.md
@@ -12,6 +12,9 @@ online version: https://pnp.github.io/powershell/cmdlets/Get-PnPTenantInfo.html
 ## SYNOPSIS
 Gets information about any tenant
 
+** Required Permissions **
+Graph: CrossTenantInformation.ReadBasic.All
+
 ## SYNTAX
 
 ### Current Tenant (default)

diff --git a/src/Commands/Admin/GetTenantInfo.cs b/src/Commands/Admin/GetTenantInfo.cs
@@ -1,4 +1,5 @@
 using Microsoft.SharePoint.Client;
+using PnP.PowerShell.Commands.Attributes;
 using PnP.PowerShell.Commands.Base;
 using PnP.PowerShell.Commands.Utilities.REST;
 using System;
@@ -7,6 +8,7 @@
 namespace PnP.PowerShell.Commands.Admin
 {
     [Cmdlet(VerbsCommon.Get, "PnPTenantInfo")]
+    [RequiredApiDelegatedOrApplicationPermissions("graph/CrossTenantInformation.ReadBasic.All")]
     [OutputType(typeof(Model.TenantInfo))]
     public class GetTenantInfo : PnPAdminCmdlet
     {
@@ -35,13 +37,13 @@ protected override void ExecuteCmdlet()
             var requestUrl = BuildRequestUrl();
 
             WriteVerbose($"Making call to {requestUrl} to request tenant information");
-            var results = RestHelper.Get<Model.TenantInfo>(Connection.HttpClient, requestUrl, graphAccessToken);
+            var results = GraphHelper.Get<Model.TenantInfo>(this, Connection, requestUrl, graphAccessToken);
             WriteObject(results, true);
         }
 
         private string BuildRequestUrl()
         {
-            var baseUrl = $"https://{Connection.GraphEndPoint}/v1.0/tenantRelationships/";
+            var baseUrl = $"/v1.0/tenantRelationships/";
             var query = string.Empty;
             switch (ParameterSetName)
             {