Fixed an issue in JsonPolicyWriter.WritePolicyToString() method where it was not using prettyPrint parameter for indentation.

Author: ashishdhingra

generator/.DevConfigs/31d96afb-c4e2-4057-b0cc-14b36797961d.json

@@ -0,0 +1,9 @@
+{
+  "core": {
+    "changeLogMessages": [
+      "Fixed an issue in JsonPolicyWriter.WritePolicyToString() method where it was not using prettyPrint parameter for indentation."
+    ],
+    "type": "patch",
+    "updateMinimum": false
+  }
+}

sdk/src/Core/Amazon.Auth/AccessControlPolicy/Internal/JsonPolicyWriter.cs

@@ -53,7 +53,7 @@ public static string WritePolicyToString(bool prettyPrint, Policy policy)
                 // indentSize is only available in net8+ and net472
                 JsonWriterOptions options = new JsonWriterOptions
                 {
-                    Indented = true,
+                    Indented = prettyPrint
                 };
                 using (var stream = new MemoryStream())
                 using (var generator = new Utf8JsonWriter(stream, options))

sdk/test/UnitTests/Custom/PolicyTests.cs

@@ -249,5 +249,41 @@ public void HandleAnonymousWhenConvertingPrincipals()
             Assert.AreEqual(1, statement.Principals.Count);
             Assert.AreEqual(Principal.Anonymous, statement.Principals.First());
         }
+
+        [TestMethod]
+        public void TestPrettyPrintIndentationDisabled()
+        {
+            string testPolicy = @"{
+                ""Version"": ""2012-10-17"",
+                ""Statement"": [
+                {
+                    ""Sid"": ""AllowS3ListBucket"",
+                    ""Effect"": ""Allow"",
+                    ""Action"": [
+                    ""s3:ListBucket""
+                    ],
+                    ""Resource"": [
+                    ""arn:aws:s3:::your-bucket-name""
+                    ]
+                }
+                ]
+            }";
+
+            var policy = Policy.FromJson(testPolicy);
+            string policyString = policy.ToJson(false);
+
+            Assert.IsFalse(policyString.Contains("\n"));
+        }
+
+        [TestMethod]
+        public void TestPrettyPrintIndentationEnabled()
+        {
+            string testPolicy = @"{""Version"": ""2012-10-17"", ""Statement"": [{""Sid"": ""AllowS3ListBucket"", ""Effect"": ""Allow"", ""Action"": [ ""s3:ListBucket"" ], ""Resource"": [ ""arn:aws:s3:::your-bucket-name""]}]}";
+
+            var policy = Policy.FromJson(testPolicy);
+            string policyString = policy.ToJson(true);
+
+            Assert.IsTrue(policyString.Contains("\n"));
+        }
     }
 }