NpgsqlRest
diff --git a/‎NpgsqlRest/Defaults/DefaultEndpoint.cs‎
Lines changed: 3 additions & 1 deletion b/‎NpgsqlRest/Defaults/DefaultEndpoint.cs‎
Lines changed: 3 additions & 1 deletion
diff --git a/‎NpgsqlRest/MiddlewareExtension.cs‎
Lines changed: 7 additions & 4 deletions b/‎NpgsqlRest/MiddlewareExtension.cs‎
Lines changed: 7 additions & 4 deletions
diff --git a/‎NpgsqlRestTestWebApi/Program.cs‎
Lines changed: 1 addition & 1 deletion b/‎NpgsqlRestTestWebApi/Program.cs‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎NpgsqlRestTestWebApi/appsettings.Development.json‎
Lines changed: 1 addition & 1 deletion b/‎NpgsqlRestTestWebApi/appsettings.Development.json‎
Lines changed: 1 addition & 1 deletion
@@ -90,7 +90,9 @@ internal static class DefaultEndpoint
             CrudType.Delete => Method.DELETE,
             _ => Method.POST
         };
-        var requestParamType = method == Method.GET ? RequestParamType.QueryString : RequestParamType.BodyJson;
+        var requestParamType = method == Method.GET || method == Method.DELETE ? 
+            RequestParamType.QueryString : 
+            RequestParamType.BodyJson;
 
         var originalUrl = url;
         var originalMethod = method;
 
@@ -73,6 +73,7 @@ public static IApplicationBuilder UseNpgsqlRest(this IApplicationBuilder builder
             }
 
             JsonObject? jsonObj = null;
+            IDictionary<string, JsonNode?> bodyDict = default!;
             string? body = null;
 
             var len = tupleArray.Length;
@@ -300,8 +301,6 @@ await options.ValidateParametersAsync(new(
                                 continue;
                             }
                         }
-
-                        var bodyDict = (IDictionary<string, JsonNode?>)jsonObj;
                         bool shouldContinue = false;
                         foreach (var key in bodyDict.Keys)
                         {
@@ -811,10 +810,14 @@ async Task ParseJsonBody()
                     try
                     {
                         jsonObj = node?.AsObject();
+#pragma warning disable CS8600 // Converting null literal or possible null value to non-nullable type.
+                        bodyDict = jsonObj?.ToDictionary();
+#pragma warning restore CS8600 // Converting null literal or possible null value to non-nullable type.
                     }
-                    catch (InvalidOperationException)
+                    catch (Exception e)
                     {
-                        logger?.LogWarning("Could not parse JSON body {0}, skipping path {1}.", body, context.Request.Path);
+                        logger?.LogWarning("Could not parse JSON body {0}, skipping path {1}. Error: {2}", 
+                            body, context.Request.Path, e.Message);
                     }
                 }
             }
 
@@ -96,7 +96,7 @@
 
     SourceCreated = sources =>
     {
-        sources.Add(new CrudSource());
+        sources.Add(new CrudSource()); 
     },
 });
 app.Run();
 
@@ -1,5 +1,5 @@
 {
   "ConnectionStrings": {
-    //"Default": "Host=127.0.0.1;Port=5432;Database=npgsql_rest_test;Username=postgres;Password=postgres"
+    "Default": "Host=127.0.0.1;Port=5432;Database=npgsql_rest_test;Username=postgres;Password=postgres"
   }
 }
Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,7 @@ public static IApplicationBuilder UseNpgsqlRest(this IApplicationBuilder builder`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`JsonObject? jsonObj = null;`
	`76`	`+ IDictionary<string, JsonNode?> bodyDict = default!;`
`76`	`77`	`string? body = null;`
`77`	`78`
`78`	`79`	`var len = tupleArray.Length;`
`@@ -300,8 +301,6 @@ await options.ValidateParametersAsync(new(`
`300`	`301`	`continue;`
`301`	`302`	`}`
`302`	`303`	`}`
`303`		`-`
`304`		`- var bodyDict = (IDictionary<string, JsonNode?>)jsonObj;`
`305`	`304`	`bool shouldContinue = false;`
`306`	`305`	`foreach (var key in bodyDict.Keys)`
`307`	`306`	`{`
`@@ -811,10 +810,14 @@ async Task ParseJsonBody()`
`811`	`810`	`try`
`812`	`811`	`{`
`813`	`812`	`jsonObj = node?.AsObject();`
	`813`	`+#pragma warning disable CS8600 // Converting null literal or possible null value to non-nullable type.`
	`814`	`+ bodyDict = jsonObj?.ToDictionary();`
	`815`	`+#pragma warning restore CS8600 // Converting null literal or possible null value to non-nullable type.`
`814`	`816`	`}`
`815`		`- catch (InvalidOperationException)`
	`817`	`+ catch (Exception e)`
`816`	`818`	`{`
`817`		`- logger?.LogWarning("Could not parse JSON body {0}, skipping path {1}.", body, context.Request.Path);`
	`819`	`+ logger?.LogWarning("Could not parse JSON body {0}, skipping path {1}. Error: {2}",`
	`820`	`+ body, context.Request.Path, e.Message);`
`818`	`821`	`}`
`819`	`822`	`}`
`820`	`823`	`}`
Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,7 @@`
`96`	`96`
`97`	`97`	`SourceCreated = sources =>`
`98`	`98`	`{`
`99`		`- sources.Add(new CrudSource());`
	`99`	`+ sources.Add(new CrudSource());`
`100`	`100`	`},`
`101`	`101`	`});`
`102`	`102`	`app.Run();`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`	`2`	`"ConnectionStrings": {`
`3`		`- //"Default": "Host=127.0.0.1;Port=5432;Database=npgsql_rest_test;Username=postgres;Password=postgres"`
	`3`	`+ "Default": "Host=127.0.0.1;Port=5432;Database=npgsql_rest_test;Username=postgres;Password=postgres"`
`4`	`4`	`}`
`5`	`5`	`}`