Supporting empty payloads for POST, PATCH and PUT

Author: polterguy

plugins/magic.lambda.http/magic.lambda.http/services/MagicHttp.cs

@@ -391,10 +391,6 @@ static object GetRequestContentContent(
             // Checking if caller supplied a [payload] value or not, alternatives are structured lambda object.
             if (payloadNode.Value == null)
             {
-                // Verifying user supplied some sort of structured lambda object type of content.
-                if (!payloadNode.Children.Any())
-                    throw new HyperlambdaException($"No [payload] value or children supplied to [{input.Name}]");
-
                 // Figuring out Content-Type of request payload to make sure we correctly transform into the specified value.
                 var contentType = headers.ContainsKey("Content-Type") ?
                     ContentType.Parse(headers["Content-Type"]) :
@@ -409,8 +405,7 @@ static object GetRequestContentContent(
             else
             {
                 // [payload] contains a value object of some sort.
-                return payloadNode?.GetEx<object>() ??
-                    throw new HyperlambdaException($"No [payload] value supplied to [{input.Name}]");
+                return payloadNode?.GetEx<object>();
             }
         }
 
@@ -422,7 +417,7 @@ async Task<object> GetRequestFileContentAsync(Node input)
             // If no [content] was given we check if caller supplied a [filename] argument.
             var filename = input.Children.FirstOrDefault(x => x.Name == "filename")?.GetEx<string>();
             if (filename == null)
-                throw new HyperlambdaException($"No [payload] or [filename] argument supplied to [{input.Name}]");
+              return "";
 
             // Caller supplied a [filename] argument, hence using it as a stream content object.
             if (await _fileService.ExistsAsync(_rootResolver.AbsolutePath(filename)))

plugins/magic.lambda.http/magic.lambda.http/services/helpers/RequestTransformers.cs

@@ -29,6 +29,10 @@ internal static object TransformToJson(
             Node payloadNode,
             string slotName)
         {
+            // Short circuiting if we have no nodes.
+            if (!payloadNode.Children.Any())
+                return "";
+
             /*
              * Automatically [unwrap]'ing all nodes, since there are no reasons why you'd want
              * to pass in an expression as JSON token to an endpoint.