fix for policy allowAllInFolder part

Signed-off-by: Neil South <neil.south@answerdigital.com>

Author: neildsouth

src/S3Policy/PolicyExtensions.cs

@@ -139,7 +139,7 @@ public static Policy ToPolicy(PolicyRequest[] policyRequests)
                         Action = new string[] { "s3:*" },
                         Effect = "Allow",
                         Resource = policyRequests
-                            .Select(pr => System.IO.Path.Join(pr.BucketName, pr.FolderName, "*"))
+                            .Select(pr => $"{pr.BucketName}/{pr.FolderName}/*")
                             .Distinct()
                             .ToArray(),
                     },

src/S3Policy/Tests/Extensions/PolicyExtensionsTest.cs

@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+using Monai.Deploy.Storage.S3Policy.Policies;
 using Newtonsoft.Json;
 
 namespace Monai.Deploy.Storage.S3Policy.Tests.Extensions
@@ -100,6 +101,22 @@ public void ToPolicy_NullFolder_ThrowsException()
             Assert.Throws<ArgumentNullException>(() => PolicyExtensions.ToPolicy("test-bucket", null));
         }
 
+        [Fact]
+        public async Task ToPolicy_Should_Set_Correct_Allow_All_Path()
+        {
+            const string bucketName = "test-bucket";
+            const string payloadId = "00000000-1000-0000-0000-000000000000";
+
+            var policys = new PolicyRequest[] { new PolicyRequest(bucketName, payloadId) };
+
+            var policyMade = PolicyExtensions.ToPolicy(policys);
+
+            Assert.EndsWith(
+                $"{bucketName}/{payloadId}/*",
+                policyMade.Statement.First(p => p.Sid == "AllowAllS3ActionsInUserFolder").Resource?.First());
+
+        }
+
         #endregion ToPolicy
     }
 }