Skip to content

Commit 06c2fe2

Browse files
authored
Merge pull request #1184 from SherpasGroup/fix-unauthorized-themes
Catch unauthorized exception when getting tenant themes.
2 parents 4af8a8b + 6e5effc commit 06c2fe2

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

src/lib/PnP.Framework/Provisioning/ObjectHandlers/ObjectHierarchySequenceSites.cs

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -256,9 +256,16 @@ public override TokenParser ProvisionObjects(Tenant tenant, Model.ProvisioningHi
256256
ClientObjectList<Microsoft.Online.SharePoint.TenantManagement.ThemeProperties> tenantThemes = null;
257257
if (TenantExtensions.IsCurrentUserTenantAdmin((ClientContext)tenant.Context))
258258
{
259-
tenantThemes = tenant.GetAllTenantThemes();
260-
tenant.Context.Load(tenantThemes);
261-
tenant.Context.ExecuteQueryRetry();
259+
try
260+
{
261+
tenantThemes = tenant.GetAllTenantThemes();
262+
tenant.Context.Load(tenantThemes);
263+
tenant.Context.ExecuteQueryRetry();
264+
}
265+
catch (ServerUnauthorizedAccessException)
266+
{
267+
WriteMessage("Unauthorized getting tenant themes so skipping.", ProvisioningMessageType.Warning);
268+
}
262269
}
263270

264271
foreach (var sitecollection in sequence.SiteCollections)

0 commit comments

Comments
 (0)