Merge pull request #2802 from gautamdsheth/bugfix/2775

Fix #2775: issue with complex JSON payload in REST methods

CHANGELOG.md

@@ -66,6 +66,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 - Fixed issue with `Set-PnPContentType` not allowing you to update basic properties of a content type [#2760](https://github.com/pnp/powershell/pull/2760)
 - Fixed `Add-PnPField` not supporting a ReturnType to be set for calculated fields when created on the site level [#2765](https://github.com/pnp/powershell/pull/2765)
 - Fixed issue with `Invoke-PnPSPRestMethod` throwing error when the response string is empty. [#2784](https://github.com/pnp/powershell/pull/2784)
+- Fixed issue with `Invoke-PnPSPRestMethod` and `Invoke-PnPGraphMethod` throwing error when passing complex JSON object as payload.
 - Fixed issue with `Add-PnPListItem` and `Set-PnPListItem` not correctly setting the Purview `Unlocked by default`. [#2800](https://github.com/pnp/powershell/pull/2800)
 
 ### Contributors

src/Commands/Base/InvokeSPRestMethod.cs

@@ -9,6 +9,7 @@
 using System.Net.Http;
 using System.Net.Http.Headers;
 using System.Text.Json;
+using System.Text.Json.Serialization;
 
 namespace PnP.PowerShell.Commands.Admin
 {
@@ -92,7 +93,7 @@ protected override void ExecuteCmdlet()
                         ContentType = "application/json";
                     }
                     var contentString = Content is string ? Content.ToString() :
-                        JsonSerializer.Serialize(Content);
+                        JsonSerializer.Serialize(Content, new JsonSerializerOptions() { ReferenceHandler = ReferenceHandler.IgnoreCycles, WriteIndented = true });
                     request.Content = new StringContent(contentString, System.Text.Encoding.UTF8);
                     request.Content.Headers.ContentType = MediaTypeHeaderValue.Parse(ContentType);
                 }

src/Commands/Graph/InvokeGraphMethod.cs

@@ -7,6 +7,7 @@
 using System.Collections.Generic;
 using System.Net.Http;
 using System.Net.Http.Headers;
+using System.Text.Json.Serialization;
 
 namespace PnP.PowerShell.Commands.Base
 {
@@ -104,7 +105,7 @@ private HttpContent GetHttpContent()
                     ContentType = "application/json";
                 }
                 var contentString = Content is string ? Content.ToString() :
-                    JsonSerializer.Serialize(Content);
+                    JsonSerializer.Serialize(Content, new JsonSerializerOptions() { ReferenceHandler = ReferenceHandler.IgnoreCycles, WriteIndented = true });
 
                 HttpContent httpContent = new StringContent(contentString, System.Text.Encoding.UTF8);
                 httpContent.Headers.ContentType = MediaTypeHeaderValue.Parse(ContentType);