Skip to content

Commit e1b9684

Browse files
committed
fix SAS token usage
1 parent bcbd961 commit e1b9684

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/UmbracoFileSystemProviders.Azure/AzureFileSystem.cs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -615,7 +615,7 @@ private static CloudBlobContainer CreateContainer(CloudBlobClient cloudBlobClien
615615
}
616616

617617
CloudBlobContainer container = cloudBlobClient.GetContainerReference(containerName.ToLowerInvariant());
618-
if (!cloudBlobClient.Credentials.IsSAS)
618+
if (cloudBlobClient.Credentials.IsSAS)
619619
{
620620
// Shared access signatures (SAS) have some limitations compared to shared access keys
621621
// read more on: https://docs.microsoft.com/en-us/azure/storage/common/storage-dotnet-shared-access-signature-part-1
@@ -624,11 +624,16 @@ private static CloudBlobContainer CreateContainer(CloudBlobClient cloudBlobClien
624624
if (isAccountSas)
625625
{
626626
container.CreateIfNotExists();
627+
628+
// permissions can't be set!
627629
}
628630

629-
container.SetPermissions(new BlobContainerPermissions { PublicAccess = accessType });
631+
return container;
630632
}
631633

634+
container.CreateIfNotExists();
635+
container.SetPermissions(new BlobContainerPermissions { PublicAccess = accessType });
636+
632637
return container;
633638
}
634639

0 commit comments

Comments
 (0)